- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: packaging: updated dependencies on vdsm
by msivak＠redhat.com 12 Sep '14

12 Sep '14

Martin Sivák has submitted this change and it was merged. Change subject: packaging: updated dependencies on vdsm ...................................................................... packaging: updated dependencies on vdsm Require latest vdsm in order to ensure it's updated. Change-Id: I5c5de8ab42a6640da25e1ddbfc3c0020340cd306 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 8 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32841 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c5de8ab42a6640da25e1ddbfc3c0020340cd306 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: Generate empty list for empty option
by juan.hernandez＠redhat.com 12 Sep '14

12 Sep '14

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Generate empty list for empty option ...................................................................... cli: Generate empty list for empty option Currently when a command receives an option that represents a list of values but without any value the CLI automatically adds the string "None" as the only value. This can cause issues. What the CLI should generate in this cases is an empty list. Change-Id: Ic95dadcbfd7f77b49524df0ffd05dcdcf7af8f1e Bug-Url: https://bugzilla.redhat.com/1085463 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/ovirtcli/command/command.py 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32783 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic95dadcbfd7f77b49524df0ffd05dcdcf7af8f1e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: db: separating seq creation from table creation.
by emesika＠redhat.com 12 Sep '14

12 Sep '14

Eli Mesika has submitted this change and it was merged. Change subject: db: separating seq creation from table creation. ...................................................................... db: separating seq creation from table creation. Separating sequence creation from table creation script. this will have in future version squashing and review of patches. Change-Id: Icfec6b09ea1e7a045f3e3d491c1a10e08a3c6188 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A packaging/dbscripts/create_sequences.sql M packaging/dbscripts/create_tables.sql M packaging/dbscripts/dbfunc-common.sh 3 files changed, 8 insertions(+), 10 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved Yair Zaslavsky: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32809 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfec6b09ea1e7a045f3e3d491c1a10e08a3c6188 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: Update mom version to 0.4.2
by alitke＠redhat.com 11 Sep '14

11 Sep '14

Adam Litke has submitted this change and it was merged. Change subject: Update mom version to 0.4.2 ...................................................................... Update mom version to 0.4.2 Change-Id: Ifabf01bfc9d5b4d7b9c5109860f6604189b2d695 Signed-off-by: Adam Litke <alitke(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/32834 --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32834 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifabf01bfc9d5b4d7b9c5109860f6604189b2d695 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: engine_page: use augeas instead of Management()
by dougsland＠redhat.com 11 Sep '14

11 Sep '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: use augeas instead of Management() ...................................................................... engine_page: use augeas instead of Management() To manage values in /etc/default/ovirt we use Management() which oVirt Node provides. However, for such task, we should manage the values to be added into /etc/default/ovirt directly in the Plugin VDSM to avoid any possible issue involving other layer. Change-Id: I063836cd373bcf81a4b3f07848d6e0240d4ef959 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1088875 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 13 insertions(+), 3 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32832 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I063836cd373bcf81a4b3f07848d6e0240d4ef959 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: use augeas instead of Management()
by fabiand＠redhat.com 11 Sep '14

11 Sep '14

Fabian Deutsch has submitted this change and it was merged. Change subject: engine_page: use augeas instead of Management() ...................................................................... engine_page: use augeas instead of Management() To manage values in /etc/default/ovirt we use Management() which oVirt Node provides. However, for such task, we should manage the values to be added into /etc/default/ovirt directly in the Plugin VDSM to avoid any possible issue involving other layer. Change-Id: I063836cd373bcf81a4b3f07848d6e0240d4ef959 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1088875 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 13 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Douglas Schilling Landgraf: Verified -- To view, visit http://gerrit.ovirt.org/32148 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I063836cd373bcf81a4b3f07848d6e0240d4ef959 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: do not display non password authn profiles within login
by piotr.kliczewski＠gmail.com 11 Sep '14

11 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: aaa: do not display non password authn profiles within login ...................................................................... aaa: do not display non password authn profiles within login they are useless in this location. Topic: AAA Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120720 Change-Id: I74df82dcead1b88124b44cbec2ea43ef85768fce Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAAAProfileListQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/aaa/ProfileEntry.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/LoginModel.java 6 files changed, 22 insertions(+), 14 deletions(-) Approvals: Alon Bar-Lev: Verified Alexander Wels: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32801 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74df82dcead1b88124b44cbec2ea43ef85768fce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: Fix group population for legacy provider
by piotr.kliczewski＠gmail.com 11 Sep '14

11 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: aaa: Fix group population for legacy provider ...................................................................... aaa: Fix group population for legacy provider Legacy provider did not populate groups for added users in a correct way. Change-Id: Id1066170758ecddd296b6f253c5bd386cea22008 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120720 Topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapBrokerCommandBase.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapSearchByQueryParameters.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/LdapSearchUserByQueryCommand.java 4 files changed, 29 insertions(+), 11 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32812 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id1066170758ecddd296b6f253c5bd386cea22008 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: ovirt-engine_upgrade_stable_to_3.4: more cleanups
by eedri＠redhat.com 11 Sep '14

11 Sep '14

Eyal Edri has submitted this change and it was merged. Change subject: ovirt-engine_upgrade_stable_to_3.4: more cleanups ...................................................................... ovirt-engine_upgrade_stable_to_3.4: more cleanups Added cleanup of /etc/ovirt-engine and /etc/ovirt-engine-setup.conf.d. Change-Id: I80678687af42de43ee45505c2817a7d2c57faa64 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/ovirt-engine_upgrade_stable_to_3.4/upgrade.sh 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Eyal Edri: Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32816 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80678687af42de43ee45505c2817a7d2c57faa64 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine:slave's mac in Broker, to lower case
by masayag＠redhat.com 11 Sep '14

11 Sep '14

Moti Asayag has submitted this change and it was merged. Change subject: engine:slave's mac in Broker, to lower case ...................................................................... engine:slave's mac in Broker, to lower case slave's mac in older vdsm versions, is in upper case, resulting slave's mac is saved and repressent in upper case Change-Id: Ibfebc2aea4bc3605253bfb072fc4ad19c0fd2270 Bug-Url: https://bugzilla.redhat.com/1132422 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Eliraz Levi: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32808 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibfebc2aea4bc3605253bfb072fc4ad19c0fd2270 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0