-
Notifications
You must be signed in to change notification settings - Fork 592
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
chore(ci): reboot after dist upgrade in fabfile #13503
chore(ci): reboot after dist upgrade in fabfile #13503
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
Oops! Looks like you failed the Howto
♻️ Updated: ✅ The check is passing the Python Format Check after the last commit. |
Python formatting is wrong: |
fab does do a dist upgrade, but that only has full effect if there is a reboot afterwards (e.g. to load new kernel modules). Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
25959ec
to
b636f83
Compare
I think here it would be good to do an integration test run on your fork before the merge. |
I have done that before creation of the PR |
Summary
fab does do a dist upgrade, but that only has full effect if there is a
reboot afterwards (e.g. to load new kernel modules).
Test Plan
tbd.