Douglas Schilling Landgraf has uploaded a new change for review.
Change subject: BZ#832577: node can't be approved ......................................................................
BZ#832577: node can't be approved
Because of ovirt-node's readonly filesystem, directory creation at runtime has to be handled carefully. /rhev/data-center used to be created by the vdsm rpm when it is installed. It's now listed in vdsm.spec as %ghost which means it is *not* laid down by default. At runtime, it fails to be created because of the read-only filesystem.
Regression introduced by commit ee1e68d3416d8fd728df75c0a41dd3db48f9138d Patch provided by: Mike Burns mburns@redhat.com
Change-Id: I3818661c886118d34620e5308434e57bad92913f Signed-off-by: Douglas Schilling Landgraf dougsland@redhat.com --- M vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/79/5379/1 -- To view, visit http://gerrit.ovirt.org/5379 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I3818661c886118d34620e5308434e57bad92913f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf dougsland@redhat.com
Michael Burns has posted comments on this change.
Change subject: BZ#832577: node can't be approved ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/5379 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I3818661c886118d34620e5308434e57bad92913f 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: Michael Burns mburns@redhat.com
Dan Kenigsberg has posted comments on this change.
Change subject: BZ#832577: node can't be approved ......................................................................
Patch Set 1: Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/5379 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I3818661c886118d34620e5308434e57bad92913f 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: Michael Burns mburns@redhat.com Gerrit-Reviewer: Xu He Jie xuhj@linux.vnet.ibm.com
Michael Burns has posted comments on this change.
Change subject: BZ#832577: node can't be approved ......................................................................
Patch Set 1: Verified
-- To view, visit http://gerrit.ovirt.org/5379 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I3818661c886118d34620e5308434e57bad92913f 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: Michael Burns mburns@redhat.com Gerrit-Reviewer: Xu He Jie xuhj@linux.vnet.ibm.com
Dan Kenigsberg has submitted this change and it was merged.
Change subject: BZ#832577: node can't be approved ......................................................................
BZ#832577: node can't be approved
Because of ovirt-node's readonly filesystem, directory creation at runtime has to be handled carefully. /rhev/data-center used to be created by the vdsm rpm when it is installed. It's now listed in vdsm.spec as %ghost which means it is *not* laid down by default. At runtime, it fails to be created because of the read-only filesystem.
Regression introduced by commit ee1e68d3416d8fd728df75c0a41dd3db48f9138d Patch provided by: Mike Burns mburns@redhat.com
Change-Id: I3818661c886118d34620e5308434e57bad92913f Signed-off-by: Douglas Schilling Landgraf dougsland@redhat.com --- M vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Michael Burns: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/5379 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: I3818661c886118d34620e5308434e57bad92913f 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: Michael Burns mburns@redhat.com Gerrit-Reviewer: Xu He Jie xuhj@linux.vnet.ibm.com
vdsm-patches@lists.fedorahosted.org