Dan Kenigsberg has posted comments on this change.
Change subject: rename rhevm bridge to brVDSM ......................................................................
Patch Set 7: I would prefer that you didn't submit this
(5 inline comments)
please not that vdsm_cli/vdscli.py.in:49 still has
proposed_addr = netinfo.ifconfig()['rhevm']['addr']
which should be changed, too. It would much nicer if clientIF._getServerIP() is moved to netinfo, and used by vdscli (in a separate patch).
.................................................... File vds_bootstrap/vds_bootstrap.py.in Line 598: #add @VDSMBRIDGE@ bridge all replacements in this file is of logging strings and comments. Do we really need autoconf for that? wouldn't it be more economical to say "management bridge" instead?
.................................................... File vdsm_reg/deployUtil.py.in Line 87: REMOTE_SSH_KEY_FILE = "/engine.ssh.key.txt" wouldn't this kill registration to RHEV-3.0 ?
Line 559: if key != '\n' and not key.endswith("== ovirt-engine") or key.startswith("#"): this has to be a well-defined constant. and again, this change kills registration to rhev-m-3.0.
Line 779: "supported! Please configure " + MGT_BRIDGE_NAME + " bridge manually and re-install.") now it's easy to break this line!
Line 1421: parser.add_option("-d", "--download-" + MGT_BRIDGE_NAME + "-cert", action="store_true", that's "engine" here!
and it must be the same as in the caller script vdsm_reg/vdsm-config
python "$DEPLOY_UTIL" --download-rhevm-cert --server-address="$vdc_managment_server" --server-port="$vdc_managment_port" --fingerprint="$management_server_fingerprint"
-- To view, visit http://gerrit.ovirt.org/311 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Id1c98658af80ad52a14934a962be4945c91ff48d Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Roy Golan rgolan@redhat.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Chris Wright chrisw@sous-sol.org Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Roy Golan rgolan@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com