[node-patches] Change in ovirt-node[master]: set password correctly every time apply is hit on the CIM pa...

jboggs at redhat.com jboggs at redhat.com
Tue May 8 20:01:45 UTC 2012


Joey Boggs has posted comments on this change.

Change subject: set password correctly every time apply is hit on the CIM page
......................................................................


Patch Set 1: I would prefer that you didn't submit this

If I set the password, then apply. Once it comes back to the cim page and try setting the password again, it will fail to try setting it.

There's logic around the status of the enable/disable to see its starting state and current state, need to break it out of there to set the password regardless of the enable/disable being toggled.

--
To view, visit http://gerrit.ovirt.org/3890
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc537275287a327da254978127836ce59fe87a87
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>



More information about the node-patches mailing list