Change in vdsm[master]: ceph: Clear only ovirt secrets

danken at redhat.com danken at redhat.com
Sat Jun 27 19:11:46 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ceph: Clear only ovirt secrets
......................................................................


ceph: Clear only ovirt secrets

When clearing other secrets, unregister only ovirt secrets. This allow
the administrator to add other secrets to libvirt. This is not strictly
needed since we kind of own libvirt, but is more polite.

Change-Id: I8b86bd43a6d33a31ef2ee638a16f46657113316d
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41302
Tested-by: Daniel Erez <derez at redhat.com>
Reviewed-by: Daniel Erez <derez at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/vmSecretTests.py
M vdsm/virt/secret.py
2 files changed, 30 insertions(+), 16 deletions(-)

Approvals:
  Nir Soffer: Verified
  Adam Litke: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Daniel Erez: Verified; Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/41302
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8b86bd43a6d33a31ef2ee638a16f46657113316d
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list