-
Notifications
You must be signed in to change notification settings - Fork 314
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
[sup] Install a package only if missing for start
& load
.
#3425
[sup] Install a package only if missing for start
& load
.
#3425
Conversation
This resolves an issue in `hab-sup start` and `hab-sup load` where the Supervisor was unconditionally attempting to install a package saving the Service spec. Without an explicit update strategy, the Supervisor should find an load the latest version of an installed package on disk. Otherwise, newer releases of a package will be download and installed whenever a new candidate is present in the Builder API. Signed-off-by: Fletcher Nichol <fnichol@nichol.ca>
Thanks for the pull request! Here is what will happen next:
Thank you for contributing! |
@fnichol Good catch! @thesentinels approve |
🤘 I am testing your branch against master before merging it. We do this to ensure that the master branch is never failing tests. |
Travis CI has started testing this PR. |
💖 Travis CI reports this PR passed. It always makes me feel nice when humans approve of one anothers work. I'm merging this PR now. I just want you and the contributor to answer me one question: |
@fnichol should we patch release this? |
@reset Yes: without this any Docker images would install new versions on boot, and services in production would likely upgrade themselves starting back up from a |
This resolves an issue in
hab-sup start
andhab-sup load
where theSupervisor was unconditionally attempting to install a package saving
the Service spec. Without an explicit update strategy, the Supervisor
should find an load the latest version of an installed package on disk.
Otherwise, newer releases of a package will be download and installed
whenever a new candidate is present in the Builder API.
Signed-off-by: Fletcher Nichol fnichol@nichol.ca