-
Notifications
You must be signed in to change notification settings - Fork 71
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
Refactor the usage of packages keyword in cloud-init, to avoid auto upgrade of packages #1705
Conversation
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.
I think these need -y as well like some apt commands below
46e475d
to
9ec7095
Compare
Something still wrong for Ubuntu2204 VM and SLES15SP4 BuildHost VM. |
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.
see remark about runcmd: sections
9ec7095
to
d32fab0
Compare
d32fab0
to
b2007a2
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.
Looks good now
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.
see nitpicks + 1 question
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.
thanks for the nitpick corrections
What does this PR change?
Don't upgrade packages in the systems deployed in a SUMA/Uyuni test environment.
The cloud team added the module
package-update-upgrade-install
to SUSE cloud-init configuration. This triggers an not-desired update of packages in some of the systems part of our test environment.This is triggered due to the usage of the keyword
packages
in our YAML file.This PR replace the keyword
packages
byruncmd zypper in
Example: