April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: pki: comply with rfc2459 UTCTime restriction
by piotr.kliczewski＠gmail.com 24 Apr '15

24 Apr '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: pki: comply with rfc2459 UTCTime restriction ...................................................................... pki: comply with rfc2459 UTCTime restriction Change-Id: If9f99ed24280f57c682e447781463db443c430c6 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1210486 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/bin/pki-enroll-request.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40163 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9f99ed24280f57c682e447781463db443c430c6 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(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]: webadmin : Fix Volumes tab space used column
by piotr.kliczewski＠gmail.com 24 Apr '15

24 Apr '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin : Fix Volumes tab space used column ...................................................................... webadmin : Fix Volumes tab space used column Fix Volumes tab space used column Change-Id: I48d0961cf0bf3d697069ac7902a3b39ee07e1335 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1209745 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVolumeView.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39637 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I48d0961cf0bf3d697069ac7902a3b39ee07e1335 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine, webadmin: Show RDMA port if present
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine,webadmin: Show RDMA port if present ...................................................................... engine,webadmin: Show RDMA port if present If volume supports RDMA transport, then the details view needs to support displaying the RDMA port values. Change-Id: I4cfc22a9bc6461b31a50ffe1c1350bf827802570 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/BrickProperties.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterServerService.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeStatusReturnForXmlRpc.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/BrickPropertiesModel.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/BrickAdvancedDetailsPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/BrickAdvancedDetailsPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterServiceView.java 8 files changed, 78 insertions(+), 9 deletions(-) Approvals: Kanagaraj M: Looks good to me, but someone else must approve Sahina Bose: Verified; Looks good to me, approved Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39570 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4cfc22a9bc6461b31a50ffe1c1350bf827802570 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: gluster brick network - delete action for constraint
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine: gluster brick network - delete action for constraint ...................................................................... engine: gluster brick network - delete action for constraint Added action for delete of network constraint of gluster_volume_bricks Change-Id: I30ad509154781f20dfa157add9ba1030cf527e36 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_1280_fix_constraint_gluster_brick_network.sql 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Sahina Bose: Verified -- To view, visit https://gerrit.ovirt.org/40126 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30ad509154781f20dfa157add9ba1030cf527e36 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(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-gluster]: gluster : add gluster prefix to gluster storage device configs
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster : add gluster prefix to gluster storage device configs ...................................................................... gluster : add gluster prefix to gluster storage device configs Add gluster prefix to all config variables for storage device. Change-Id: I96e261ad48f404add0b700c35139df8020cb7349 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetUnusedGlusterBricksQuery.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostGlusterStorageDevicesListModel.java 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40200 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96e261ad48f404add0b700c35139df8020cb7349 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: add is_gluster_brick flag to storage device
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: add is_gluster_brick flag to storage device ...................................................................... gluster: add is_gluster_brick flag to storage device Add is_gluster_brick column to storage device table. This flag helps to identify a mounted device as a gluster brick. This flag will be set to true for the bricks created from ovirt. Change-Id: I375a7340eb76db9cf2bbf142923929bf9ede748a Bug-Url: https://bugzilla.redhat.com/1208089 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateBrickCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetUnusedGlusterBricksQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/StorageDevice.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/StorageDeviceDaoDbFacadeImpl.java M packaging/dbscripts/storage_device_sp.sql A packaging/dbscripts/upgrade/03_05_1430_add_is_brick_flag-to_storage_device_table.sql 6 files changed, 29 insertions(+), 11 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40199 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I375a7340eb76db9cf2bbf142923929bf9ede748a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: webadmin: Display unit for stripe size in create brick dialog.
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: webadmin: Display unit for stripe size in create brick dialog. ...................................................................... webadmin: Display unit for stripe size in create brick dialog. Add the unit (KB) to the strip size label in create brick dialog. Change-Id: If735d639e574bd81fba984484c4624fd8c8cfd3b Bug-Url: https://bugzilla.redhat.com/1209426 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40198 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If735d639e574bd81fba984484c4624fd8c8cfd3b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: webadmin: add directory to brick mount point
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: webadmin: add directory to brick mount point ...................................................................... webadmin: add directory to brick mount point Add a directory with name of the brick under brick mount point in Add Brick Pop up. This required becase gluster requires a directory under the mount point not the mount point itself as a brick directory. Change-Id: I44d0fe15988b8193f0866441097b178a98e88b93 Bug-Url: https://bugzilla.redhat.com/1209443 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40197 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44d0fe15988b8193f0866441097b178a98e88b93 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: webadmin: fix action availability for create brick
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: webadmin: fix action availability for create brick ...................................................................... webadmin: fix action availability for create brick Enable/Disable create brick action based on the selection of storage devices. CreateBrick will be enabled only when there is a free device(Device with canCreateBrick flag true) selected and there is no used device (Device with canCreateBrick false) selected. Only selected devices will be shown in the create brick dialog instead of all available devices. Change-Id: Iecaff8f5863bd3107ba109e6f1f9d695d8ee67aa Bug-Url: https://bugzilla.redhat.com/1208125 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostGlusterStorageDevicesListModel.java 1 file changed, 39 insertions(+), 27 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40196 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iecaff8f5863bd3107ba109e6f1f9d695d8ee67aa Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: fix NPE in gluster sync job for volume capacity info
by sabose＠redhat.com 24 Apr '15

24 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: fix NPE in gluster sync job for volume capacity info ...................................................................... gluster: fix NPE in gluster sync job for volume capacity info Fix NPE in GlusterSyncJob.refreshVolumeDetails. When all bricks are down in a volume, volume capacity information can't be retrived but volume status will work. So we can't update the capacity info for volume but brick status can be updated. Change-Id: Iab4c5cdc38c53aa7b35417ad3db10b1cb4ac0f2a Bug-Url: https://bugzilla.redhat.com/1212290 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/40195 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab4c5cdc38c53aa7b35417ad3db10b1cb4ac0f2a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0