Skip to content
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

NM/device: Handle activation failures for connections #201

Merged
merged 1 commit into from Dec 19, 2018

Conversation

tyll
Copy link
Member

@tyll tyll commented Dec 18, 2018

When a connection is activated, the user data for the active connection
callback does not contain a device object. Therefore report the
connection ID instead of the device instead.

Signed-off-by: Till Maas opensource@till.name

fixes #198

@tyll tyll requested review from cathay4t and EdDev December 18, 2018 23:09
@coveralls
Copy link

coveralls commented Dec 18, 2018

Coverage Status

Coverage decreased (-0.4%) to 68.848% when pulling 345c222 on tyll:activate_connection into db1ac7a on nmstate:master.

libnmstate/nm/device.py Outdated Show resolved Hide resolved
@tyll tyll force-pushed the activate_connection branch 2 times, most recently from c15f61b to ff2c35f Compare December 19, 2018 10:11
libnmstate/nm/device.py Show resolved Hide resolved
libnmstate/nm/device.py Outdated Show resolved Hide resolved
@tyll tyll force-pushed the activate_connection branch 3 times, most recently from 08cc600 to 502e0b9 Compare December 19, 2018 13:34
When a connection is activated, the user data for the active connection
callback does not contain a device object. Therefore report the
connection ID instead of the device instead.

Signed-off-by: Till Maas <opensource@till.name>
@tyll
Copy link
Member Author

tyll commented Dec 19, 2018

ci test please

@tyll tyll merged commit a3721fe into nmstate:master Dec 19, 2018
@tyll tyll deleted the activate_connection branch December 19, 2018 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

_active_connection_callback() wrongfully assumes that only devices are activated
3 participants