- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: upgrade: creating a new disk for engine-appliance-4.0
by stirabos＠redhat.com 31 May '16

31 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: upgrade: creating a new disk for engine-appliance-4.0 ...................................................................... upgrade: creating a new disk for engine-appliance-4.0 creating a new disk for engine-appliance-4.0 to preserve the engine-appliance-3.6 disk till the migration end in order to provide a quick/seamless rollback on errors. Depends-on: https://gerrit.ovirt.org/#/c/54537/ Change-Id: I92bf4c677f15dbe432178f705b513e980fa8f466 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M po/POTFILES.in M src/ovirt_hosted_engine_setup/constants.py M src/plugins/gr-he-common/vm/boot_disk.py M src/plugins/gr-he-common/vm/misc.py M src/plugins/gr-he-setup/vm/runvm.py M src/plugins/gr-he-upgradeappliance/Makefile.am M src/plugins/gr-he-upgradeappliance/core/misc.py A src/plugins/gr-he-upgradeappliance/engine/Makefile.am A src/plugins/gr-he-upgradeappliance/engine/__init__.py A src/plugins/gr-he-upgradeappliance/engine/add_vm_disk.py M src/plugins/gr-he-upgradeappliance/vm/configurevm.py M src/plugins/gr-he-upgradeappliance/vm/runvm.py 13 files changed, 633 insertions(+), 52 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58136 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92bf4c677f15dbe432178f705b513e980fa8f466 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: moving bridge and CA acquiring to gr-he-common
by stirabos＠redhat.com 31 May '16

31 May '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: packaging: moving bridge and CA acquiring to gr-he-common ...................................................................... packaging: moving bridge and CA acquiring to gr-he-common Moving engine CA acquiring to gr-he-common since also the upgrade path will need it and on an earlier stage. Moving also bridge plugin to gr-he-common since we have to detect the host address from the bridge. Change-Id: I7e7bc76f726e466185b2886f2d5572f2473ae1df Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M po/POTFILES.in M src/ovirt_hosted_engine_setup/Makefile.am M src/ovirt_hosted_engine_setup/constants.py A src/ovirt_hosted_engine_setup/engineapi.py M src/plugins/gr-he-common/Makefile.am M src/plugins/gr-he-common/engine/Makefile.am M src/plugins/gr-he-common/engine/__init__.py A src/plugins/gr-he-common/engine/ca.py A src/plugins/gr-he-common/network/Makefile.am A src/plugins/gr-he-common/network/__init__.py R src/plugins/gr-he-common/network/bridge.py M src/plugins/gr-he-setup/engine/add_host.py M src/plugins/gr-he-setup/network/Makefile.am M src/plugins/gr-he-setup/network/__init__.py 15 files changed, 475 insertions(+), 242 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58131 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e7bc76f726e466185b2886f2d5572f2473ae1df Gerrit-PatchSet: 6 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: fix monitoring of guest agent nics
by ahadas＠redhat.com 31 May '16

31 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: fix monitoring of guest agent nics ...................................................................... core: fix monitoring of guest agent nics We used to to compare the hash that is being calculated by the engine (of network interfaces reported by the guest agent) with the hash in the dynamic VM data reported by VDSM. The problem is that VDSM doesn't report this hash, therefore it is always zero and therefore we update the dynamic data of a VM that is installed with guest agent on each monitoring cycle. This patch fixes this by changing the mentioned check so the computed hash will be compared to the one that was calculated previously. Change-Id: I38533ac778ffef528340d33cfbb64589d5d49fe8 Bug-Url: https://bugzilla.redhat.com/1340722 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Vinzenz Feenstra: Looks good to me, but someone else must approve Arik Hadas: Verified; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/58303 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38533ac778ffef528340d33cfbb64589d5d49fe8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix broken DAO tests
by alkaplan＠redhat.com 31 May '16

31 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: core: Fix broken DAO tests ...................................................................... core: Fix broken DAO tests The assumption is that stored mac-pool ranges are in lower-case should apply also to the fixtures file. Change-Id: I490a4b9e04cd90153de5652902bee174db6425b4 Bug-Url: https://bugzilla.redhat.com/1324125 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/MacPoolDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml 2 files changed, 12 insertions(+), 12 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58312 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I490a4b9e04cd90153de5652902bee174db6425b4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-image-uploader[master]: ovirt-image-uploader: port to SDK version 4
by rmartins＠redhat.com 31 May '16

31 May '16

Rafael Martins has submitted this change and it was merged. Change subject: ovirt-image-uploader: port to SDK version 4 ...................................................................... ovirt-image-uploader: port to SDK version 4 Please note that 'list' action behaviour changed, due to some limitations of ovirt-engine-sdk4: - Not showing the datacenter where the storage domain belongs - Status column isn't very reliable. Change-Id: I39029e5f3f13e3b748c417d1bc4f2ce0c349ab2d Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M configure.ac M ovirt-image-uploader.spec.in M src/__main__.py 3 files changed, 57 insertions(+), 112 deletions(-) Approvals: Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58271 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39029e5f3f13e3b748c417d1bc4f2ce0c349ab2d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: fixed missing mac_pool_id in LabelTestDao
by alkaplan＠redhat.com 31 May '16

31 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: core: fixed missing mac_pool_id in LabelTestDao ...................................................................... core: fixed missing mac_pool_id in LabelTestDao Change-Id: Ibe197bcd632a23a3bb2e58ec197de2b65ab82659 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/LabelDaoTest.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58328 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe197bcd632a23a3bb2e58ec197de2b65ab82659 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: centos-7: Added auto accept defaults option
by lveyde＠redhat.com 31 May '16

31 May '16

Lev Veyde has submitted this change and it was merged. Change subject: centos-7: Added auto accept defaults option ...................................................................... centos-7: Added auto accept defaults option An option to automatically accept defaults was added to the answer file. Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/ovirt-answer 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58335 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: centos-7: Update VDSM bridge setup script for the new API
by lveyde＠redhat.com 31 May '16

31 May '16

Lev Veyde has submitted this change and it was merged. Change subject: centos-7: Update VDSM bridge setup script for the new API ...................................................................... centos-7: Update VDSM bridge setup script for the new API The bridge setup script was updated to use the new API, as there is a change both in the module name and the parameter types of some of the functions used. Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/oVirtLiveFiles/root/usr/libexec/vdsm/hooks/before_vdsm_start/20-setupBridge.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58334 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: centos-7: Remove nfs-server workaround
by lveyde＠redhat.com 31 May '16

31 May '16

Lev Veyde has submitted this change and it was merged. Change subject: centos-7: Remove nfs-server workaround ...................................................................... centos-7: Remove nfs-server workaround This seems no longer to be required in 4.0, and actually causes the nfs-server to fail to start. Change-Id: Id70d15f1a185c5279471528ba63a953444995af0 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58333 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id70d15f1a185c5279471528ba63a953444995af0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: kickstart: remove kernel plugin
by lveyde＠redhat.com 31 May '16

31 May '16

Lev Veyde has submitted this change and it was merged. Change subject: kickstart: remove kernel plugin ...................................................................... kickstart: remove kernel plugin Remove ovirt-host-deploy kernel plugins, as it fails to run under ovirt-live, and causes the host-deploy to fail. Change-Id: I387c50ff241df0b00f33b094f47a72584122501f Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58332 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I387c50ff241df0b00f33b094f47a72584122501f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0