Allon Mureinik has posted comments on this change.

Change subject: core: handle error when iso domain in maintenance
......................................................................


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

@Ayal - there are two flows here:
1. automatic syncing - which is invoked only on ACTIVE domains anyway
2. GUI invocation, which to me seems uselsess to have an audit log if we know 
in advance that the domain is in some funky state.

@Alissa - like the change, but the patch contains a lot of extra "noise".
Please clean it up

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ide53f42c087ffbc9240cccc033e809ff8117ec8c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to