- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[sdk_4.0]: packaging: spec: set valid Url for home page
by juan.hernandez＠redhat.com 27 Jul '16

27 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: set valid Url for home page ...................................................................... packaging: spec: set valid Url for home page Previous Url returned 404 after website migration. Missing a proper home page, pointing to oVirt website. Change-Id: I1323dcabc479353894fb7b0169d6442df55ec576 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit d74a1cf545f04baeaadbabb62a42938c83210ed7) --- M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61449 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1323dcabc479353894fb7b0169d6442df55ec576 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: minor optimization in vm-analyzer
by ahadas＠redhat.com 27 Jul '16

27 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: minor optimization in vm-analyzer ...................................................................... core: minor optimization in vm-analyzer First check whether we get balloon information from VDSM and only then check if balloon is enabled on the host. This reason for this is that checking the host is a relatively costly operation in term of CPU time (specifically because of the cloning of VDS object). Not sure that the second check is needed anymore, but I prefer not to remove it as part of this patch. Change-Id: Ide71a7a5edcb44f05ca1b58e84d4ce3481fa5806 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, 3 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61365 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide71a7a5edcb44f05ca1b58e84d4ce3481fa5806 Gerrit-PatchSet: 6 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-sdk-ruby[master]: packaging: spec: set valid Url for home page
by juan.hernandez＠redhat.com 27 Jul '16

27 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: set valid Url for home page ...................................................................... packaging: spec: set valid Url for home page Previous Url returned 404 after website migration. Missing a proper home page, pointing to oVirt website. Change-Id: I1323dcabc479353894fb7b0169d6442df55ec576 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61445 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1323dcabc479353894fb7b0169d6442df55ec576 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: packaging: spec: rename spec file
by juan.hernandez＠redhat.com 27 Jul '16

27 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: rename spec file ...................................................................... packaging: spec: rename spec file Since the desired RPM name is rubygem-ovirt-engine-sdk4, renaming the spec file accordingly. Fixes: rubygem-ovirt-engine-sdk4.src: E: invalid-spec-name Change-Id: I2406ad021f7ea9fc8e0939fa678aadf2b6b3b1af Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build.py 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/61446 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2406ad021f7ea9fc8e0939fa678aadf2b6b3b1af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: packaging: spec: set valid Source
by juan.hernandez＠redhat.com 27 Jul '16

27 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: set valid Source ...................................................................... packaging: spec: set valid Source Set valid Source URL for the GEM. Change-Id: I4610c7a2d78f18427bda3093213f04192cd1670a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/spec.el7.in M packaging/spec.fc23.in M packaging/spec.fc24.in 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61444 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4610c7a2d78f18427bda3093213f04192cd1670a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.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: protection from NPE in the initialization of RunVm
by ahadas＠redhat.com 27 Jul '16

27 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: protection from NPE in the initialization of RunVm ...................................................................... core: protection from NPE in the initialization of RunVm Need to verify that the VM to run exists before accessing it. Change-Id: I2e3e7ff30777077cb1e96581574c484d7f71d65b Bug-Url: https://bugzilla.redhat.com/1360265 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/61395 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e3e7ff30777077cb1e96581574c484d7f71d65b Gerrit-PatchSet: 3 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[ovirt-engine-4.0.2]: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurel...
by oourfali＠redhat.com 27 Jul '16

27 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ...................................................................... aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD should not be displayed in the log file. Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Bug-Url: https://bugzilla.redhat.com/1360387 Signed-off-by: Ravi Nori <rnori(a)redhat.com> (cherry picked from commit ad7138d73787d1e93de6fcf1d0fe7f6bd8887eee) --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61443 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(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]: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurel...
by oourfali＠redhat.com 27 Jul '16

27 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ...................................................................... aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD should not be displayed in the log file. Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Bug-Url: https://bugzilla.redhat.com/1360387 Signed-off-by: Ravi Nori <rnori(a)redhat.com> (cherry picked from commit ad7138d73787d1e93de6fcf1d0fe7f6bd8887eee) --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61442 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.0]: build: oVirt Live 4.0.2 RC2
by sbonazzo＠redhat.com 27 Jul '16

27 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: oVirt Live 4.0.2 RC2 ...................................................................... build: oVirt Live 4.0.2 RC2 Change-Id: I2485d7506b4e5784c6075d32ee611f2d0611c2e9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61441 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2485d7506b4e5784c6075d32ee611f2d0611c2e9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurel...
by sbonazzo＠redhat.com 27 Jul '16

27 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ...................................................................... aaa: ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD handled insecurely in log file ENGINE_HTTPS_PKI_TRUST_STORE_PASSWORD should not be displayed in the log file. Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Bug-Url: https://bugzilla.redhat.com/1360387 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61430 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11f2239dd6cbc8f624a8b14655d071823fae8e37 Gerrit-PatchSet: 2 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: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0