Change in vdsm[master]: BZ#729251 vds_bootstrap: Removed a reference to an obselete ...

danken at redhat.com danken at redhat.com
Wed Sep 7 13:26:36 UTC 2011


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#729251 vds_bootstrap: Removed a reference to an obselete netconsole option.
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

Thanks for your patch, but I think that we should not remove -n as long as RHEV-M is capable of sending it.

....................................................
File vds_bootstrap/vds_bootstrap.py
Line 795:         opts, args = getopt.getopt(sys.argv[1:], "r:O:t:n:u:")
keeping n: here is not nice - but it can be done only after fixing RHEV-M's backend/manager/modules/bll/src/main/java/org/nogah/bll/VdsInstaller.java

--
To view, visit http://gerrit.usersys.redhat.com/909
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie0b90a78f7c9c49af7b70e50db71f6621274ad4f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer <ghammer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>


More information about the vdsm-patches mailing list