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@redhat.com Gerrit-Reviewer: Antoni Segura Puimedon asegurap@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server