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
nixos/networkmanager: fix merging options #72916
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 get an infinite recursion
after cherry-picking onto nixos-unstable:
error: while evaluating the attribute 'config.system.build.toplevel' at nixos/modules/system/activation/top-level.nix:274:5:
while evaluating 'foldr' at lib/lists.nix:52:20, called from nixos/modules/system/activation/top-level.nix:138:12:
while evaluating 'fold'' at lib/lists.nix:55:15, called from lib/lists.nix:59:8:
while evaluating the attribute 'assertions' at undefined position:
while evaluating anonymous function at lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at lib/modules.nix:336:9:
while evaluating the option `assertions':
while evaluating the attribute 'isDefined' at lib/modules.nix:373:5:
while evaluating the attribute 'values' at lib/modules.nix:362:9:
while evaluating the attribute 'values' at lib/modules.nix:456:7:
while evaluating anonymous function at lib/modules.nix:348:28, called from lib/modules.nix:348:17:
while evaluating 'dischargeProperties' at lib/modules.nix:415:25, called from lib/modules.nix:349:62:
while evaluating the attribute 'condition' at lib/modules.nix:500:14:
while evaluating the attribute 'condition' at lib/modules.nix:500:14:
while evaluating the attribute 'enable' at undefined position:
while evaluating anonymous function at lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at lib/modules.nix:336:9:
while evaluating the option `networking.wireless.enable':
while evaluating the attribute 'isDefined' at lib/modules.nix:373:5:
while evaluating the attribute 'values' at lib/modules.nix:362:9:
while evaluating the attribute 'values' at lib/modules.nix:456:7:
while evaluating anonymous function at lib/modules.nix:348:28, called from lib/modules.nix:348:17:
while evaluating 'dischargeProperties' at lib/modules.nix:415:25, called from lib/modules.nix:349:62:
while evaluating 'dischargeProperties' at lib/modules.nix:415:25, called from lib/modules.nix:421:11:
while evaluating the attribute 'condition' at lib/modules.nix:500:14:
while evaluating the attribute 'condition' at lib/modules.nix:500:14:
while evaluating the attribute 'condition' at lib/modules.nix:500:14:
while evaluating the attribute 'networking.wireless.enable' at undefined position:
while evaluating anonymous function at lib/modules.nix:75:45, called from undefined position:
while evaluating the attribute 'value' at lib/modules.nix:336:9:
infinite recursion encountered, at lib/modules.nix:336:9
Incorrect merging of modules resulted in dhcpcd being enabled causing flaky network connection. NixOS#64364 Fixing it uncovered an infinite recursion from the same commit, previously masked by the incorrect merge. We can just drop the `mkDefault` for `networking.wireless.enable` as it is already `false` by default. Closes: NixOS#72416
819b9c1
to
894fdfa
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.
stopping the following units: dhcpcd.service
👍
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 to me, the inf rec resolution of removing the wireless.enable = false
also makes sense after thinking about it.
aha! Yes, I've been having some problems and had to manually stop dhcpcd when using NetworkManager. But since that problem only appeared after the iwd changes were merged, I assumed the problem must lie somewhere else. |
I'm brewing up some changes to the module system such that this can't happen anymore |
[needs: port to stable] can be removed, i suppose. It is hard to see what actually needs port to stable... |
Sorry, forgot to mention. It was backported in #72951 |
Incorrect merging of modules in #64364 resulted in dhcpcd being enabled causing flaky network connection.
Fixing it uncovered an infinite recursion from the same commit, previously masked by the incorrect merge.
We can just drop the
mkDefault
fornetworking.wireless.enable
as it is alreadyfalse
by default.Closes: #72416
cc @laikq, @alexeymuranov (reporters), @JohnAZoidberg (cause of breakage), @worldofpeace (backporter)