- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: sdk: verify credentials before storing proxy in cache
by mpastern＠redhat.com 16 Aug '12

16 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: verify credentials before storing proxy in cache ...................................................................... sdk: verify credentials before storing proxy in cache Change-Id: I7c38ae6f1830ec11fecaf8dca9cfa94e1c3b8e42 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/ovirtsdk/api.py 2 files changed, 6 insertions(+), 6 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7248 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c38ae6f1830ec11fecaf8dca9cfa94e1c3b8e42 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[master]: sdk: cleanup
by mpastern＠redhat.com 16 Aug '12

16 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: cleanup ...................................................................... sdk: cleanup Change-Id: I3d5cf935c0474d1a10ba56d0409698989ed06d5a Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/errors.py D src/ovirtsdk/infrastructure/threadpool.py 2 files changed, 0 insertions(+), 105 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7215 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d5cf935c0474d1a10ba56d0409698989ed06d5a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[master]: sdk: implement insecure flag #848046
by mpastern＠redhat.com 16 Aug '12

16 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: implement insecure flag #848046 ...................................................................... sdk: implement insecure flag #848046 by default if not specified key_file, cert_file, ca_file, SSL connection initiation will fail, unless raised 'insecure' flag https://bugzilla.redhat.com/show_bug.cgi?id=848046 Change-Id: Ib3ab575aae4d5615de76b79ed93234aec354efed Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/connectionspool.py M src/ovirtsdk/infrastructure/errors.py M src/ovirtsdk/web/connection.py 5 files changed, 23 insertions(+), 6 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7211 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib3ab575aae4d5615de76b79ed93234aec354efed Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: implement server identity check
by mpastern＠redhat.com 16 Aug '12

16 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: implement server identity check ...................................................................... sdk: implement server identity check Change-Id: I5daf24ed7e5dfd7d0e2fb117bf62cf2e5f209148 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/codegen/entrypoint/entrypoint.py M src/codegen/main.py M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/connectionspool.py M src/ovirtsdk/web/connection.py A src/ovirtsdk/web/httpsconnection.py 6 files changed, 77 insertions(+), 11 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7209 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5daf24ed7e5dfd7d0e2fb117bf62cf2e5f209148 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: collection-based-options definition doesnt look like cl...
by mpastern＠redhat.com 16 Aug '12

16 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: collection-based-options definition doesnt look like cli format #833788 ...................................................................... cli: collection-based-options definition doesnt look like cli format #833788 https://bugzilla.redhat.com/show_bug.cgi?id=833788 new collection based arguments syntax is: --<type>.<collection-obj>.<collection-member> "<collection-item>.<collection-item-property>=value,..." e.g: --power_management-options-option "option.name=n1,option.value=v1" --power_management-options-option "option.name=n2,option.value=v2" Change-Id: Ica82f155511beb0726e1b4f32d31d7438ee56612 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/cli/command/command.py M src/cli/messages.py M src/ovirtcli/command/action.py M src/ovirtcli/command/command.py M src/ovirtcli/command/create.py M src/ovirtcli/command/delete.py M src/ovirtcli/command/update.py A src/ovirtcli/utils/multivaluedict.py 8 files changed, 111 insertions(+), 10 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7207 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica82f155511beb0726e1b4f32d31d7438ee56612 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: keep install path when package name altered (#845...
by ydary＠redhat.com 16 Aug '12

16 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: keep install path when package name altered (#845024) ...................................................................... packaging: keep install path when package name altered (#845024) https://bugzilla.redhat.com/show_bug.cgi?id=845024 Change-Id: I660ed747b058e307a1672a9dc23a4d0eb983946a Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M Makefile M packaging/ovirt-engine-dwh.spec.in 2 files changed, 6 insertions(+), 6 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7254 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I660ed747b058e307a1672a9dc23a4d0eb983946a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Configure image-uploader on engine-setup. (#BZ788...
by juan.hernandez＠redhat.com 16 Aug '12

16 Aug '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Configure image-uploader on engine-setup. (#BZ788079) ...................................................................... packaging: Configure image-uploader on engine-setup. (#BZ788079) https://bugzilla.redhat.com/show_bug.cgi?id=788079 This patch configures image-uplodaer on engine-setup. Change-Id: I9bff6a1580119ead4e0609f253b8df3f2a5e46e9 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-setup.py 2 files changed, 2 insertions(+), 1 deletion(-) Approvals: Alex Lourie: Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved Kiril Nesenko: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7253 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bff6a1580119ead4e0609f253b8df3f2a5e46e9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: ManageDomainsResult returns null when DB error raise...
by yzaslavs＠redhat.com 16 Aug '12

16 Aug '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: ManageDomainsResult returns null when DB error raise it (#818479) ...................................................................... engine: ManageDomainsResult returns null when DB error raise it (#818479) https://bugzilla.redhat.com/show_bug.cgi?id=818479 Added variable of default message, so if error occurred during varification of the paramerts, the default message will be the output. Change-Id: If21f117208a53a80ebfefbbe4f6a0c188e35a3bd Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ConfigurationProvider.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ManageDomains.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ManageDomainsResult.java 3 files changed, 89 insertions(+), 41 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7202 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If21f117208a53a80ebfefbbe4f6a0c188e35a3bd Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: Block 3.1 upgrade if DC/cluster 2.2 present
by juan.hernandez＠redhat.com 16 Aug '12

16 Aug '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Block 3.1 upgrade if DC/cluster 2.2 present ...................................................................... packaging: Block 3.1 upgrade if DC/cluster 2.2 present * Added the DC + cluster version check before starting the upgrade. If 2.2 version is found, raises an Exception. Change-Id: I0277b73628872e8c0fa163fee0b89bd5cb724f05 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/fedora/setup/engine-upgrade.py 1 file changed, 42 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5186 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0277b73628872e8c0fa163fee0b89bd5cb724f05 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fix exception for unlock VM.
by masayag＠redhat.com 15 Aug '12

15 Aug '12

Moti Asayag has submitted this change and it was merged. Change subject: core: Fix exception for unlock VM. ...................................................................... core: Fix exception for unlock VM. Fix exception to use unlock VM for VM instead vm id Change-Id: Ia696ef0f7e0a66a99e8f9c53458961d26d77d5b7 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7222 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia696ef0f7e0a66a99e8f9c53458961d26d77d5b7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>

1 0