-
Notifications
You must be signed in to change notification settings - Fork 2
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
[APP-6881] Add disable wifi power save option #43
Conversation
if isNew { | ||
w.logger.Infof("Updated %s to: %q", wifiPowerSaveFilepath, contents) | ||
// Reload NetworkManager to apply changes | ||
if err := w.nm.Reload(0); err != nil { |
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 confirmed looking at network manager logs that the config file logs without warning or errors
@Otterverse I can only get the change to apply by actually rebooting the device. Rebooting NetworkManager is not enough. Maybe am I missing something? I tried:
and while i see on the networkmanager logs I don't really see powersave changing. Rebooting the pi does work. |
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.
LGTM! I think we can get a beta out tomorrow and start some testing.
1b455bd
to
bdcc338
Compare
bdcc338
to
16c1aba
Compare
No description provided.