-
Notifications
You must be signed in to change notification settings - Fork 789
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade etcd to version 3.4 #894
Conversation
Thank you for this PR @lazzarello Please find some comments inline. |
@ktsakalozos trying to build a version with changes to the install hook but I'm getting a false positive error. I'm new to the snapcraft build process so apologies in advance if I'm missing something obvious.
|
@lazzarello what i do if i get this is to do a clean first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The check ideally should be in the configure
hook instead of install
.
There are certain helper functions in the util.sh
where you can add or remove options.
To add "options"
refresh_opt_in_config <option name> <value> <file>
In your case it will look like this refresh_opt_in_config enable-v2 true etcd
Give it a try.
Thanks for the comments. I'll follow up on this but it's got a little lower in priority on my side. update got some more time. I'll take this one over the finish line. |
@ktsakalozos lemme know if you can get to a review. CI build tests pass now. |
97899ec
to
de098b6
Compare
de098b6
to
b363aa6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM +1 Thank you @lazzarello
Changes the default version of etcd to 3.4 and forces API version 2 for Flanneld.