Change in ovirt-engine[master]: core: profiles: make AddUserProfile update the key
Arik Hadas has submitted this change and it was merged.
Change subject: core: profiles: make AddUserProfile update the key
......................................................................
core: profiles: make AddUserProfile update the key
To accomodate the need of REST API, we make AddUserProfileCommand
detect when the client wants just to update the SSH Public key
parameters, and we handle this case inside the command as special
case of Update, trasparently.
During the verification of this patch it was discovered that the
update command was not refreshing the SSHPublicKeyID after each change.
This is harmless for {UserPortal,Webadmin} UI, but incorrect for
the REST API.
So this patch also make sure that each succesfully update
changes the key ID.
Change-Id: Ia197bddb74186fa7c696f7e7f28e8fbe5e487e08
Bug-Url: https://bugzilla.redhat.com/1283499
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddUserProfileCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateUserProfileCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UserProfilesOperationCommandBase.java
3 files changed, 49 insertions(+), 14 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Francesco Romani: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/48994
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia197bddb74186fa7c696f7e7f28e8fbe5e487e08
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>