Dan Kenigsberg has submitted this change and it was merged.

Change subject: ceph: Add option to clear other secrets
......................................................................


ceph: Add option to clear other secrets

Typically engine will add new secrets when activating a Cinder/Ceph
storage domain or when a user add new secret to an active domain. Engine
will remove secrets when deactivating a domain, moving the host to
maintenance, or when a user delete a secret form an active domain.

When host is unreachable when engine try to unregister secrets, the host
will contain stale or unneeded secrets. When the host becomes UP again,
engine would like to register all the secrets required for the current
active storage domains, and clear any other secret.

Change-Id: I476e5aa0bd4a48d1837de09c59597f180fa38823
Signed-off-by: Nir Soffer <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/40853
Tested-by: Daniel Erez <[email protected]>
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Adam Litke <[email protected]>
Continuous-Integration: Jenkins CI
---
M client/vdsClient.py
M tests/vmSecretTests.py
M tests/vmfakelib.py
M vdsm/API.py
M vdsm/rpc/bindingxmlrpc.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/virt/secret.py
7 files changed, 36 insertions(+), 11 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I476e5aa0bd4a48d1837de09c59597f180fa38823
Gerrit-PatchSet: 20
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to