-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
[WIP] iwd: 0.22 -> 1.0 #72321
[WIP] iwd: 0.22 -> 1.0 #72321
Conversation
ell: 0.24 -> 0.26
Please don't merge quite yet-- they changed details in the dbus interface, so this update breaks iwd usage with NetworkManager. I put together this fix which does the job for me, but would ideally have something more official and perhaps more complete ;). The #iwd channel on freenode has some chat about this today, but haven't been following too closely (yet). |
Also at some point I had to add this, I believe it's in the 1.0 release:
And we may consider teaching NixOS to prefer a certain value for persistent naming when using iwd (they want it off). |
Oh and if we currently do much for generating config, they changed the keys to be camelcase and often rewording. It's not suitable for nixpkgs master yet, but my iwd module has a bit more beef in this area, and demonstrates what I mean: |
Thank you both for doing this work! |
Oooh, we should try to get some of that in nixpkgs. I believe people have been having issues because there's no config installed for iwd, and dhcp woes/resolvconf. |
What exactly is broken. I am using this right now with networkmanager and have not had any issues for the last few days. Also, I set |
note, it looks like the upcoming NM release has a fix for IWD 1.0. Not sure if it's relevant here or not, but seemed like a shot: NetworkManager/NetworkManager@186d22a |
|
I thought this was blocked because of incompatibilities with Network Manager's API.. Does it mean NM's users who use IWD will get disconnected in their next |
Well I don't think NM was patched. They can still roll back though, so its not too bad I guess. |
The Networkmanager patch could be cherry-picked. |
Yes, and the NM patch is also backwards compatible (an improvement on what I originally put together and linked earlier), so safe to pick. If there's not a tagged release with this yet, of course :). |
The fix is in https://gitlab.freedesktop.org/NetworkManager/NetworkManager/blob/1.20.6/NEWS#L9 which there's a PR for #73598. It will have to to staging though |
Going to close as obsolete. Sorry about a committer pushing this, duplicated work is frequent in nixpkgs unfortunately. |
just came here because I got hit by this issue. The NetworkManager upgrade hasn't left the staging branch yet. |
So NetworkManager was patched to staging while iwd was updated in |
Correct.
|
If you downgrade ell too with IWD and you can get back up and running.
That's what I did.
…On Thu, Nov 28, 2019, 05:21 zimbatm ***@***.***> wrote:
Correct.
I tried reverting the IWD upgrade. This doesn't work, there were some
build failures, I don't remember the details. Presumably some other
packages are now depending on the latest IWD.
I tried cherry-picking the NetworkManager upgrade. Now I get this new
error instead:
Nov 27 15:00:05 no1 NetworkManager[1026]: <warn> [1574863205.7424] device (wlp5s0): No agents were available for this request.
Nov 27 15:00:05 no1 NetworkManager[1026]: <error> [1574863205.7425] device (wlp5s0): Activation: (wifi) Network.Connect failed: GDBus.Error:net.connman.iwd.Aborted: Operation aborted
Nov 27 15:00:05 no1 NetworkManager[1026]: <warn> [1574863205.7433] device (wlp5s0): Activation: failed for connection 'Leazero'
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#72321?email_source=notifications&email_token=AACP25EHKL56DFQ6LDTCM73QV6SUVA5CNFSM4JG7OIE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFMJRYI#issuecomment-559454433>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACP25AQ5HJ7WFVS2MAETPTQV6SUVANCNFSM4JG7OIEQ>
.
|
ell: 0.24 -> 0.26
Notify maintainers
cc @dtzWill @Mic92