Change in ovirt-engine[master]: core: delete last domain fails with engine-manage-domain (#8...
Oved Ourfali has submitted this change and it was merged.
Change subject: core: delete last domain fails with engine-manage-domain (#851069)
......................................................................
core: delete last domain fails with engine-manage-domain (#851069)
Setting AdUserPassword to empty string failed because
AdUserPassword.type is CompositePassword.
CompositePassword must contain the char ':' otherwise set
operation on this value will fail.
The fix lets us also assign empty string to CompositePassword
when we want to erase this value in db.
Change-Id: I9d4904a916f2055288e99114f2f267face16797b
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M
backend/manager/tools/engine-config/src/main/java/org/ovirt/engine/core/config/entity/helper/CompositePasswordValueHelper.java
1 file changed, 4 insertions(+), 0 deletions(-)
Approvals:
Oved Ourfali: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7604
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9d4904a916f2055288e99114f2f267face16797b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>