Dan Kenigsberg has submitted this change and it was merged.

Change subject: ceph: Clear all secrets during startup and shutdown
......................................................................


ceph: Clear all secrets during startup and shutdown

Libvirt secrets are needed only when vdsm is connected to engine. To
ensure that we don't leave stale secrets or secrets which are not needed
by the active storage domains, we clear all secrets when the service
starting or stopping. Engine will register the required secrets when
connecting to the host or when activating or deactivating storage
domains.

Change-Id: I5c5aa36ed752642745b7febc785750b438c102bb
Signed-off-by: Nir Soffer <nsof...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40968
Reviewed-by: Daniel Erez <de...@redhat.com>
Tested-by: Daniel Erez <de...@redhat.com>
Reviewed-by: Francesco Romani <from...@redhat.com>
Reviewed-by: Adam Litke <ali...@redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M tests/vmSecretTests.py
M vdsm/clientIF.py
M vdsm/virt/secret.py
3 files changed, 42 insertions(+), 0 deletions(-)

Approvals:
  Adam Litke: Looks good to me, but someone else must approve
  Nir Soffer: Verified
  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/40968
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5c5aa36ed752642745b7febc785750b438c102bb
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to