Change in ovirt-engine[master]: core: setting DiskVmElement on extend Cinder disk
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: core: setting DiskVmElement on extend Cinder disk
......................................................................
core: setting DiskVmElement on extend Cinder disk
UpdateVmDiskCommand -> buildExtendCinderDiskParameters:
diskInterface is missing from DiskVmElement, hence the command
is failing on validation. To mitigate the issue, fetched and cloned
oldDiskVmElement.
Change-Id: I0063ee101bba67223040bed43fa775608345d11c
Bug-Url: https://bugzilla.redhat.com/1359489
Signed-off-by: Daniel Erez <derez(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/UpdateVmDiskCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Daniel Erez: Verified
Allon Mureinik: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/61264
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0063ee101bba67223040bed43fa775608345d11c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-guest-agent[master]: windows: Apply timer usage parameters for best performance
by vfeenstr@redhat.com
Vinzenz Feenstra has submitted this change and it was merged.
Change subject: windows: Apply timer usage parameters for best performance
......................................................................
windows: Apply timer usage parameters for best performance
The configuration should be now done like this:
- For versions above Windows 2003/Vista we're going to delete the
USEPLATFORMCLOCK value to let the windows kernel decide the best
option by itself
This functionality can be disabled by setting the value
general.apply_timer_configuration in the configuration to false
Change-Id: I8a344002ac4206a6125755a4b0195777cdfadcf9
Bug-Url: https://bugzilla.redhat.com/1335309
Bug-Url: https://bugzilla.redhat.com/1354532
Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com>
---
M configurations/default.ini
M configurations/ovirt-guest-agent.ini
M ovirt-guest-agent/GuestAgentWin32.py
3 files changed, 23 insertions(+), 10 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, but someone else must approve
Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/60635
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8a344002ac4206a6125755a4b0195777cdfadcf9
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-guest-agent
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr(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: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: core: Change default value of DwhHeartBeatInterval
by piotr.kliczewski@gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: core: Change default value of DwhHeartBeatInterval
......................................................................
core: Change default value of DwhHeartBeatInterval
Lowers default value of DwhHeartBeatInterval in vdc_options from 30
seconds to 15 seconds (if users changed default value using
engine-config we will preserve value defined by users).
Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b
Bug-Url: https://bugzilla.redhat.com/1344935
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
M packaging/etc/engine-config/engine-config.properties
3 files changed, 5 insertions(+), 2 deletions(-)
Approvals:
Martin Peřina: Verified; Looks good to me, approved
Shirly Radco: Looks good to me, but someone else must approve
Juan Hernandez: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/61265
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0.2]: userportal: webadmin: Use correct trust store for HTTPS conn...
by piotr.kliczewski@gmail.com
Piotr Kliczewski has submitted this change and it was merged.
Change subject: userportal: webadmin: Use correct trust store for HTTPS connections
......................................................................
userportal: webadmin: Use correct trust store for HTTPS connections
In oVirt 4.0 we have changed handling of external CA used to sign HTTPS
certificate (more details in https://bugzilla.redhat.com/1336838 ).
This patch fixes trust store used to connect to reports instance.
Change-Id: I2ce813a507b25bf221eb4a1159b58748a00ceab6
Bug-Url: https://bugzilla.redhat.com/1358160
Signed-off-by: Martin Perina <mperina(a)redhat.com>
(cherry picked from commit 56822543cac46130514404eb4bb8e53d5d306ecd)
---
M backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Martin Peřina: Verified
Alexander Wels: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/61192
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2ce813a507b25bf221eb4a1159b58748a00ceab6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0.2
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0.2]: webadmin: remove tooltip underline from table column headers
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: webadmin: remove tooltip underline from table column headers
......................................................................
webadmin: remove tooltip underline from table column headers
Removed underlines from table column headers that have tool tips. This
provides consistency with PatternFly in the UX.
Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10
Bug-Url: https://bugzilla.redhat.com/1230159
Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractCheckboxHeader.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractHeader.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/SafeHtmlHeader.java
3 files changed, 7 insertions(+), 30 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Greg Sheremeta: Looks good to me, approved
Scott Dickerson: Verified
--
To view, visit https://gerrit.ovirt.org/61230
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0.2
Gerrit-Owner: Scott Dickerson <sdickers(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: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[ovirt-engine-4.0]: webadmin: remove tooltip underline from table column headers
by oourfali@redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: webadmin: remove tooltip underline from table column headers
......................................................................
webadmin: remove tooltip underline from table column headers
Removed underlines from table column headers that have tool tips. This
provides consistency with PatternFly in the UX.
Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10
Bug-Url: https://bugzilla.redhat.com/1230159
Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractCheckboxHeader.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractHeader.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/SafeHtmlHeader.java
3 files changed, 7 insertions(+), 30 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Greg Sheremeta: Looks good to me, approved
Scott Dickerson: Verified
--
To view, visit https://gerrit.ovirt.org/61215
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Scott Dickerson <sdickers(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: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in jenkins[master]: yamlize the scan_security_and_license_ovirt-engine_master job
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: yamlize the scan_security_and_license_ovirt-engine_master job
......................................................................
yamlize the scan_security_and_license_ovirt-engine_master job
Change-Id: Ia2f05310897787234079612db8839a36dddf7010
Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com>
---
A jobs/confs/yaml/jobs/scan-security-and-license-ovirt-engine.yaml
1 file changed, 39 insertions(+), 0 deletions(-)
Approvals:
Sharon Naftaly: Verified; Ready for review
Eyal Edri: Looks good to me; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/60165
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia2f05310897787234079612db8839a36dddf7010
Gerrit-PatchSet: 4
Gerrit-Project: jenkins
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: Sharon Naftaly <snaftaly(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in jenkins[master]: remove emails from setup-upgrade job
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: remove emails from setup-upgrade job
......................................................................
remove emails from setup-upgrade job
most jobs doesn't email so not reason to keep
this specific job, since we monitor the unstable
queue constantly and also we have daily unstable
report sent to infra.
In addition, this job was added to per patch
so we don't want emails sent on every patch
failing
Change-Id: I57fbba9d30296af552392cf730cc3011dc946a12
Signed-off-by: Eyal Edri <eedri(a)redhat.com>
---
M jobs/confs/yaml/templates/engine-upgrade.yaml
1 file changed, 0 insertions(+), 7 deletions(-)
Approvals:
Eyal Edri: Verified; Ready for merge
Barak Korren: Looks good to me
Gil Shinar: Looks good to me
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/61176
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I57fbba9d30296af552392cf730cc3011dc946a12
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <david(a)dcaro.es>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[master]: core: Removed duplicate LOW_UILIZATION parameter
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: core: Removed duplicate LOW_UILIZATION parameter
......................................................................
core: Removed duplicate LOW_UILIZATION parameter
The LOW_UTILIZATION parameter appeared twice in
PowerSavingBalancePolicyUnit parameters.
This patch removes one of the appearances.
Change-Id: Ia3b7928c72962d5f98dfe3eb969b94f9b0101f63
Bug-Url: https://bugzilla.redhat.com/1351556
Signed-off-by: Jenny Tokar <jtokar(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingBalancePolicyUnit.java
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Martin Sivák: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
Jenny Tokar: Verified
--
To view, visit https://gerrit.ovirt.org/61071
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3b7928c72962d5f98dfe3eb969b94f9b0101f63
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months
Change in ovirt-engine[master]: core: Change default value of DwhHeartBeatInterval
by mperina@redhat.com
Martin Peřina has submitted this change and it was merged.
Change subject: core: Change default value of DwhHeartBeatInterval
......................................................................
core: Change default value of DwhHeartBeatInterval
Lowers default value of DwhHeartBeatInterval in vdc_options from 30
seconds to 15 seconds (if users changed default value using
engine-config we will preserve value defined by users).
Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b
Bug-Url: https://bugzilla.redhat.com/1344935
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
M packaging/etc/engine-config/engine-config.properties
3 files changed, 5 insertions(+), 2 deletions(-)
Approvals:
Shirly Radco: Looks good to me, but someone else must approve
Martin Peřina: Verified; Looks good to me, approved
Eli Mesika: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/61234
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 4 months