Skip to content
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.

Sign up for GitHub

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

Jump to bottom

updatehub.go: UpgradeToInstallation is not being set after an update #197

Closed
texierp opened this issue Mar 27, 2018 · 2 comments · Fixed by #198
Closed

updatehub.go: UpgradeToInstallation is not being set after an update #197

texierp opened this issue Mar 27, 2018 · 2 comments · Fixed by #198

Comments

@texierp
Copy link

texierp commented Mar 27, 2018 •
edited
Loading

On the agent start, both validate and rollback states are ignored
when the "UpgradeToInstallation" is < 0). So even if an new installation
is achieved, UpgradeToInstallation is equal to -1, example:

... "update": { "download-dir": "/tmp/updatehub", "supported-install-modes": [ "tarball" ], "upgrade-to-installation": -1 },
@otavio
Copy link
Member

otavio commented Mar 27, 2018

@texierp are you willing to take a look at this issue?

@texierp
Copy link
Author

texierp commented Mar 27, 2018

Yes @otavio no problem

texierp added a commit to texierp/updatehub that referenced this issue Mar 27, 2018
@texierp
fixes load and update of UpgradeToInstallation variable
0b144da 
Without this commit, the agent code reading 'UpgradeToInstallation'
from '/etc/updatehub.conf' and not from persistents data (/data)

Also fixes the update of 'UpgradeToInstallation' after an update.

Fixes UpdateHub#197

Signed-off-by: Pierre-Jean TEXIER <texier.pj2@gmail.com>
@texierp texierp mentioned this issue Mar 27, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fixes load and update of UpgradeToInstallation variable texierp/updatehub
2 participants
@otavio @texierp