Ido Barkan has posted comments on this change.

Change subject: split network restoration from vdsmd.service
......................................................................


Patch Set 13:

(1 comment)

https://gerrit.ovirt.org/#/c/29441/13/vdsm/vdsm-restore-net-config
File vdsm/vdsm-restore-net-config:

Line 172:                       help=force_option_help)
Line 173: 
Line 174:     options, args = parser.parse_args()
Line 175: 
Line 176:     if len(args) < 1 and not options.force:
the path to the resotre_nets mark can be internal here
Line 177:         parser.error('unless passing --force, %s must be supplied 
with the '
Line 178:                      'path of a file-mark for testing if networks 
where '
Line 179:                      'already restored.' % (parser.get_prog_name()))
Line 180: 


-- 
To view, visit https://gerrit.ovirt.org/29441
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I70cdf1e8a1e7063ec850081ac6199728d4cde381
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <toni+ov...@midokura.com>
Gerrit-Reviewer: Antoni Segura Puimedon <toni+ov...@midokura.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to