August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine : Host upgrade shows process terminated message
by masayag＠redhat.com 10 Aug '16

10 Aug '16

Moti Asayag has submitted this change and it was merged. Change subject: engine : Host upgrade shows process terminated message ...................................................................... engine : Host upgrade shows process terminated message Host upgrade process should not check for maintenance command status after upgrade host command has started execution. Change-Id: I6847ab45d1fda206f18a41917069d973be917885 Bug-Url: https://bugzilla.redhat.com/1361511 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/HostUpgradeCallback.java 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61990 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6847ab45d1fda206f18a41917069d973be917885 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Cannot import KVM VMs as clone
by piotr.kliczewski＠gmail.com 10 Aug '16

10 Aug '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Cannot import KVM VMs as clone ...................................................................... core: Cannot import KVM VMs as clone Bug-Url: https://bugzilla.redhat.com/1364164 Change-Id: Ie46428f4b05a89e6061bafadb92b8fb2499c5d6b Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmFromExternalProviderCommand.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62177 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie46428f4b05a89e6061bafadb92b8fb2499c5d6b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Ignore failures fetching cluster policy
by piotr.kliczewski＠gmail.com 10 Aug '16

10 Aug '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Ignore failures fetching cluster policy ...................................................................... restapi: Ignore failures fetching cluster policy Currently when a cluster is fetched in version 3 of the API an additional query is performed to retrieve the details of its associated scheduling policy, as that is how it used to work. But this additional query may require additional permissions for some users, and when this happens the complete cluster fetch fails. To alleviate that problem this patch changes the API so that failures to fetch the scheduling policy will just be ignored. As most users won't be interersted in those details this should solve most use cases. When a user without the required permissions fetches the cluster she will get only the reference, and not the details: GET /clusters Authorization: myuser@mydomain:... Filter: true <cluster id="123" href="/clusters/123"> ... <scheduling_policy id="456" href="/schedulingpolicies/456"/> ... </cluster> When the user has the required permissions he will get all the details of the scheduling policy. Change-Id: I23b0ef3830f8291d725fa8fce241c8e0c12309be Bug-Url: https://bugzilla.redhat.com/1364048 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 12fffe432e03396ed62e3984116df46ea7d78e5a) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ClusterOutAdapter.java 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: 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/62173 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23b0ef3830f8291d725fa8fce241c8e0c12309be Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: Adding build-artifacts-manual.sh script
by Shirly Radco 10 Aug '16

10 Aug '16

Shirly Radco has submitted this change and it was merged. Change subject: Adding build-artifacts-manual.sh script ...................................................................... Adding build-artifacts-manual.sh script Adding build-artifacts-manual.sh script and related repo and packages files. Change-Id: I8e0527e15907ceaf47b3c18edee73e893bb7fefa Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A automation/build-artifacts-manual.packages A automation/build-artifacts-manual.repos A automation/build-artifacts-manual.sh 3 files changed, 68 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Sharon Naftaly: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61776 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e0527e15907ceaf47b3c18edee73e893bb7fefa Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-3.6]: Adding build-artifacts-manual.sh script
by Shirly Radco 10 Aug '16

10 Aug '16

Shirly Radco has submitted this change and it was merged. Change subject: Adding build-artifacts-manual.sh script ...................................................................... Adding build-artifacts-manual.sh script Adding build-artifacts-manual.sh script to build artifacts from an uploaded tarball Change-Id: I62606dd8126f74cfcdd1d9caaeadf121a9646616 Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A automation/build-artifacts-manual.packages A automation/build-artifacts-manual.packages.el6 A automation/build-artifacts-manual.repos A automation/build-artifacts-manual.repos.el6 A automation/build-artifacts-manual.sh 5 files changed, 83 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Sharon Naftaly: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61394 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62606dd8126f74cfcdd1d9caaeadf121a9646616 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.6 Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-4.0]: Adding build-artifacts-manual.sh script
by Shirly Radco 10 Aug '16

10 Aug '16

Shirly Radco has submitted this change and it was merged. Change subject: Adding build-artifacts-manual.sh script ...................................................................... Adding build-artifacts-manual.sh script Adding build-artifacts-manual.sh script and related repo and packages files. Change-Id: Ia55d1452a61ea5f77ea9f3ac42f67909f6c846ac Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A automation/build-artifacts-manual.packages A automation/build-artifacts-manual.repos A automation/build-artifacts-manual.sh 3 files changed, 68 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Looks good to me, approved Sharon Naftaly: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61780 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia55d1452a61ea5f77ea9f3ac42f67909f6c846ac Gerrit-PatchSet: 7 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-4.0 Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: UpdateHostValidatorTest type inference
by amureini＠redhat.com 10 Aug '16

10 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: UpdateHostValidatorTest type inference ...................................................................... core: UpdateHostValidatorTest type inference Change-Id: Ie95c07e9ab43a52618f10b814e25db2e486a1d61 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/UpdateHostValidatorTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62174 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie95c07e9ab43a52618f10b814e25db2e486a1d61 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: added missing mapping for el7 ppc64le
by eedri＠redhat.com 10 Aug '16

10 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: added missing mapping for el7 ppc64le ...................................................................... added missing mapping for el7 ppc64le - Added missing ppc64le mappings to el7 and fedora 24 files in mock runner - Removed epel debug repos since they don't exists for ppc64le and we don't need them Change-Id: Idbc279b726b3c53666402fe1289c153f8fbc78c2 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M mock_configs/epel-7-ppc64le.cfg M mock_configs/epel-7-ppc64le_proxied.cfg M mock_configs/mock_runner.sh 3 files changed, 2 insertions(+), 10 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/62122 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idbc279b726b3c53666402fe1289c153f8fbc78c2 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gal Ben Haim <galbh2(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cache vm static data needed for monitoring
by ahadas＠redhat.com 10 Aug '16

10 Aug '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cache vm static data needed for monitoring ...................................................................... core: cache vm static data needed for monitoring Cache the data we need for the monitoring in VmManager. Change-Id: I2e168460a00c887b00cb26c819bc2dc23e706f32 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java 1 file changed, 51 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/61041 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e168460a00c887b00cb26c819bc2dc23e706f32 Gerrit-PatchSet: 13 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: Ignore failures fetching cluster policy
by juan.hernandez＠redhat.com 09 Aug '16

09 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Ignore failures fetching cluster policy ...................................................................... restapi: Ignore failures fetching cluster policy Currently when a cluster is fetched in version 3 of the API an additional query is performed to retrieve the details of its associated scheduling policy, as that is how it used to work. But this additional query may require additional permissions for some users, and when this happens the complete cluster fetch fails. To alleviate that problem this patch changes the API so that failures to fetch the scheduling policy will just be ignored. As most users won't be interersted in those details this should solve most use cases. When a user without the required permissions fetches the cluster she will get only the reference, and not the details: GET /clusters Authorization: myuser@mydomain:... Filter: true <cluster id="123" href="/clusters/123"> ... <scheduling_policy id="456" href="/schedulingpolicies/456"/> ... </cluster> When the user has the required permissions he will get all the details of the scheduling policy. Change-Id: I23b0ef3830f8291d725fa8fce241c8e0c12309be Bug-Url: https://bugzilla.redhat.com/1364048 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ClusterOutAdapter.java 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Roy Golan: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/62172 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23b0ef3830f8291d725fa8fce241c8e0c12309be Gerrit-PatchSet: 2 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: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0