Michael Pasternak has posted comments on this change.

Change subject: core: Allow activate an already active domain
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

....................................................
Commit Message
Line 6: 
Line 7: core: Allow activate an already active domain
Line 8: 
Line 9: Using rest to try to activate an already active domain was an
Line 10: illegal action. It's now allowed, with an AuditLog alert.
this is inconsistent with other resources in the system, take vm for instance,
it fails with CDA message "vm cannot be started, it already running".
Line 11: 
Line 12: Change-Id: Ibe8ad96a5bd40adb56b7a63c4ae5ef75a550e5b2
Line 13: Bug-Url: https://bugzilla.redhat.com/849427


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibe8ad96a5bd40adb56b7a63c4ae5ef75a550e5b2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to