Douglas Schilling Landgraf has uploaded a new change for review.
Change subject: vds_bootstrap: search if vdsm is already installed ......................................................................
vds_bootstrap: search if vdsm is already installed
If vdsm is already installed into the machine, let's locate it using "rpm -q (getPackageInfo())" instead of use yumSearch() or yumSearchVersion(). The current behaviour: if yum cannot find vdsm into the repo (even if vdsm is already installed into the system) the node installation will fail. With this patch, if the system already has vdsm, it will continue the installation and will jump the yum functions for vdsm.
Change-Id: I82beb5599225cfd3aa98ac7aa23b7152493421a9 --- M vds_bootstrap/vds_bootstrap.py 1 file changed, 15 insertions(+), 7 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/40/1240/1 -- To view, visit http://gerrit.ovirt.org/1240 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I82beb5599225cfd3aa98ac7aa23b7152493421a9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf dougsland@redhat.com
Douglas Schilling Landgraf has posted comments on this change.
Change subject: vds_bootstrap: search if vdsm is already installed ......................................................................
Patch Set 1: Verified
-- To view, visit http://gerrit.ovirt.org/1240 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I82beb5599225cfd3aa98ac7aa23b7152493421a9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com
Dan Kenigsberg has posted comments on this change.
Change subject: vds_bootstrap: search if vdsm is already installed ......................................................................
Patch Set 1: I would prefer that you didn't submit this
Douglas, the yum search is intentional. Bootstrapping has to make sure that the host would get vdsm updates when they are ready. This is done by verifying that a repository with vdsm exists.
Thanks, Dan.
-- To view, visit http://gerrit.ovirt.org/1240 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I82beb5599225cfd3aa98ac7aa23b7152493421a9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com
Douglas Schilling Landgraf has abandoned this change.
Change subject: vds_bootstrap: search if vdsm is already installed ......................................................................
Patch Set 1: Abandoned
The bootstrapping design requires a repository with vdsm exists.
-- To view, visit http://gerrit.ovirt.org/1240 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon Gerrit-Change-Id: I82beb5599225cfd3aa98ac7aa23b7152493421a9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com
vdsm-patches@lists.fedorahosted.org