Change in vdsm[master]: bootstrap: send complete bootstrap from engine

dfediuck at redhat.com dfediuck at redhat.com
Tue Aug 7 19:54:47 UTC 2012


Doron Fediuck has posted comments on this change.

Change subject: bootstrap: send complete bootstrap from engine
......................................................................


Patch Set 1:

The current /tmp used in the Java must be replaced, and you got the patch for it. Even if you do not use it now, you still shouldn't be forcing other components to use only that.

As for /tmp in the host, it may be mounted somewhere else, with permissions / selinux restrictions, including tmpfs with size limitations. So for all these cases, we made sure /tmp will work, but going forward, there's no reason to keep hard-coding it.

The above are all non-theoretical as well. All are cases I faced in the past, and bz's opened to fix it.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I01ddee19cbb65341ed22435ed9e735a0ad3c2abd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>


More information about the vdsm-patches mailing list