-
Notifications
You must be signed in to change notification settings - Fork 371
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
Cleanup SUSE config #432
Cleanup SUSE config #432
Conversation
Hi @bear454, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! TTYL, MSBOT; |
Hi All, What's holding up this commit? If it's the AutoUpdate.Enabled=n change then we could possible revert that part, but clean up the rest. Steve |
# Enable RDMA management and set up, should only be used in HPC images | ||
# OS.EnableRDMA=y | ||
|
||
# Enable or disable goal state processing auto-update, default is enabled | ||
# AutoUpdate.Enabled=y | ||
AutoUpdate.Enabled=n |
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.
@bear454 We need to leave AutoUpdate.Enabled
set to true (aka "y"). It is best for Azure customers.
@szarkos Just other things going on and have not looked. I agree with reverting the change to |
1d4d383
to
a784ee1
Compare
Okay... |
@bear454 Looks good. |
re: BSC#997475 - waagent.conf in python-azure-agent package should use ext4 for ephemeral disk filed by @szarkos
While correcting the SUSE config, a few other omissions were noted.
WRT to 1d4d383, this change is currently being made as part of image generation; moving to the config file reduces the number of times we need to touch
/etc/waagent.conf
.