Ramesh N has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 4: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/65008
To unsubscribe, visit https://gerrit.ovirt.org/settin
Ramesh N has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 3:
@Dan: I manually built and installed the vdsm packages with this patch. I
didn't see any issue with the gluster mana
Ramesh N has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/65008
To unsubscribe, visit https://gerrit.ovirt.org/settings
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/64296
To unsubscribe, visit https://gerrit.ov
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 6:
PPC build is failing because of package installation issue. It fails to install
the package tzdata.
Hello Piotr Kliczewski, Yaniv Bronhaim, Dan Kenigsberg, Francesco Romani,
Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/64296
to review the following change.
Change subject: gluster: set selinux labels while creating bricks
..
Ramesh N has restored this change.
Change subject: gluster: add support for arbiter volumes
..
Restored
--
To view, visit https://gerrit.ovirt.org/52559
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageTy
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/62773/5/vdsm/gluster/storagedev.py
File vdsm/gluster/stora
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 5:
Gluster has an selinux policy which expects the brick dirs to have specific
selinux context/label s
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/62773
To unsubscribe, visit https://gerrit.ov
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/62773/4/lib/vdsm/gluster/exception.py
File lib/vdsm/gluste
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/62773/4/vdsm/gluster/storagedev.py
File vdsm/gluster/stora
Ramesh N has posted comments on this change.
Change subject: gluster: fix help test for glusterVolumehealInfo in vdsClient
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/63480
To unsubscribe, visit http
Ramesh N has uploaded a new change for review.
Change subject: gluster: fix help test for glusterVolumehealInfo in vdsClient
..
gluster: fix help test for glusterVolumehealInfo in vdsClient
There is a format issue with the hel
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 4:
(4 comments)
https://gerrit.ovirt.org/#/c/62773/4/lib/vdsm/gluster/exception.py
File lib/vdsm/glust
Ramesh N has abandoned this change.
Change subject: gluster: fix python-blivet dependency issue for python3
..
Abandoned
--
To view, visit https://gerrit.ovirt.org/56945
To unsubscribe, visit https://gerrit.ovirt.org/settings
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/62773/2/vdsm/gluster/storagedev.py
File vdsm/gluster/stora
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/62773/2/vdsm/gluster/storagedev.py
File vdsm/gluster/stora
Ramesh N has posted comments on this change.
Change subject: gluster: set selinux labels while creating bricks
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/62773
To unsubscribe, visit https://gerrit.ov
Ramesh N has uploaded a new change for review.
Change subject: gluster: set selinux labels while creating bricks
..
gluster: set selinux labels while creating bricks
brick should have correct selinux labels on the brick mount
po
Ramesh N has posted comments on this change.
Change subject: gluster: introduce vdsm.gluster module
..
Patch Set 7: Verified+1
--
To view, visit https://gerrit.ovirt.org/61019
To unsubscribe, visit https://gerrit.ovirt.org/set
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 11: Verified+1
--
To view, visit https://gerrit.ovirt.org/59600
To unsubscribe, visit https://gerrit.ovirt.org/
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 10:
(7 comments)
https://gerrit.ovirt.org/#/c/59600/10//COMMIT_MSG
Commit Message:
Line 18: brick is brought d
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/59600/9/vdsm.spec.in
File vdsm.spec.in:
Line 1648: %license COPYIN
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 9:
(4 comments)
https://gerrit.ovirt.org/#/c/59600/9/lib/vdsm/gluster/fence.py
File lib/vdsm/gluster/fence.py:
Ramesh N has posted comments on this change.
Change subject: gluster: introduce vdsm.gluster module
..
Patch Set 4:
(7 comments)
https://gerrit.ovirt.org/#/c/61019/4/tests/storageServerTests.py
File tests/storageServerTests.py
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 9: Verified+1
--
To view, visit https://gerrit.ovirt.org/59600
To unsubscribe, visit https://gerrit.ovirt.org/s
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 8:
(5 comments)
https://gerrit.ovirt.org/#/c/59600/8/lib/api/vdsm-api.yml
File lib/api/vdsm-api.yml:
PS8, Line
Ramesh N has posted comments on this change.
Change subject: gluster: introduce vdsm.gluster module
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/61019
To unsubscribe, visit https://gerrit.ovirt.org/set
Ramesh N has posted comments on this change.
Change subject: gluster: introduce vdsm.gluster module
..
Patch Set 1:
(2 comments)
>>1. please arrange imports in sort order and move vdsm. imports to vdsm.
>>section.
Done
>>2.
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 6: Verified+1
--
To view, visit https://gerrit.ovirt.org/59600
To unsubscribe, visit https://gerrit.ovirt.org/s
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 5:
(2 comments)
Sent a new patch-set to fix the comments.
https://gerrit.ovirt.org/#/c/59600/5/lib/api/vdsm-ap
Ramesh N has posted comments on this change.
Change subject: Move gluster.exception to lib/vdsm/gluster
..
Patch Set 1: -Code-Review Verified+1
--
To view, visit https://gerrit.ovirt.org/61019
To unsubscribe, visit https://ger
Ramesh N has posted comments on this change.
Change subject: Move gluster.exception to lib/vdsm/gluster
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/61019
To unsubscribe, visit https://gerrit.ovirt.
Ramesh N has uploaded a new change for review.
Change subject: Move gluster.exception to lib/vdsm/gluster
..
Move gluster.exception to lib/vdsm/gluster
Change-Id: Ib97d2728c97d295b83ed452e116e9422184de10f
Signed-off-by: Ramesh N
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 4:
Thanks Martin for your suggestions. I will post a new patch -set with the
suggested changes.
--
To view, v
Ramesh N has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 1:
@Dan Kenigsberg, Can you merge this patch ?
--
To view, visit https://gerrit.ovirt.org/60379
T
Ramesh N has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/60379
To unsubscribe, visit https://gerri
Ramesh N has uploaded a new change for review.
Change subject: api: fixing default value for GlusterHost.createBrick
..
api: fixing default value for GlusterHost.createBrick
Change default value for raidParamas to an empty dict
Ramesh N has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/60206
To unsubscribe, visit https://gerri
Ramesh N has uploaded a new change for review.
Change subject: api: fixing default value for GlusterHost.createBrick
..
api: fixing default value for GlusterHost.createBrick
Change default value for raidParamas to an empty dict
Ramesh N has abandoned this change.
Change subject: gluster: remove clientIf frmo GlusterApi
..
Abandoned
--
To view, visit https://gerrit.ovirt.org/59929
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-Message
Ramesh N has posted comments on this change.
Change subject: gluster: remove clientIf frmo GlusterApi
..
Patch Set 1:
clientIF was used in bindingxmlrpc.py and I don't think it can be removed.
Abandoning this patch.
--
To vi
Ramesh N has posted comments on this change.
Change subject: gluster: remove clientIf frmo GlusterApi
..
Patch Set 1: Verified-1
I feel still clientIF is used some where in vdsm server. I am not able to
execute any gluster ver
Ramesh N has uploaded a new change for review.
Change subject: gluster: remove clientIf frmo GlusterApi
..
gluster: remove clientIf frmo GlusterApi
Currently clientIf is initialized in GlusterApi but it not used
any where. So re
Ramesh N has posted comments on this change.
Change subject: Move gluster package to vdsm.gluster
..
Patch Set 8: Verified+1
--
To view, visit https://gerrit.ovirt.org/57530
To unsubscribe, visit https://gerrit.ovirt.org/setti
Ramesh N has posted comments on this change.
Change subject: gluster:enforce gluster fencing policies
..
Patch Set 2:
(6 comments)
https://gerrit.ovirt.org/#/c/59600/2/vdsm/API.py
File vdsm/API.py:
Line 54: from vdsm.define i
Ramesh N has uploaded a new change for review.
Change subject: gluster:enforce gluster fencing policies
..
gluster:enforce gluster fencing policies
Enforce gluster related fencing policies before fencing
the host.
1. skipFecin
Ramesh N has posted comments on this change.
Change subject: Move gluster package to vdsm.gluster
..
Patch Set 2:
(1 comment)
Overall changes looks to correct. Let me verify once.
https://gerrit.ovirt.org/#/c/57530/2/vdsm.spe
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 12: Verified+1
--
To view, visit https://gerrit.ovirt.org/56738
To unsubscribe, visit https://gerrit.ovirt.org/setti
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 11: Verified+1
--
To view, visit https://gerrit.ovirt.org/56738
To unsubscribe, visit https://gerrit.ovirt.org/setti
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 10:
(2 comments)
https://gerrit.ovirt.org/#/c/56738/10/lib/api/vdsmapi-gluster-schema.json
File lib/api/vdsmapi-glus
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/56738/9/tests/glusterVolumeHealInfo.xml
File tests/glusterVolumeHealInfo
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 9: Verified+1
--
To view, visit https://gerrit.ovirt.org/56738
To unsubscribe, visit https://gerrit.ovirt.org/settin
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 8: -Code-Review Verified+1
--
To view, visit https://gerrit.ovirt.org/56738
To unsubscribe, visit https://gerrit.ovi
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/56738
To unsubscribe, visit https://gerrit.ovirt.org/set
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 7:
(4 comments)
Patch-set to follow.
https://gerrit.ovirt.org/#/c/56738/7/lib/api/vdsmapi-gluster-schema.json
File
Ramesh N has posted comments on this change.
Change subject: gluster: add gluster heal info verb
..
Patch Set 5:
(3 comments)
https://gerrit.ovirt.org/#/c/56738/5/client/vdsClientGluster.py
File client/vdsClientGluster.py:
PS
Ramesh N has uploaded a new change for review.
Change subject: gluster: fix for lvchange regression in rhel 6.8
..
gluster: fix for lvchange regression in rhel 6.8
There is a regression in lvchange command. Changes are not
app
Ramesh N has uploaded a new change for review.
Change subject: gluster: fix python-blivet dependency issue for python3
..
gluster: fix python-blivet dependency issue for python3
Until now blivet API had both 'python-blivet' and
Ramesh N has uploaded a new change for review.
Change subject: gluster: add gluster heal info verb
..
gluster: add gluster heal info verb
Add gluster heal info to get list of unsynced entries
present in the given gluster volum
Ramesh N has uploaded a new change for review.
Change subject: gluster: add support for arbiter volumes
..
gluster: add support for arbiter volumes
Add support for arbiter gluster volume creations. Also
include arbiter count
Ramesh N has posted comments on this change.
Change subject: gluster: create bricks without MBR partitions
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50850
To unsubscribe, visit https://gerrit.ovirt.
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50853
To unsubscribe, visit https://gerrit.
Ramesh N has posted comments on this change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50854
To unsubscribe, visit https://gerrit
Ramesh N has posted comments on this change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50851
To unsubscribe, visit https://ger
Ramesh N has posted comments on this change.
Change subject: gluster add mount options for gluster brick.
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50852
To unsubscribe, visit https://gerrit.ovirt.o
Ramesh N has posted comments on this change.
Change subject: gluster: fix xfs filesystem is created with wrong inode size
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/50849
To unsubscribe, visit https:
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50853
to review the following change.
Change subject: gluster: fix size conversion issues in brick create
..
gluster:
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50852
to review the following change.
Change subject: gluster add mount options for gluster brick.
..
gluster add moun
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50851
to review the following change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
glust
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50854
to review the following change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
gluster:
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50850
to review the following change.
Change subject: gluster: create bricks without MBR partitions
..
gluster: create
Hello Manoj Pillai, Timothy Asir, Bala.FA, Dan Kenigsberg,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50849
to review the following change.
Change subject: gluster: fix xfs filesystem is created with wrong inode size
Ramesh N has posted comments on this change.
Change subject: gluster: create bricks without MBR partitions
..
Patch Set 1:
This is a special branch for gluster :-). But Yes. I have to back port the
patches to ovirt 3.6 as wel
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50808
To unsubscribe, visit https://gerrit.
Ramesh N has posted comments on this change.
Change subject: gluster add mount options for gluster brick.
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50807
To unsubscribe, visit https://gerrit.ovirt.o
Ramesh N has posted comments on this change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50809
To unsubscribe, visit https://gerrit
Ramesh N has posted comments on this change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50806
To unsubscribe, visit https://ger
Ramesh N has posted comments on this change.
Change subject: gluster: create bricks without MBR partitions
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50805
To unsubscribe, visit https://gerrit.ovirt.
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50809
to review the following change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
gluster:
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50806
to review the following change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
glust
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50808
to review the following change.
Change subject: gluster: fix size conversion issues in brick create
..
gluster:
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50805
to review the following change.
Change subject: gluster: create bricks without MBR partitions
..
gluster: create
Hello Nir Soffer, Sahina Bose,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/50807
to review the following change.
Change subject: gluster add mount options for gluster brick.
..
gluster add moun
Ramesh N has posted comments on this change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
Patch Set 3:
I have verified this patch in Fedora, RHEL 6 and RHEL 7 platforms.
Also i tried creating brick w
Ramesh N has posted comments on this change.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/50802
To unsubscribe, visit https://gerrit
Ramesh N has uploaded a new change for review.
Change subject: gluster: ensure LV size is multiple of VG's PE value
..
gluster: ensure LV size is multiple of VG's PE value
Size of LV should be multiple of PE value. But this ca
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 7: Verified+1
--
To view, visit https://gerrit.ovirt.org/50705
To unsubscribe, visit https://gerrit.
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 6:
(5 comments)
Thanks for your suggestions Nir. I will fix them in the next patch set.
https://ger
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 6: Verified+1
--
To view, visit https://gerrit.ovirt.org/50705
To unsubscribe, visit https://gerrit.
Ramesh N has posted comments on this change.
Change subject: gluster add mount options for gluster brick.
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50770
To unsubscribe, visit https://gerrit.ovirt.o
Ramesh N has posted comments on this change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
Patch Set 7:
Splitted the original patch into 3 patches https://gerrit.ovirt.org/50769
https://gerrit.ovir
Ramesh N has posted comments on this change.
Change subject: gluster: fix alignment issue for brick creation on JBOD
..
Patch Set 7: Verified+1
--
To view, visit https://gerrit.ovirt.org/47959
To unsubscribe, visit https://ger
Ramesh N has posted comments on this change.
Change subject: gluster: create bricks without MBR partitions
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/50769
To unsubscribe, visit https://gerrit.ovirt.
Ramesh N has uploaded a new change for review.
Change subject: gluster add mount options for gluster brick.
..
gluster add mount options for gluster brick.
Mount options "inode64,noatime" should be used to
mount the gluster bri
Ramesh N has posted comments on this change.
Change subject: gluster: fix size conversion issues in brick create
..
Patch Set 5:
(7 comments)
https://gerrit.ovirt.org/#/c/50705/5/vdsm/gluster/storagedev.py
File vdsm/gluster/st
Ramesh N has uploaded a new change for review.
Change subject: gluster: create bricks without MBR partitions
..
gluster: create bricks without MBR partitions
During gluster brick creation, PVs should be
created directly on the
Ramesh N has posted comments on this change.
Change subject: gluster: fix performance issue in brick creation
..
Patch Set 6:
(3 comments)
https://gerrit.ovirt.org/#/c/47959/6//COMMIT_MSG
Commit Message:
Line 12: 1. PVs shou
Ramesh N has posted comments on this change.
Change subject: gluster: fix performance issue in brick creation
..
Patch Set 3:
I have splited this patch into two patches new one at
https://gerrit.ovirt.org/50705. I have verifie
1 - 100 of 143 matches
Mail list logo