April 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: pki: update POSIX sh compatibility
by bazulay＠redhat.com 18 Apr '13

18 Apr '13

Barak Azulay has submitted this change and it was merged. Change subject: pki: update POSIX sh compatibility ...................................................................... pki: update POSIX sh compatibility Change-Id: I3ae7b6a9d88b87d134676acbd216a075bbad76f8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/conf/ca/CreateCA.sh M backend/manager/conf/ca/SignReq.sh M backend/manager/conf/ca/installCA.sh 3 files changed, 84 insertions(+), 91 deletions(-) Approvals: Barak Azulay: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13901 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ae7b6a9d88b87d134676acbd216a075bbad76f8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Fix the phrasing of error message
by dfediuck＠redhat.com 18 Apr '13

18 Apr '13

Doron Fediuck has submitted this change and it was merged. Change subject: webadmin: Fix the phrasing of error message ...................................................................... webadmin: Fix the phrasing of error message message was: Cannot edit VM. Dedicated Host Cluster is not identical to VM Cluster. changed to: Cannot edit VM. VM is pinned to a specific host. The host's cluster must be the same as the selected VM cluster. Change-Id: I298bacc9ce5c2ca9729d79ee3136a71360d329d5 Bug-Url: https://bugzilla.redhat.com/950693 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 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 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14054 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I298bacc9ce5c2ca9729d79ee3136a71360d329d5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: build: re-use external gettext
by Alon Bar-Lev 18 Apr '13

18 Apr '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: re-use external gettext ...................................................................... build: re-use external gettext Change-Id: Ie10b6003d1c5ebb32e9c6cea216a56c9b5422e15 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore M ChangeLog M Makefile.am M configure.ac M ovirt-host-deploy.spec.in 5 files changed, 3 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie10b6003d1c5ebb32e9c6cea216a56c9b5422e15 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: core : fix message when removing template from export domain
by mlipchuk＠redhat.com 18 Apr '13

18 Apr '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core : fix message when removing template from export domain ...................................................................... core : fix message when removing template from export domain Change-Id: I44b8f026bf381c114157f557375381fff4ca8d8c Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Maor Lipchuk: Looks good to me, approved Liron Ar: Verified -- To view, visit http://gerrit.ovirt.org/14056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44b8f026bf381c114157f557375381fff4ca8d8c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in otopi[master]: build: re-use external gettext
by Alon Bar-Lev 18 Apr '13

18 Apr '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: re-use external gettext ...................................................................... build: re-use external gettext Change-Id: Ic124ffe0e24eb9b469535db221f64bdce2ff0351 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore M ChangeLog M Makefile.am M configure.ac M otopi.spec.in 5 files changed, 3 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14036 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic124ffe0e24eb9b469535db221f64bdce2ff0351 Gerrit-PatchSet: 3 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: gluster: starting glusterd after install
by Alon Bar-Lev 18 Apr '13

18 Apr '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: gluster: starting glusterd after install ...................................................................... gluster: starting glusterd after install glusterd service was not started after installing. Added an event which will get triggered after the gluster installation which starts the glusterd service. Change-Id: I4f65ea6cbb54c844fb42ad1a14c61c438ac0865a Signed-off-by: Aravinda VK <avishwan(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/gluster/packages.py 2 files changed, 14 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Aravinda VK: Verified -- To view, visit http://gerrit.ovirt.org/14030 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f65ea6cbb54c844fb42ad1a14c61c438ac0865a Gerrit-PatchSet: 7 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Aravinda VK <avishwan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Aravinda VK <avishwan(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Auto recovery ignores hosts with problematic NICs
by masayag＠redhat.com 18 Apr '13

18 Apr '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Auto recovery ignores hosts with problematic NICs ...................................................................... engine: Auto recovery ignores hosts with problematic NICs Such hosts cannot be automatically recovered since the NICs are down and the host will just go back to non-operational state. Change-Id: I70bff4ba5541156c31fb8afdd38ff6f706c0d869 Bug-Url: https://bugzilla.redhat.com/953115 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AutoRecoveryManagerTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 3 files changed, 124 insertions(+), 50 deletions(-) Approvals: Mike Kolesnik: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14047 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70bff4ba5541156c31fb8afdd38ff6f706c0d869 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>

1 0

Change in ovirt-engine[master]: findbugs: Method fail to close stream
by shavivi＠redhat.com 18 Apr '13

18 Apr '13

Shahar Havivi has submitted this change and it was merged. Change subject: findbugs: Method fail to close stream ...................................................................... findbugs: Method fail to close stream Change-Id: I5bea678c1aaeeb6dfe92b16e380e4bc631e3d09c Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/BaseDAOTestCase.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DbFacadeDAOTest.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/archivers/tar/TarTest.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ssh/TestCommon.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ssh/TransferTest.java 5 files changed, 69 insertions(+), 11 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14037 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5bea678c1aaeeb6dfe92b16e380e4bc631e3d09c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.2]: packaging: post ovirt-image-uploader-3.2.2
by knesenko＠redhat.com 18 Apr '13

18 Apr '13

Kiril Nesenko has submitted this change and it was merged. Change subject: packaging: post ovirt-image-uploader-3.2.2 ...................................................................... packaging: post ovirt-image-uploader-3.2.2 Change-Id: I174d8008553764953b19727b3223eb809322aa99 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M configure.ac 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14045 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I174d8008553764953b19727b3223eb809322aa99 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine-cli[cli_3.2]: cli: Implement Session-TTL header support #928314
by mpastern＠redhat.com 18 Apr '13

18 Apr '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: Implement Session-TTL header support #928314 ...................................................................... cli: Implement Session-TTL header support #928314 https://bugzilla.redhat.com/show_bug.cgi?id=928314 Change-Id: I4a404493235255d795c3554af4ffe7db471f2cc1 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-cli.spec.in M setup.py M src/cli/settings.py M src/ovirtcli/command/connect.py M src/ovirtcli/context.py M src/ovirtcli/options.py M src/ovirtcli/settings.py M src/ovirtcli/shell/connectcmdshell.py 8 files changed, 42 insertions(+), 20 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14031 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a404493235255d795c3554af4ffe7db471f2cc1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0