On 2/3/22 15:45, Laine Stump wrote: > On 2/2/22 1:04 PM, Michal Prívozník wrote: >> >> Laine, any thoughts? > > I somehow thought this had already been pushed, so I was surprised when > it showed up again :-/ > > I think the only issue I had before was that I thought the new unit > tests were more testing the test setup than the actual code, but Dan > convinced me otherwise. > > So I'm fine with this change. > Cool, pushed now. > A newly discovered (but pre-existing, so non-consequential to these > patches) problem associated with vlans is that we don't differentiate > between "set vlan 0" and "don't set any vlan", which I hadn't even > considered before, until this BZ came up: > > https://bugzilla.redhat.com/2035726 > > (The reporter is trying to use <tag id='-1'/> to "unset" the vlan tag > for an interface) > Ah, but the bug report is for openvswitch port profile, so that's doubly unrelated :-) Michal
