| Summary: | kded module fails to connect to NM on NM restarts | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Fabian Vogt <fabian> |
| Component: | Networking in general | Assignee: | Jan Grulich <jgrulich> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | anidev.aelico |
| Priority: | NOR | ||
| Version First Reported In: | master | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/networkmanager-qt/fa472e6a5e2637a681f61e75cf0988bf847f46ed | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
|
Description
Fabian Vogt
2018-10-27 09:52:31 UTC
Attempt at fixing this: https://phabricator.kde.org/D16471 Just want to chime and confirm I have this exact issue with kde 5.51.0 / Qt 5.11.2 plasma-nm 5.14.2 / NetworkManager 1.14.0, on OpenSUSE. Happens everytime NM is restarted, which in my case happens when I update the package. I've had this issue for a while now. Also seems like #398524 is the same issue. Git commit fa472e6a5e2637a681f61e75cf0988bf847f46ed by Fabian Vogt. Committed on 06/11/2018 at 19:17. Pushed by fvogt into branch 'master'. Listen for added DBus interfaces instead of registered services Summary: When the service is registered, the interfaces might not be available yet. So wait for those instead. This was RFC because: - I'm not sure whether it's guaranteed that InterfacesAdded is emitted on service registration as well or both connections are necessary - I don't know how it's possible to test this as org.kde.fakenetwork does not support the org.freedesktop.ObjectManager API Test Plan: Restarting NM while kded5 keeps running caused "nmcli up" to fail because of missing secrets. With this patch, it works successfully. Reviewers: #frameworks, jgrulich Reviewed By: jgrulich Subscribers: kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D16471 M +16 -8 src/manager.cpp M +3 -1 src/manager_p.h M +12 -3 src/secretagent.cpp M +1 -0 src/secretagent.h M +1 -1 src/secretagent_p.h https://commits.kde.org/networkmanager-qt/fa472e6a5e2637a681f61e75cf0988bf847f46ed Bulk transfer as requested in T17796 |