Change in vdsm[master]: vdsm-tool: add node register tool

Alon Bar-Lev alonbl at redhat.com
Fri Dec 20 15:53:35 UTC 2013


Alon Bar-Lev has posted comments on this change.

Change subject: vdsm-tool: add node register tool
......................................................................


Patch Set 10:

Registration has nothing to do with vdsm but with engine. Setting up ssh keys or establish engine web ca trust has nothing to do with vdsm.
Also, registration can trigger different component deployment, such as gluster or open stack.

If you want to pull that component as vdsm dependency you are entitled to.

If you want to collapse it to vdsm, also ok... although as I wrote above I do not understand the connection.

In any case the implementation should provide what the initial design states: python api, compatibility with newer and older engines, use the new registration protocol, support http redirect, etc...

-- 
To view, visit http://gerrit.ovirt.org/22494
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I65b508d2a365108950c0758134a6d9c024d04f2c
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list