Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 6:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/751/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64_merged/347/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4732/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/724/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el6-x86_64_merged/352/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/753/ : 
SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_merged/351/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64_merged/341/
 : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/757/ : 
SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-18 Thread shtripat
Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 5: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-18 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 6:

* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-18 Thread danken
Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


gluster: Fetch disperse count and redundacy count for volume

Modified the verb glusterVolumeInfo to fetch disperse count
and redundancy count also.

Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Signed-off-by: Shubhendu Tripathi shtri...@redhat.com
Reviewed-on: https://gerrit.ovirt.org/38305
Reviewed-by: Piotr Kliczewski piotr.kliczew...@gmail.com
Reviewed-by: Dan Kenigsberg dan...@redhat.com
---
M tests/gluster_cli_tests.py
M vdsm/gluster/cli.py
M vdsm/rpc/vdsmapi-gluster-schema.json
3 files changed, 27 insertions(+), 10 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Shubhendu Tripathi: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-18 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 5: Code-Review+2

Raising Timothy's and Pioter's score.

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 4: Code-Review-1

(1 comment)

https://gerrit.ovirt.org/#/c/38305/4/vdsm/rpc/vdsmapi-gluster-schema.json
File vdsm/rpc/vdsmapi-gluster-schema.json:

Line 573: # @volumeType:Type of the volume
Line 574: #
Line 575: # @disperseCount: Disperse count of the volume
Line 576: #
Line 577: # @redundancyCount: Redundancy count of the volume
Please align parameter descriptions in line.
Line 578: #
Line 579: # @bricksInfo:details related to bricks(new in version 4.14.0)
Line 580: #
Line 581: # Since: 4.10.3


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 5: Code-Review+1

Schema change looks good

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 5:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 2:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 1:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-15 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 4:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread tjeyasin
Timothy Asir has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 4: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 4:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16633/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16804/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread shtripat
Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/38305/3/vdsm/rpc/vdsmapi-gluster-schema.json
File vdsm/rpc/vdsmapi-gluster-schema.json:

Line 579: {'type': 'VolumeInfo',
Line 580:  'data': {'brickCount': 'int', 'bricks': ['str'], 'options': ['str'],
Line 581:   'transportType': 'transType', 'uuid': 'UUID', 'volumeName': 
'str',
Line 582:   'status': 'volumeStatus', 'volumeType': 'str',
Line 583:   'disperseCount': 'int', 'redundancyCount': 'int',
 please update the parms description
done
Line 584:   'bricksInfo': ['BricksInfo']}}
Line 585: 
Line 586: ##
Line 587: # @StatusOption:


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread tjeyasin
Timothy Asir has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3: Verified-1

(1 comment)

https://gerrit.ovirt.org/#/c/38305/3/vdsm/rpc/vdsmapi-gluster-schema.json
File vdsm/rpc/vdsmapi-gluster-schema.json:

Line 579: {'type': 'VolumeInfo',
Line 580:  'data': {'brickCount': 'int', 'bricks': ['str'], 'options': ['str'],
Line 581:   'transportType': 'transType', 'uuid': 'UUID', 'volumeName': 
'str',
Line 582:   'status': 'volumeStatus', 'volumeType': 'str',
Line 583:   'disperseCount': 'int', 'redundancyCount': 'int',
please update the parms description
Line 584:   'bricksInfo': ['BricksInfo']}}
Line 585: 
Line 586: ##
Line 587: # @StatusOption:


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread tjeyasin
Timothy Asir has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3: -Verified Code-Review-1

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread shtripat
Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/38305/4/vdsm/rpc/vdsmapi-gluster-schema.json
File vdsm/rpc/vdsmapi-gluster-schema.json:

Line 573: # @volumeType:Type of the volume
Line 574: #
Line 575: # @disperseCount: Disperse count of the volume
Line 576: #
Line 577: # @redundancyCount: Redundancy count of the volume
 Please align parameter descriptions in line.
done
Line 578: #
Line 579: # @bricksInfo:details related to bricks(new in version 4.14.0)
Line 580: #
Line 581: # Since: 4.10.3


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-13 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 5:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16646/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16817/ : SUCCESS

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-11 Thread shtripat
Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 2:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16198/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15398/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16368/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/1174/ : 
FAILURE

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16204/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15404/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16374/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/1180/ : 
FAILURE

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-03 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 1:

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16183/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15383/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16353/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/1159/ : 
FAILURE

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-03 Thread shtripat
Shubhendu Tripathi has uploaded a new change for review.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..

gluster: Fetch disperse count and redundacy count for volume

Modified the verb glusterVolumeInfo to fetch disperse count
and redundancy count also.

Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Signed-off-by: Shubhendu Tripathi shtri...@redhat.com
---
M vdsm/gluster/cli.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/05/38305/1

diff --git a/vdsm/gluster/cli.py b/vdsm/gluster/cli.py
index e60da27..c578621 100644
--- a/vdsm/gluster/cli.py
+++ b/vdsm/gluster/cli.py
@@ -367,6 +367,8 @@
 value['distCount'] = el.find('distCount').text
 value['stripeCount'] = el.find('stripeCount').text
 value['replicaCount'] = el.find('replicaCount').text
+value['disperseCount'] = el.find('disperseCount').text
+value['redundancyCount'] = el.find('redundancyCount').text
 transportType = el.find('transport').text
 if transportType == '0':
 value['transportType'] = [TransportType.TCP]


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-03 Thread dnarayan
Darshan N has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 3: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-03 Thread shtripat
Shubhendu Tripathi has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/38305/2/vdsm/gluster/cli.py
File vdsm/gluster/cli.py:

Line 353: ...}
 can you please add the newly added fields here.
done


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: gluster: Fetch disperse count and redundacy count for volume

2015-03-03 Thread dnarayan
Darshan N has posted comments on this change.

Change subject: gluster: Fetch disperse count and redundacy count for volume
..


Patch Set 2:

(1 comment)

Can you please add these two entries to vdsmapi-gluster-schema.json

https://gerrit.ovirt.org/#/c/38305/2/vdsm/gluster/cli.py
File vdsm/gluster/cli.py:

Line 353: ...}
can you please add the newly added fields here.


-- 
To view, visit https://gerrit.ovirt.org/38305
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eb9190a7531751c4c62e3eff425c3526101b9d1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi shtri...@redhat.com
Gerrit-Reviewer: Bala.FA barum...@redhat.com
Gerrit-Reviewer: Darshan N dnara...@redhat.com
Gerrit-Reviewer: Timothy Asir tjeya...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches