[node-patches] Change in ovirt-node[master]: BZ#815825 validate vdsmcert against cacert

danken at redhat.com danken at redhat.com
Thu May 3 04:35:58 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#815825 validate vdsmcert against cacert
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File libvirtd.upstart
Line 6: pre-start script
It is bad enough for vdsm to configure libvirt; I am not happy about l libvirt conf script configuring libvirt.

Can you think of another place to better-fit this cert backup function?

Since certificate is security-sensitive file, I'm not sure out-of-context restoration is acceptable (assume the backed up version is obsolete).

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d9de5d131fdaca0f875b14d21a97943c63b1770
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list