I suggest INFO message should be only logged when port admin state is disabled, not when it is enabled.
There is already an INFO message in OVS neutron agent logged upon change of admin state containing the state - it is in the details of "Port X updated. Details: {...}" message. The confusion in this bug possibly comes from the fact that the admin state is somewhat hidden in the long details map.
Adding extra INFO message only for the case when admin state is disabled would remove the confusion about why port VLAN tag is set dead vlan (4095). Logging extra INFO message upon enabling admin state does not seem useful as there would still be (already existing) message about port update.
I suggest INFO message should be only logged when port admin state is disabled, not when it is enabled.
There is already an INFO message in OVS neutron agent logged upon change of admin state containing the state - it is in the details of "Port X updated. Details: {...}" message. The confusion in this bug possibly comes from the fact that the admin state is somewhat hidden in the long details map.
Adding extra INFO message only for the case when admin state is disabled would remove the confusion about why port VLAN tag is set dead vlan (4095). Logging extra INFO message upon enabling admin state does not seem useful as there would still be (already existing) message about port update.