Change in jenkins[master]: Set the jdk used by master upgrade jobs to java-1.8.0-openjdk
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: Set the jdk used by master upgrade jobs to java-1.8.0-openjdk
......................................................................
Set the jdk used by master upgrade jobs to java-1.8.0-openjdk
Change-Id: If84d01265c04fb354772c5501e856c4bbe4561a3
Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com>
---
M jobs/confs/projects/ovirt-engine/ovirt-engine_upgrade-engine.yaml
M jobs/confs/yaml/templates/engine-upgrade.yaml
2 files changed, 3 insertions(+), 1 deletion(-)
Approvals:
Sharon Naftaly: Verified; Ready for review
Juan Hernandez: Looks good to me
Eyal Edri: Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/46661
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If84d01265c04fb354772c5501e856c4bbe4561a3
Gerrit-PatchSet: 8
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaro(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com>
9 years, 5 months
Change in jenkins[master]: ovirt-engine-extension-aaa-ldap: exclude fedora on 3.5
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: ovirt-engine-extension-aaa-ldap: exclude fedora on 3.5
......................................................................
ovirt-engine-extension-aaa-ldap: exclude fedora on 3.5
Change-Id: Ica0cb6945e102632483f13a98fd035af22c73a27
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/confs/projects/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml
1 file changed, 21 insertions(+), 9 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48405
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ica0cb6945e102632483f13a98fd035af22c73a27
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: core: when starting Live Merge, check for NOT_STARTED child ...
by amureini@redhat.com
Allon Mureinik has submitted this change and it was merged.
Change subject: core: when starting Live Merge, check for NOT_STARTED child commands
......................................................................
core: when starting Live Merge, check for NOT_STARTED child commands
When a Live Merge is started, the RemoveSnapshotCommandCallback does not
look for child commands with the NOT_STARTED command status. This
causes a race where the child commands may not be detected properly by
the parent, causing the actual merge process to be skipped.
This adds the needed check to the command callback, eliminating the race
condition and ensuring the merge will occur.
Change-Id: If47d26d8bedaf0bb4a39cae472533399178b4fdd
Bug-Url: https://bugzilla.redhat.com/1280160
Signed-off-by: Greg Padgett <gpadgett(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandCallback.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Greg Padgett: Verified
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/48400
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If47d26d8bedaf0bb4a39cae472533399178b4fdd
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-engine[master]: webadmin: GUI refactor NumaNode guid to NumaNode id
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: webadmin: GUI refactor NumaNode guid to NumaNode id
......................................................................
webadmin: GUI refactor NumaNode guid to NumaNode id
Fix GUI, display pinned NUMA nodes by their index (not Guid)
---------------------------------------------------------------
In NumaSupportModel class.
Refactor p2vNumaNodesMap map. To store pNodes and their pinned vNodes.
Change the key from Guid to Integer. Refactor all members using this
map.
In order to prevent guid consistency maintenance whenever a host is
re-added/removed.
Change-Id: Ia1261c81ddbc2e18fd273c25945e9cc24032bdd2
Bug-Url: https://bugzilla.redhat.com/1177259
Signed-off-by: Dudi Maroshi <dudi(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/numa/NumaSupportPopupView.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/numa/NumaSupportModel.java
2 files changed, 11 insertions(+), 11 deletions(-)
Approvals:
Dudi Maroshi: Verified
Jenkins CI: Passed CI tests
Daniel Erez: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/47340
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia1261c81ddbc2e18fd273c25945e9cc24032bdd2
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: packaging: ensuring it saves broker.conf during upgrades
by msivak@redhat.com
Martin Sivák has submitted this change and it was merged.
Change subject: packaging: ensuring it saves broker.conf during upgrades
......................................................................
packaging: ensuring it saves broker.conf during upgrades
/var/lib/ovirt-hosted-engine-ha/broker.conf got populated by rpm
install with its default and /etc/ovirt-hosted-engine-ha/broker.conf
got written over that only if empty.
Only after that the upgrade procedure will copy to the shared storage.
Change-Id: I79e556d741a03c464ebd9c8a97b3c71c657e0234
Bug-Url: https://bugzilla.redhat.com/1279548
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M ovirt-hosted-engine-ha.spec.in
1 file changed, 7 insertions(+), 9 deletions(-)
Approvals:
Martin Sivák: Looks good to me, approved
Simone Tiraboschi: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48352
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I79e556d741a03c464ebd9c8a97b3c71c657e0234
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: ovirt-hosted-engine-ha-1.3
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-hosted-engine-ha[master]: packaging: ensuring it saves broker.conf during upgrades
by msivak@redhat.com
Martin Sivák has submitted this change and it was merged.
Change subject: packaging: ensuring it saves broker.conf during upgrades
......................................................................
packaging: ensuring it saves broker.conf during upgrades
/var/lib/ovirt-hosted-engine-ha/broker.conf got populated by rpm
install with its default and /etc/ovirt-hosted-engine-ha/broker.conf
got written over that only if empty.
Only after that the upgrade procedure will copy to the shared storage.
Change-Id: I79e556d741a03c464ebd9c8a97b3c71c657e0234
Bug-Url: https://bugzilla.redhat.com/1279548
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M ovirt-hosted-engine-ha.spec.in
1 file changed, 7 insertions(+), 9 deletions(-)
Approvals:
Martin Sivák: Looks good to me, approved
Simone Tiraboschi: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48315
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I79e556d741a03c464ebd9c8a97b3c71c657e0234
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-node-plugin-vdsm[ovirt-3.6]: configure: Increase release
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: configure: Increase release
......................................................................
configure: Increase release
Change-Id: Id4df471720d688778f929bd30f665742c46b0c21
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48383
---
M configure.ac
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48383
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id4df471720d688778f929bd30f665742c46b0c21
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in ovirt-node-plugin-vdsm[ovirt-3.6]: engine_page: Add validation for None type
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: engine_page: Add validation for None type
......................................................................
engine_page: Add validation for None type
Currently there is no validation which might break when
users try to set the root password using TUI. This
patch adds the validation.
Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1279329
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48319
Continuous-Integration: Jenkins CI
Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com>
(cherry picked from commit 761bee37a93b6557e0729e499169a7c96f936c54)
Reviewed-on: https://gerrit.ovirt.org/48381
---
M src/engine_page.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48381
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
9 years, 5 months
Change in ovirt-node-plugin-vdsm[master]: engine_page: Add validation for None type
by dougsland@redhat.com
Douglas Schilling Landgraf has submitted this change and it was merged.
Change subject: engine_page: Add validation for None type
......................................................................
engine_page: Add validation for None type
Currently there is no validation which might break when
users try to set the root password using TUI. This
patch adds the validation.
Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1279329
Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48319
Continuous-Integration: Jenkins CI
Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com>
---
M src/engine_page.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests
Fabian Deutsch: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/48319
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months
Change in jenkins[master]: vdsm: add missing builder for fc23
by sbonazzo@redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: vdsm: add missing builder for fc23
......................................................................
vdsm: add missing builder for fc23
Change-Id: Icaa66988713f22de517eba007b2a2f5a05c1bf6c
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/confs/projects/vdsm/vdsm_create-rpms.yaml
1 file changed, 5 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge
--
To view, visit https://gerrit.ovirt.org/48380
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icaa66988713f22de517eba007b2a2f5a05c1bf6c
Gerrit-PatchSet: 3
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 5 months