Change in ovirt-engine[ovirt-engine-3.5]: packaging: setup: engine-cleanup refuses to remove a differe...
by piotr.kliczewski@gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: packaging: setup: engine-cleanup refuses to remove a different version
......................................................................
packaging: setup: engine-cleanup refuses to remove a different version
engine-cleanup should refuse to remove a version
of the engine where the x or y version number
are different or where the z is major
Change-Id: I06b0d3437bfca4f4e63be4c212f15e98f5617bfd
Bug-Url: https://bugzilla.redhat.com/1118360
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
(cherry picked from commit e0f57f0ae5887da0148fc8528a1a32851ec4ac36)
---
M packaging/setup/plugins/ovirt-engine-remove/base/core/misc.py
1 file changed, 30 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Verified
--
To view, visit http://gerrit.ovirt.org/30750
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I06b0d3437bfca4f4e63be4c212f15e98f5617bfd
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 4 months
Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: validate unicast MAC address
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: validate unicast MAC address
......................................................................
packaging: setup: validate unicast MAC address
Change-Id: I6af3523bff99d153691996152f39b7d4750eeaf5
Bug-Url: https://bugzilla.redhat.com/1116785
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
(cherry picked from commit 6c1b4046ea12306dc8dd556f0ca41f70defdbfe6)
---
M src/ovirt_hosted_engine_setup/util.py
M src/plugins/ovirt-hosted-engine-setup/vm/mac.py
2 files changed, 24 insertions(+), 20 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30926
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6af3523bff99d153691996152f39b7d4750eeaf5
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-1.2
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 4 months
Change in ovirt-hosted-engine-setup[master]: packaging: setup: validate unicast MAC address
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: validate unicast MAC address
......................................................................
packaging: setup: validate unicast MAC address
Change-Id: I6af3523bff99d153691996152f39b7d4750eeaf5
Bug-Url: https://bugzilla.redhat.com/1116785
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M src/ovirt_hosted_engine_setup/util.py
M src/plugins/ovirt-hosted-engine-setup/vm/mac.py
2 files changed, 24 insertions(+), 20 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
Jiří Moskovčák: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/30784
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6af3523bff99d153691996152f39b7d4750eeaf5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 4 months
Change in ovirt-release[master]: Updated EL dependencies repo
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: Updated EL dependencies repo
......................................................................
Updated EL dependencies repo
- provide EL7 dependencies for 3.5
- add python-ioprocess from EPEL repository for vdsm on 3.5 and master
Change-Id: I1c2dba4d15c807635911c41724329c0e02669df4
Bug-Url: https://bugzilla.redhat.com/1124802
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M ovirt-release-3.5/Makefile.am
M ovirt-release-3.5/configure.ac
R ovirt-release-3.5/ovirt-el6-deps.repo.in
C ovirt-release-3.5/ovirt-el7-deps.repo.in
M ovirt-release-3.5/ovirt-release35.spec.in
M ovirt-release-master/ovirt-el6-deps.repo.in
M ovirt-release-master/ovirt-el7-deps.repo.in
7 files changed, 23 insertions(+), 23 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/30921
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1c2dba4d15c807635911c41724329c0e02669df4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 4 months
Change in ovirt-engine[ovirt-engine-3.5]: core: Fixes StackOverflowError during fencing
by piotr.kliczewski@gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: core: Fixes StackOverflowError during fencing
......................................................................
core: Fixes StackOverflowError during fencing
Fixes StackOverflowError appeared during execution of fencing operation.
This fix removes commandContex instance and overriden getContext() from
RestartVdsCommand, because command context in CommandBase should be set
always during creation of command and successful locks aquiring is
checked prior to executeCommand call in CommandBase.
Also when RestartVdsCommand and its inherited commands are using global
command context, VdsKdumpDetection command executed from
VdsNotRespondingTreatmentCommand should also use global command context
and not its copy.
The overriden getContext() has been introduced in
Iaf021010188e3c3662fbf8a057da4c58f2600c02.
Change-Id: Icbe5ad9194b19a390f45c99198087a9e5b68eb9c
Bug-Url: https://bugzilla.redhat.com/1124141
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestartVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsKdumpDetectionCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java
3 files changed, 2 insertions(+), 16 deletions(-)
Approvals:
Martin Peřina: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30906
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icbe5ad9194b19a390f45c99198087a9e5b68eb9c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 4 months
Change in jenkins[master]: repo_closure: added EL7 support
by dcaroest@redhat.com
David Caro has submitted this change and it was merged.
Change subject: repo_closure: added EL7 support
......................................................................
repo_closure: added EL7 support
Added EL7 support for repository closure checking.
Also fixed URLs to Gluster repositories for EL6.
Change-Id: I77e7f654318589dcecc4e27c7285c98a74c1164d
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/packaging/repo_closure_check.sh
1 file changed, 46 insertions(+), 23 deletions(-)
Approvals:
Sandro Bonazzola: Verified
Eyal Edri: Looks good to me, but someone else must approve
David Caro: Looks good to me, approved
Ohad Basan: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/30777
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I77e7f654318589dcecc4e27c7285c98a74c1164d
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
10 years, 4 months
Change in ovirt-engine[master]: engine : Reason does not get updated when trying to shut dow...
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: engine : Reason does not get updated when trying to shut down a VM
......................................................................
engine : Reason does not get updated when trying to shut down a VM
When trying to stop the VM while it is
powering up or shutting down, the reason
comment does not get saved in the DB
Change-Id: I8f70177206de3fd3081826ea898ab2a7fb87357b
Bug-Url: https://bugzilla.redhat.com/1114903
Signed-off-by: Ravi Nori <rnori(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ShutdownVmCommand.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Ravi Nori: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30902
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8f70177206de3fd3081826ea898ab2a7fb87357b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori(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
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 4 months
Change in ovirt-node-plugin-vdsm[ovirt-3.5]: configure.ac: increase rpm release
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: configure.ac: increase rpm release
......................................................................
configure.ac: increase rpm release
Change-Id: I3f66aac520e0390b2fb0b81d58169288d3e54c0f
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
---
M configure.ac
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30920
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3f66aac520e0390b2fb0b81d58169288d3e54c0f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 4 months
Change in ovirt-node-plugin-vdsm[ovirt-3.5]: Include hosted engine plugin into iso form 3.5 on
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: Include hosted engine plugin into iso form 3.5 on
......................................................................
Include hosted engine plugin into iso form 3.5 on
>From 3.5 on we want to include the hosted-engine plugin by default into
the iso for oVirt.
Change-Id: Idda19745018541a51866aca2108510e67c907c03
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1038616
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M recipe/vdsm-plugin-iso.ks
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30919
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idda19745018541a51866aca2108510e67c907c03
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years, 4 months
Change in ovirt-node-plugin-vdsm[master]: Include hosted engine plugin into iso form 3.5 on
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: Include hosted engine plugin into iso form 3.5 on
......................................................................
Include hosted engine plugin into iso form 3.5 on
>From 3.5 on we want to include the hosted-engine plugin by default into
the iso for oVirt.
Change-Id: Idda19745018541a51866aca2108510e67c907c03
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1038616
Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org>
---
M recipe/vdsm-plugin-iso.ks
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/30918
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idda19745018541a51866aca2108510e67c907c03
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years, 4 months