Alon Bar-Lev has posted comments on this change.
Change subject: bootstrap: send complete bootstrap from engine ......................................................................
Patch Set 1:
Again,
Not sure I follow, this is far from this patch discussion.
If you want alternate location with configuration it can be added in another patch.
This patch keeps current configuration sane, while making the necessary fixes that were on the table.
If we are to put the outputs in different location we should bass this location to the bootstrap script, and make sure we clean after ourselves.
While /tmp is self cleaned and accessible for anyone, see FHS spec.
I don't buy the tmpfs nor selinux arguments:
1. selinux that will not allow user to access /tmp violates the FHS spec.
2. tmpfs should be large enough to contain 500K, as it should be used by sane system, had we used large data, I would have agreed to use /var/tmp instead, but these are the only locations to put such craft.
-- 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@redhat.com Gerrit-Reviewer: Alon Bar-Lev alonbl@redhat.com Gerrit-Reviewer: Barak Azulay bazulay@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Doron Fediuck dfediuck@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Eyal Edri eedri@redhat.com Gerrit-Reviewer: Juan Hernandez juan.hernandez@redhat.com Gerrit-Reviewer: Ofer Schreiber oschreib@redhat.com