July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: update vm through vm-manager
by ahadas＠redhat.com 28 Jul '16

28 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: update vm through vm-manager ...................................................................... core: update vm through vm-manager UpdateVm that updates the static VM data is now using VmManager. That will allow us to cache relevant things later on. Change-Id: I38fb87e536c5e84aa3aad34e134a073831601134 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java 2 files changed, 13 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/61040 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38fb87e536c5e84aa3aad34e134a073831601134 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Update to model 4.1.7
by juan.hernandez＠redhat.com 28 Jul '16

28 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Update to model 4.1.7 ...................................................................... restapi: Update to model 4.1.7 The more relevant changes included in the new version of the model are the following: * Add `logicalName` attribute to the disk attachment type. * Fix the name of the parameter to get virtual machine from affinity label, should be `vm` instead of `host`. Change-Id: I42822fee5e938c983b070d5f9952b77072b98249 Related-To: https://bugzilla.redhat.com/1358271 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61647 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I42822fee5e938c983b070d5f9952b77072b98249 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Support /v4 suffix in API URL
by juan.hernandez＠redhat.com 28 Jul '16

28 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Support /v4 suffix in API URL ...................................................................... Support /v4 suffix in API URL This patch add support for /v4 suffix in API URL. Change-Id: I86b46c97664b5bc3e9e88a11231abf53a4c376c3 Related-To: https://bugzilla.redhat.com/1360779 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 5dea6ab41bd360d3fc9bae3988341e78714f8429) --- M sdk/lib/ovirtsdk4/http.rb M sdk/spec/connection_create_spec.rb 2 files changed, 31 insertions(+), 10 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61635 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86b46c97664b5bc3e9e88a11231abf53a4c376c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: virt-viewer with sso-token support, rhel7
by tjelinek＠redhat.com 28 Jul '16

28 Jul '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: virt-viewer with sso-token support, rhel7 ...................................................................... core: virt-viewer with sso-token support, rhel7 Fix of minimal required version of remote-viewer supporting 'sso-token' for 'rhel7' system. Change-Id: Ie7b818e7dafdc63ff1c4afd117643b67875c2e30 Bug-Url: https://bugzilla.redhat.com/1285883 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved; Passed CI tests Jakub Niedermertl: Verified Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61636 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7b818e7dafdc63ff1c4afd117643b67875c2e30 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Support /v4 suffix in API URL
by juan.hernandez＠redhat.com 28 Jul '16

28 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Support /v4 suffix in API URL ...................................................................... Support /v4 suffix in API URL This patch add support for /v4 suffix in API URL. Change-Id: I86b46c97664b5bc3e9e88a11231abf53a4c376c3 Related-To: https://bugzilla.redhat.com/1360779 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/http.rb M sdk/spec/connection_create_spec.rb 2 files changed, 31 insertions(+), 10 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61622 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86b46c97664b5bc3e9e88a11231abf53a4c376c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: Add support to pass SSO token to SDK connection
by omachace＠redhat.com 28 Jul '16

28 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add support to pass SSO token to SDK connection ...................................................................... Add support to pass SSO token to SDK connection This patch add support to pass SSO token to SDK connection class, so if user has SSO token, he don't have to authenticate with username and password Change-Id: I136189476744aa4cc65a20e6e3d4ff4da9f12a17 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 5f43e49180e177e042714e727f81e961b5d190b2) --- M sdk/src/main/java/org/ovirt/engine/sdk4/ConnectionBuilder.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java 2 files changed, 23 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61632 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I136189476744aa4cc65a20e6e3d4ff4da9f12a17 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Add support to pass SSO token to SDK connection
by omachace＠redhat.com 28 Jul '16

28 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add support to pass SSO token to SDK connection ...................................................................... Add support to pass SSO token to SDK connection This patch add support to pass SSO token to SDK connection class, so if user has SSO token, he don't have to authenticate with username and password Change-Id: I136189476744aa4cc65a20e6e3d4ff4da9f12a17 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/src/main/java/org/ovirt/engine/sdk4/ConnectionBuilder.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java 2 files changed, 23 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61621 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I136189476744aa4cc65a20e6e3d4ff4da9f12a17 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: small PatternFly tweaks on welcome page
by gshereme＠redhat.com 28 Jul '16

28 Jul '16

Greg Sheremeta has submitted this change and it was merged. Change subject: engine: small PatternFly tweaks on welcome page ...................................................................... engine: small PatternFly tweaks on welcome page Small tweaks on welcome page: * positioned container band to 13% to bottom * moved login header down so green margin shows Change-Id: I5c98928a3818b05a5e19c1edbfd0c78e47ce62fb Bug-Url: https://bugzilla.redhat.com/1347109 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M backend/manager/modules/branding/src/main/resources/META-INF/tags/obrand/header.tag M packaging/branding/ovirt.brand/common.css M packaging/branding/ovirt.brand/welcome_style.css 3 files changed, 7 insertions(+), 2 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified -- To view, visit https://gerrit.ovirt.org/61584 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c98928a3818b05a5e19c1edbfd0c78e47ce62fb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.0]: Add support to pass SSO token to SDK connection
by omachace＠redhat.com 28 Jul '16

28 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add support to pass SSO token to SDK connection ...................................................................... Add support to pass SSO token to SDK connection This patch add support to pass SSO token to SDK connection class, so if user has SSO token, he don't have to authenticate with username and password Change-Id: I4b3fc0a38cc0a94f6c87c36fd04d17d776f03357 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 2a9329d786d12f98066cbc23bb39eeaacdd78d55) --- M sdk/lib/ovirtsdk4/__init__.py 1 file changed, 15 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61615 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b3fc0a38cc0a94f6c87c36fd04d17d776f03357 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: v2v-fix message and load button enabling
by piotr.kliczewski＠gmail.com 28 Jul '16

28 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: v2v-fix message and load button enabling ...................................................................... webadmin: v2v-fix message and load button enabling While opening the "import Virtual Machine(s)" dialog, the following error message always appears on start: "Not available when no Export Domain is active", although an active export domain does exist and active. And also "Load" button is enabled while should be disabled. The reason is that checking export Domain existence for the "Export Domain" Source is done on VmListModel constructor on initializing while querying for getting the export Domain is done later on while query callback returns from backend. So while opening the dialog, exportDomain is always null. The same relevant for enabling the "Load" button. Fixing that bug requires moving the error message and button enabling checks to later on. Immediate after reading the export domain from DB. Current behaviour is as follows: 1. While opening the dialog - error message and button disabling appears only if no export doamin really exists for that DC. 2. Changing the Source from other source back to be "Export Domain" - no actul checking is done for export domain existence. The previous status is kept and displayed. 3. Selecting other Data Center for the "export Domain" source - actual checking is done by querying the DB and in case no export Domain exists then an err msg in dislayed and "load" button is disabled. Furtheremore, this fix cleans VMs list while selecting other Data Center Change-Id: I2e89d26ce8363c111f5df91bc4a214011270c920 Bug-Url: https://bugzilla.redhat.com/1340025 Signed-off-by: Sharon Gratch <sgratch(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmsModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/ImportVmsPopupPresenterWidget.java 2 files changed, 32 insertions(+), 24 deletions(-) Approvals: Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved Sharon Gratch: Verified Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61607 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e89d26ce8363c111f5df91bc4a214011270c920 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0