- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi : --parent-tag-id option doesn't exists in 'update t...
by mpastern＠redhat.com 16 Oct '13

16 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi : --parent-tag-id option doesn't exists in 'update tag' options (#1006905) ...................................................................... restapi : --parent-tag-id option doesn't exists in 'update tag' options (#1006905) update tag does not display --parent-tag-id on pressing tab as one of the valid options Change-Id: I7c439521bb8432dd4d9f3ed52b1dd24a9bfc3712 Bug-Url: https://bugzilla.redhat.com/1006905 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19952 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c439521bb8432dd4d9f3ed52b1dd24a9bfc3712 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi : Missing permissions link ast /vmpools url
by mpastern＠redhat.com 16 Oct '13

16 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi : Missing permissions link ast /vmpools url ...................................................................... restapi : Missing permissions link ast /vmpools url There is missing <link href="/api/vmpools/{id}/permissions" rel="permissions"/> element for vmpools, only at /vmpools url. It is ok for /vmpools/{vmpool:id}. Change-Id: Id9ce7e04680cf7182e89406e76bcf8d770e3ccfb Bug-Url: https://bugzilla.redhat.com/977304 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16777 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9ce7e04680cf7182e89406e76bcf8d770e3ccfb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: editConfigContent: compare strings
by sbonazzo＠redhat.com 16 Oct '13

16 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: editConfigContent: compare strings ...................................................................... packaging: setup: editConfigContent: compare strings Make editConfigContent compare values as strings when deciding if value was changed. Without this change, an excessive comment line will be added to /var/lib/pgsql/data/postgresql.conf after comparing int(150) and str(150). Change-Id: Ia747d05416ff0ed1fbd74f0d4cce2c9f6b621755 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/util.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20224 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia747d05416ff0ed1fbd74f0d4cce2c9f6b621755 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: provider: glance images listing size and limits
by amureini＠redhat.com 16 Oct '13

16 Oct '13

Allon Mureinik has submitted this change and it was merged. Change subject: provider: glance images listing size and limits ...................................................................... provider: glance images listing size and limits This patch adds two configuration parameters to control the glance image listing. In fact OpenStack glance servers by default return a limited number of images (limit_param_default) and it's up to the client to either request a longer list (using the "limit" parameter) or to iterate with multiple requests. GlanceImageListSize defines the number of images that should be returned in each request (20 as the default in glance client). GlanceImageTotalListSize defines the total number of images that should be transferred from the server (with multiple requests if necessary). Change-Id: I7b534caccbdcf121b81571384e3420f1a6efde00 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 7 files changed, 54 insertions(+), 7 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19223 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b534caccbdcf121b81571384e3420f1a6efde00 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: Add 'values' to configfile.py
by iheim＠redhat.com 16 Oct '13

16 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: Add 'values' to configfile.py ...................................................................... packaging: setup: Add 'values' to configfile.py Related-To: https://bugzilla.redhat.com/1013347 Change-Id: Id27fcd1335a2a264d2f1f6000815c7ddec578fbe Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/pythonlib/ovirt_engine/configfile.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved David Caro: Verified -- To view, visit http://gerrit.ovirt.org/20221 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id27fcd1335a2a264d2f1f6000815c7ddec578fbe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: restapi: Inconsistent schema implementation in PayloadFile #...
by iheim＠redhat.com 16 Oct '13

16 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: restapi: Inconsistent schema implementation in PayloadFile #1019618 ...................................................................... restapi: Inconsistent schema implementation in PayloadFile #1019618 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1019618 Change-Id: I82ec264bb18798fd911143741b60a4e9469efd48 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 18 insertions(+), 28 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20163 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82ec264bb18798fd911143741b60a4e9469efd48 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Fix the numbering of the last db upgrade script
by iheim＠redhat.com 16 Oct '13

16 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: core: Fix the numbering of the last db upgrade script ...................................................................... core: Fix the numbering of the last db upgrade script Change-Id: Ib4f03433f502a092a53ac0e43ba458bc7fa52f64 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- R packaging/dbscripts/upgrade/03_03_0950_pin_to_host_first_in_powersaving_policy.sql 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Verified Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20217 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4f03433f502a092a53ac0e43ba458bc7fa52f64 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: adding HA score with proper scheduling policies
by dfediuck＠redhat.com 16 Oct '13

16 Oct '13

Doron Fediuck has submitted this change and it was merged. Change subject: core: adding HA score with proper scheduling policies ...................................................................... core: adding HA score with proper scheduling policies This patch adds the HA score to VdsStatistics, filling it up where relevant, and using it when scheduling the hosted engine VM. Change-Id: I24879ed64cab829969556fd71786b32d3aaaf0c7 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/PolicyUnitImpl.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/HostedEngineHAClusterFilterPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/HostedEngineHAClusterWeightPolicyUnit.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatistics.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStatisticsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/03_04_0030_add_ha_score_to_vds_statistics.sql A packaging/dbscripts/upgrade/03_04_0040_add_ha_policy_units.sql M packaging/dbscripts/vds_sp.sql 20 files changed, 169 insertions(+), 10 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24879ed64cab829969556fd71786b32d3aaaf0c7 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Sort configure roles
by gchaplik＠redhat.com 16 Oct '13

16 Oct '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Sort configure roles ...................................................................... webadmin: Sort configure roles - Added sorting to 'Configure->roles' Change-Id: If22cba5d1114cc43be39c2e770b770582ac18033 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=953570 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20207 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If22cba5d1114cc43be39c2e770b770582ac18033 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: revert nfs config on cleanup
by iheim＠redhat.com 15 Oct '13

15 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: revert nfs config on cleanup ...................................................................... packaging: setup: revert nfs config on cleanup Revert /etc/sysconfig/nfs to its original contents on cleanup. Added a function editConfigContent in util to generalize edit loops that read conf files and do some changes to them. Used it to edit /etc/sysconfig/nfs as well as /etc/httpd/conf.d/ssl.conf and /var/lib/pgsql/data/postgresql.conf. Bug-Url: https://bugzilla.redhat.com/1013347 Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/util.py M packaging/setup/plugins/ovirt-engine-setup/apache/ssl.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 4 files changed, 203 insertions(+), 79 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0