Bug 2065637

Summary:	Snapshot rollback does not respect managed no set by nmcli
Product:	Red Hat Enterprise Linux 8	Reporter:	Filip Pokryvka <fpokryvk>
Component:	NetworkManager	Assignee:	NetworkManager Development Team <nm-team>
Status:	CLOSED MIGRATED	QA Contact:	Desktop QE <desktop-qa-list>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	8.6	CC:	bgalvani, lrintel, rkhan, sfaye, sukulkar, till
Target Milestone:	rc	Keywords:	MigratedToJIRA, Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2023-08-16 19:42:05 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Filip Pokryvka 2022-03-18 11:42:58 UTC

Description of problem:
There are following two runs of the test libnm_snapshot_reattach_unmanaged_ports_to_bridge, once device is set unmanaged by udev [1], and once it is additionally set unmanaged by `nmcli dev set portXb managed no` [2]. [1] serves as valid reproducer for bz2035519, but unmanaging device by nmcli makes the test pass (that command should have no effect).  

Version-Release number of selected component (if applicable):
1.36.0-2.el8

How reproducible:
always

Steps to Reproduce:
1. run the test libnm_snapshot_reattach_unmanaged_ports_to_bridge
2. uncoment test lines to unmanage ports via nmcli
3. run the test again

Actual results:
step 1. is failing, while step 3. is passing, but 2. should have no effect as device is already unmanaged

Expected results:
Both should fail on affected version, or both should pass on fixed version

Additional info:
[1] https://tools.dqe.lab.eng.bos.redhat.com/fpokryvk/reports/report_NetworkManager-ci_Test0_libnm_snapshot_reattach_unmanaged_ports_to_bridge.html
[2] https://tools.dqe.lab.eng.bos.redhat.com/fpokryvk/reports/report_NetworkManager-ci_Test0_libnm_snapshot_reattach_unmanaged_ports_to_bridge_managed_no.html

(reports are from rhel9, but the same results on rhel8)