June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: VdcActionParametersBase - choose if command is ended b...
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: core: VdcActionParametersBase - choose if command is ended by parent ...................................................................... core: VdcActionParametersBase - choose if command is ended by parent This patch adds the shouldBeEndedByParent flag to VdcActionParametersBase, this flags indicates whether the command should be ended by it's callback or if its parent command will end it. Change-Id: Id5342bee4989b44a878205c2d6094454b6ef5b2e Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41689 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id5342bee4989b44a878205c2d6094454b6ef5b2e Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: VdcActionParametersBase - adding VdsRunningOn member
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: VdcActionParametersBase - adding VdsRunningOn member ...................................................................... VdcActionParametersBase - adding VdsRunningOn member Change-Id: Ic5b3827b958356a239c7c45d40cc671e4706a122 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41688 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5b3827b958356a239c7c45d40cc671e4706a122 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: CommandCoordinatorImpl - init child hirerchy with all ...
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: core: CommandCoordinatorImpl - init child hirerchy with all Commands ...................................................................... core: CommandCoordinatorImpl - init child hirerchy with all Commands The child hirerchy might contain commands with no callback that the command monitors, those should be loaded as well when the command hirerchy is initated. Change-Id: I63b079db8f0db13afaafded301a07744dfb85c78 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCoordinatorImpl.java 1 file changed, 7 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40252 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I63b079db8f0db13afaafded301a07744dfb85c78 Gerrit-PatchSet: 26 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: CommandExecutor - pass context to executed commands
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: CommandExecutor - pass context to executed commands ...................................................................... CommandExecutor - pass context to executed commands Change-Id: I437bcbe11d2fcc8a700c1f9109b11b8243572fb0 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41697 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I437bcbe11d2fcc8a700c1f9109b11b8243572fb0 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: CommandBase - persist command if the parent command has a ca...
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: CommandBase - persist command if the parent command has a callback ...................................................................... CommandBase - persist command if the parent command has a callback Change-Id: If15fad1db94e7a0bdc51d02afc9b952b4e980e4b Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommandBaseTest.java 2 files changed, 20 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, but someone else must approve Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41696 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If15fad1db94e7a0bdc51d02afc9b952b4e980e4b Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: CommandBase - end job after execution only if there is...
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: core: CommandBase - end job after execution only if there is no callback ...................................................................... core: CommandBase - end job after execution only if there is no callback Change-Id: I7a5df0fc98add315ffd67f4888f7614285dcbbb9 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, but someone else must approve Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41693 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a5df0fc98add315ffd67f4888f7614285dcbbb9 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: CommandBase - check if the dc is without SPM
by laravot＠redhat.com 29 Jun '15

29 Jun '15

Liron Aravot has submitted this change and it was merged. Change subject: core: CommandBase - check if the dc is without SPM ...................................................................... core: CommandBase - check if the dc is without SPM Change-Id: Ia1d9650cbfdee74dc53529fab5101119cf63d75d Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/41695 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia1d9650cbfdee74dc53529fab5101119cf63d75d Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-live[master]: ovirt-live 3.6.0 alpha2
by sbonazzo＠redhat.com 29 Jun '15

29 Jun '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-live 3.6.0 alpha2 ...................................................................... ovirt-live 3.6.0 alpha2 Change-Id: I25c0b8f365b130335b85066620f2d8d1187c8d39 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42976 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25c0b8f365b130335b85066620f2d8d1187c8d39 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: removed unused fields.
by alkaplan＠redhat.com 29 Jun '15

29 Jun '15

Alona Kaplan has submitted this change and it was merged. Change subject: core: removed unused fields. ...................................................................... core: removed unused fields. Change-Id: I6c0727248f12d968977f0ee5d2ca1bacd1f35f3b Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/AddNetworkAttachmentCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/RemoveNetworkAttachmentCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/UpdateNetworkAttachmentCommand.java 3 files changed, 0 insertions(+), 16 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42972 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c0727248f12d968977f0ee5d2ca1bacd1f35f3b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core:[cloneFromSnapshot] Add default constructor with context
by mlipchuk＠redhat.com 28 Jun '15

28 Jun '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core:[cloneFromSnapshot] Add default constructor with context ...................................................................... core:[cloneFromSnapshot] Add default constructor with context Add constructor with context to support CoCo callbacks Change-Id: I15fd0e5df94790dd988705dcec223fdbbe66d3db Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromSnapshotCommand.java 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/42841 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15fd0e5df94790dd988705dcec223fdbbe66d3db Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0