Change in vdsm[master]: add configure variable vdsmcoredumpdir for /var/log/core

danken at redhat.com danken at redhat.com
Mon Jun 11 09:16:02 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: add configure variable vdsmcoredumpdir for /var/log/core
......................................................................


Patch Set 2:

I feel that we have a problem of discommunication... Probably gerrit comments are not the best for of interaction.

I don't recall that *I* asked you to make this configurable.

In any case, this patch is incomplete, since even after it, there are verbatim references to /var/log/core in the code. It serves only to confuse me. :)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I94ac220078f8a00fbdd8aba6125bedda44a77d15
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xu He Jie <xuhj at linux.vnet.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list