-
Notifications
You must be signed in to change notification settings - Fork 148
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: deprecate RELEASE upgrade, fix thick/thin jail upgrades #811
upgrade: deprecate RELEASE upgrade, fix thick/thin jail upgrades #811
Conversation
@michael-o What about integrating a switch to "auto continue" the upgrade process? Right now it's three commands to upgrade fully. With a switch it could do all 3 in a row. |
I'd keep it as is for the moment and consider something like |
@michael-o Update and upgrade PRs have been updated with your suggestions. Also, are you ok with deprecating 'upgrade RELEASE'? It not even wise IMO as it would end up breaking thin jails. If we want a new release, we can simply bootstrap it. Otherwise (in the case of thick jails) an 'upgrade jailname' will take care of it. |
Upgrading a RELEASE doesn't make sense, I do agree. Deprecate it. Bootstrap and remount is better. |
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.
This is logically sound to me now.
Does it test ok? |
/var/db/etcupdate/current/etc/ttys Please run 'bastille upgrade thick1 install' to finish installing updates. this error when upgrading a thick jail. |
Will resume on Monday |
now testing correctly for me. Need @michael-o to finish his testing on this next week. |
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.
Works as requested. One nit:
Please run 'bastille upgrade deblndw013x4j install' to finish installing updates.
root@deblndw013x:~/bastille (upgrade_deprecate_release *%<>)
# bastille upgrade deblndw013x4j install
src component not installed, skipped
Installing updates...
Kernel updates have been installed. Please reboot and run
'freebsd-update [options] install' again to finish installing updates.
root@deblndw013x:~/bastille (upgrade_deprecate_release *%<>)
# bastille upgrade deblndw013x4j install
src component not installed, skipped
Installing updates...Scanning /usr/local/bastille/jails/deblndw013x4j/root/usr/share/certs/blacklisted for certificates...
Scanning /usr/local/bastille/jails/deblndw013x4j/root/usr/share/certs/trusted for certificates...
done.
Bastille says to run install, but after kernel updates it does not tell to run install again.
In any case, it works.
Deprecate 'bastille upgrade RELEASE'
bastille bootstrap release
Fix #658
Implement #621 #481
Fix #372
Testing
Create and upgrade a thick jail eg (13.4-RELEASE - 14.2-RELEASE) - verify functionality and updates
Create and upgrade a thin jail - verify functionality and updates (all this does is "sed" the jail.conf and fstab to point to the selected release