Antoni Segura Puimedon has posted comments on this change.
Change subject: confNet: ifcfg's VLAN should be set only on vlan cfg
......................................................................
Patch Set 1: (1 inline comment)
small typo fix in the commit message.
....................................................
Commit Message
Line 6:
Line 7: confNet: ifcfg's VLAN should be set only on vlan cfg
Line 8:
Line 9: If a misguided script calls addNetwork with explicit VLAN option, it
Line 10: sohuld not be blindly copied to the top-most device.
s/sohuld/should/
Line 11: We set VLAN=yes on the ifcfg script for the vlan device, and that's
Line 12: that.
Line 13:
Line 14: Change-Id: Ia2c9d6c86e73104424ca8ac901bf09e4b98a47ab
--
To view, visit
http://gerrit.ovirt.org/7411
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2c9d6c86e73104424ca8ac901bf09e4b98a47ab
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server