Change in vdsm[master]: vdsm.spec.in/vdsm.rwtab: create /rhev repository

danken at redhat.com danken at redhat.com
Mon Dec 19 19:55:07 UTC 2011


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm.spec.in/vdsm.rwtab: create /rhev repository
......................................................................


Patch Set 3: (1 inline comment)

I do not think that /rhev/ matters to Vdsm much without the data-center after that. please consider using them together.

....................................................
File configure.ac
Line 54: AC_SUBST([vdsmrepo], ['rhev'])
both places should be fixed - there's nothing in /rhev proper but data-center; I think that keeping /rhev/data-center as a single string, fits nicer with the plans to move to /var/run/vdsm/bla in some point in the future. 

Would putting /rhev/data-center in vdsm.rwtab work for us?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I971a7b47e834d7b6b95874ff6437e66b1928fc82
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list