- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Adding CIFS checkbox to Volume Popup
by gchaplik＠redhat.com 05 Aug '12

05 Aug '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Adding CIFS checkbox to Volume Popup ...................................................................... webadmin: Adding CIFS checkbox to Volume Popup A new checkbox is added to enable or disable CIFS on a particular volume while creating the volume. CIFS will be enabled by defult, the user can choose to disable it. Change-Id: I187f9642ccf33c2ac22b44c762808204b8b296bc Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.ui.xml 5 files changed, 26 insertions(+), 5 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6219 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I187f9642ccf33c2ac22b44c762808204b8b296bc Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Vijay Bellur <vbellur(a)gmail.com>

1 0

Change in ovirt-engine[master]: webadmin: making cpu validations in cluster dialog condition...
by gchaplik＠redhat.com 05 Aug '12

05 Aug '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: making cpu validations in cluster dialog conditional ...................................................................... webadmin: making cpu validations in cluster dialog conditional CPU field will accept empty values while editing the cluster, if the selected cluster already have an empty cpu value. While creating a new cluster the cpu filed is mandatory. Change-Id: If0455c48f3667e1dda4484fb8d1bbf920e036efb Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/ConfigureLocalStorageModel.java 4 files changed, 23 insertions(+), 8 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5939 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0455c48f3667e1dda4484fb8d1bbf920e036efb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: document Correlation-Id header in RSDL
by mpastern＠redhat.com 05 Aug '12

05 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: document Correlation-Id header in RSDL ...................................................................... restapi: document Correlation-Id header in RSDL Change-Id: Id1c643939c0cbb68e40bb9c0691f64047bf5045f Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml 1 file changed, 179 insertions(+), 40 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6884 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id1c643939c0cbb68e40bb9c0691f64047bf5045f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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[master]: engine: Don't log MTU difference (#838300)
by mkolesni＠redhat.com 05 Aug '12

05 Aug '12

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Don't log MTU difference (#838300) ...................................................................... engine: Don't log MTU difference (#838300) https://bugzilla.redhat.com/838300 Since the unsynced networks are now being logged to the user, and the unsync status is wider than just MTU, there is no reason to log that MTU is unsynced separately. Change-Id: I4ca209a93ab90b5c7d9d2731a09aac9b9631e18b Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java D backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/CustomAuditLogKeys.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfoTest.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java 7 files changed, 0 insertions(+), 70 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6854 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ca209a93ab90b5c7d9d2731a09aac9b9631e18b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Livnat Peer <lpeer(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]: engine: Add audit log of unsync networks (#838300)
by mkolesni＠redhat.com 05 Aug '12

05 Aug '12

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Add audit log of unsync networks (#838300) ...................................................................... engine: Add audit log of unsync networks (#838300) https://bugzilla.redhat.com/838300 When getVdsCaps is executed, log all the networks that are defined on the host's NICs and are not in sync with their logical definitions. Change-Id: Ia2ffdff335719e154924319aa1f337f17708ea6d Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CollectVdsNetworkDataVDSCommand.java 1 file changed, 26 insertions(+), 0 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6853 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2ffdff335719e154924319aa1f337f17708ea6d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Livnat Peer <lpeer(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]: engine: Add audit log message for unsync networks (#838300)
by mkolesni＠redhat.com 05 Aug '12

05 Aug '12

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Add audit log message for unsync networks (#838300) ...................................................................... engine: Add audit log message for unsync networks (#838300) https://bugzilla.redhat.com/838300 Added message that will be presented when networks of a host are not in sync. Change-Id: I0873e0e06b827186aaf70b624aed14a427c415f8 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.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 3 files changed, 3 insertions(+), 0 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6852 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0873e0e06b827186aaf70b624aed14a427c415f8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Livnat Peer <lpeer(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]: restapi: Add sync networks to Setup Networks call (#838300)
by mkolesni＠redhat.com 05 Aug '12

05 Aug '12

Mike Kolesnik has submitted this change and it was merged. Change subject: restapi: Add sync networks to Setup Networks call (#838300) ...................................................................... restapi: Add sync networks to Setup Networks call (#838300) https://bugzilla.redhat.com/838300 Added the sync network functionality to the BackendHostNicResource call to SetupNetworks. If a HostNic has the field "override configuration" set and there's a network attached to the NIC, then this network will be set to be synchronized to the logical network definitions, as part of the SetupNetworks command. Change-Id: If0a66d50945b612bdc4d55ae0f519084397b34d8 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResourceTest.java 4 files changed, 58 insertions(+), 1 deletion(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6654 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0a66d50945b612bdc4d55ae0f519084397b34d8 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Add "custom configuration" to NIC (#838300)
by mkolesni＠redhat.com 05 Aug '12

05 Aug '12

Mike Kolesnik has submitted this change and it was merged. Change subject: restapi: Add "custom configuration" to NIC (#838300) ...................................................................... restapi: Add "custom configuration" to NIC (#838300) https://bugzilla.redhat.com/838300 This field will indicate if a NIC is out-of-sync, i.e. has some custom configuration that is not aligned to the logical network configuration. Change-Id: I6d6a678a0f1b793f4a218ec7eaf1ac2a2f815796 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostNicMapperTest.java 3 files changed, 20 insertions(+), 0 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6653 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d6a678a0f1b793f4a218ec7eaf1ac2a2f815796 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Added support for WANDisableEffects and WANColorDepth
by oourfali＠redhat.com 05 Aug '12

05 Aug '12

Oved Ourfali has submitted this change and it was merged. Change subject: tools: Added support for WANDisableEffects and WANColorDepth ...................................................................... tools: Added support for WANDisableEffects and WANColorDepth It has already been in vdc_options table, this patch enables the editing of this config values using the EngineConfig tool. For detailed description of the meaning of this values please consult this wiki: http://wiki.ovirt.org/wiki/Features/SPICERelatedFeatures Change-Id: Id796ae06e1d5d5a9aa6317741f02aad7b9ebb0b5 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/tools/engine-config/src/main/resources/engine-config.properties 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6873 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id796ae06e1d5d5a9aa6317741f02aad7b9ebb0b5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>

1 0

Change in ovirt-engine[master]: cleanup: remove unused IPWorksInstallWrapper
by dfediuck＠redhat.com 04 Aug '12

04 Aug '12

Doron Fediuck has submitted this change and it was merged. Change subject: cleanup: remove unused IPWorksInstallWrapper ...................................................................... cleanup: remove unused IPWorksInstallWrapper We use apache-sshd as SSH provider. Change-Id: I0be81d9ac84bc34acb6a79a53c7e5b4770fde845 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/IPWorksInstallWrapper.java D backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/hostinstall/IPWorksInstallWrapperTest.java 2 files changed, 0 insertions(+), 287 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6724 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0be81d9ac84bc34acb6a79a53c7e5b4770fde845 Gerrit-PatchSet: 5 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: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0