October 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: change cluster-policy header text
by gchaplik＠redhat.com 16 Oct '13

16 Oct '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: change cluster-policy header text ...................................................................... webadmin: change cluster-policy header text Change cluster column header from "Attached Clusters" to "Clusters on this policy" Change-Id: I1e25cfbeb2c21566fd22c5adb144fcbc225beec2 Bug-Url: https://bugzilla.redhat.com/995238 Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20156 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e25cfbeb2c21566fd22c5adb144fcbc225beec2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fencing flow fails to get to secondery PM...
by emesika＠redhat.com 16 Oct '13

16 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: core: Fencing flow fails to get to secondery PM... ...................................................................... core: Fencing flow fails to get to secondery PM... Fencing flow fails to get to secondary PM (primary is deliberately configured with wrong params) Reason for the thrown exception was an unmatched parenthesis in the log message. The second parameter passed was {2) instead of {2} Change-Id: Iec2e0d6b77f0e01e36c01ea1d397862e293ee7ef Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1013654 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19718 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec2e0d6b77f0e01e36c01ea1d397862e293ee7ef Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jasperreports-server-rpm[master]: packaging: updated spec file for JRS 5.2.0
by ydary＠redhat.com 16 Oct '13

16 Oct '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: updated spec file for JRS 5.2.0 ...................................................................... packaging: updated spec file for JRS 5.2.0 Change-Id: I78d829728c5f986a2be045c014d9c0bfb62d2e37 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M jasperreports-server.spec 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20239 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78d829728c5f986a2be045c014d9c0bfb62d2e37 Gerrit-PatchSet: 1 Gerrit-Project: jasperreports-server-rpm Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: [dal] - Fix storing of app_list and vm_fqdn values
by ofrenkel＠redhat.com 16 Oct '13

16 Oct '13

Omer Frenkel has submitted this change and it was merged. Change subject: [dal] - Fix storing of app_list and vm_fqdn values ...................................................................... [dal] - Fix storing of app_list and vm_fqdn values The application list and the fqdn have not been stored to the vm_dynamic table due to the batchmapper. Which will cause SSO not to be working. Change-Id: I49455c2531e93094cb7543df6891033b12a2d86a Bug-Url: https://bugzilla.redhat.com/1015574 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDAODbFacadeImpl.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Vinzenz Feenstra: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20236 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49455c2531e93094cb7543df6891033b12a2d86a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: bogous 'help show event' output
by mpastern＠redhat.com 16 Oct '13

16 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: bogous 'help show event' output ...................................................................... cli: bogous 'help show event' output 'help show event' command shows unreadable text in the parameter description Change-Id: Ic8dfe2a5d4fd7830a184f87deb51aabd2fcd0d41 Bug-Url: https://bugzilla.redhat.com/980854 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M src/ovirtcli/utils/methodhelper.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16780 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8dfe2a5d4fd7830a184f87deb51aabd2fcd0d41 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli 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 : --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