December 2012 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[master]: tune/iosched: set deadline iosched using udev rules
by Alon Bar-Lev 03 Dec '12

03 Dec '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: tune/iosched: set deadline iosched using udev rules ...................................................................... tune/iosched: set deadline iosched using udev rules Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=870159 Change-Id: I96e5710729e79dd222047c03eb81b8cef1c0030d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M ovirt-host-deploy.spec.in M po/POTFILES.in M src/plugins/ovirt-host-deploy/tune/Makefile.am M src/plugins/ovirt-host-deploy/tune/__init__.py A src/plugins/ovirt-host-deploy/tune/iosched.py A src/plugins/ovirt-host-deploy/tune/ovirt-iosched.rules 7 files changed, 108 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9681 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96e5710729e79dd222047c03eb81b8cef1c0030d 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-host-deploy[master]: tune: move tuned plugin to new category
by Alon Bar-Lev 03 Dec '12

03 Dec '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: tune: move tuned plugin to new category ...................................................................... tune: move tuned plugin to new category Change-Id: I66a89598e069ec2bb8f267d48a4e558089129a5c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M configure.ac M po/POTFILES.in M src/plugins/ovirt-host-deploy/Makefile.am A src/plugins/ovirt-host-deploy/tune/Makefile.am A src/plugins/ovirt-host-deploy/tune/__init__.py R src/plugins/ovirt-host-deploy/tune/tuned.py M src/plugins/ovirt-host-deploy/vdsm/Makefile.am M src/plugins/ovirt-host-deploy/vdsm/__init__.py 8 files changed, 70 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9680 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66a89598e069ec2bb8f267d48a4e558089129a5c 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-host-deploy[master]: core: misc: add space after dot for confirmation
by Alon Bar-Lev 03 Dec '12

03 Dec '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: misc: add space after dot for confirmation ...................................................................... core: misc: add space after dot for confirmation Change-Id: I62e20cc4702eaef424d0856aebdd0ebda70cc6ea Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/core/misc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9676 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62e20cc4702eaef424d0856aebdd0ebda70cc6ea 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-host-deploy[master]: core: misc: fix typo in confirm message
by Alon Bar-Lev 03 Dec '12

03 Dec '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: misc: fix typo in confirm message ...................................................................... core: misc: fix typo in confirm message Change-Id: I415f7ef8aa017c5cb276f1f7eb03b037a5ca0156 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/core/misc.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9677 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I415f7ef8aa017c5cb276f1f7eb03b037a5ca0156 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: Remove PGHack
by juan.hernandez＠redhat.com 03 Dec '12

03 Dec '12

Juan Hernandez has submitted this change and it was merged. Change subject: core: Remove PGHack ...................................................................... core: Remove PGHack This is no longer needed. Change-Id: Id3ebd70516f427172ad206ad204ac95ec2a5a29f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- D backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/PGHack.java 1 file changed, 0 insertions(+), 73 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9578 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3ebd70516f427172ad206ad204ac95ec2a5a29f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: cleanup and delete EnumCompat
by rgolan＠redhat.com 03 Dec '12

03 Dec '12

Roy Golan has submitted this change and it was merged. Change subject: core: cleanup and delete EnumCompat ...................................................................... core: cleanup and delete EnumCompat Use the native enum methods and generify uses of enums with IDs Change-Id: I0383ea74495f74ff1727561644017d8b3bae7e6f Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogSeverity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/AdRefStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DateEnumForSearch.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Disk.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Identifiable.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ImageStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/QuotaEnforcementTypeEnum.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomainSharedStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePoolStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSNiceType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmOsType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmPoolType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplateStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmType.java D backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/EnumCompat.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/DateEnumValueAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/EnumNameAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/EnumValueAutoCompleter.java M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/Jedi.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java 25 files changed, 83 insertions(+), 122 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9516 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0383ea74495f74ff1727561644017d8b3bae7e6f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Removing jdk7 extended CTOR
by yzaslavs＠redhat.com 03 Dec '12

03 Dec '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Removing jdk7 extended CTOR ...................................................................... core: Removing jdk7 extended CTOR Change-Id: If25ef37b0d9ff79737e7dfe8283322acb70b1dcb Bug-Url: https://bugzilla.redhat.com/878964 signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/exceptions/InitializationException.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9666 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If25ef37b0d9ff79737e7dfe8283322acb70b1dcb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Removing unused code GetAvailableClusterVersionsQuer...
by mkublin＠redhat.com 03 Dec '12

03 Dec '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Removing unused code GetAvailableClusterVersionsQuery ...................................................................... engine: Removing unused code GetAvailableClusterVersionsQuery Looks like that no one is using that query, so removing it Change-Id: If60af12692a532d25311db64fb547a57f4a8052e Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAvailableClusterVersionsQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetAvailableClusterVersionsParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java 5 files changed, 0 insertions(+), 112 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8946 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If60af12692a532d25311db64fb547a57f4a8052e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Removing last host in status UP in pool
by mkublin＠redhat.com 03 Dec '12

03 Dec '12

Michael Kublin has submitted this change and it was merged. Change subject: core: Removing last host in status UP in pool ...................................................................... core: Removing last host in status UP in pool If host can not connect to storage it will be moved to Non-Operational. A reson that host was left at sttaus UP was to use it as fence agent when it is the only host at pool, it is possible to use a non-operational host as fence agent Change-Id: I42b3b45e882fe84eb869c82fe83eadb7c1da5152 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectHostToStoragePoolServersCommand.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/action/StoragePoolParametersBase.java 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/irsbroker/IrsBrokerCommand.java 6 files changed, 25 insertions(+), 79 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9566 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I42b3b45e882fe84eb869c82fe83eadb7c1da5152 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Removed useless checks at ConnectHostToStoragePoolSe...
by mkublin＠redhat.com 03 Dec '12

03 Dec '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Removed useless checks at ConnectHostToStoragePoolServersCommand ...................................................................... engine: Removed useless checks at ConnectHostToStoragePoolServersCommand The following checks at canDoaction are useless, the command is called only once form InitVdsOnUp and the same checks are performed before a call to command Change-Id: I25c6f4db1108b513663b81fbc7956e66edd9c797 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectHostToStoragePoolServersCommand.java 1 file changed, 21 insertions(+), 27 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9567 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25c6f4db1108b513663b81fbc7956e66edd9c797 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0