3:54 a.m.
Darshan N has uploaded a new change for review.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
gluster: modified parsing logic in snapshot config list.
Modified snapshot config parsing logic to return only
effective-hard-limit as snapmax-hard-limit in volume
config. As the remaining two values are not relevant.
Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Signed-off-by: Darshan N <dnarayan(a)redhat.com>
---
M tests/glusterTestData.py
M vdsm/gluster/cli.py
M vdsm/rpc/vdsmapi-gluster-schema.json
3 files changed, 5 insertions(+), 18 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/58/38758/1
diff --git a/tests/glusterTestData.py b/tests/glusterTestData.py
index 640eb07..ca6ed23 100644
--- a/tests/glusterTestData.py
+++ b/tests/glusterTestData.py
@@ -1132,14 +1132,10 @@
GLUSTER_SNAPSHOT_CONFIG_LIST = {
'volume': {
'snap-vol': {
- 'snap-max-hard-limit': '256',
- 'snap-max-soft-limit': '232',
- 'effective-hard-limit': '255'
+ 'snap-max-hard-limit': '255'
},
'snap-vol2': {
- 'snap-max-hard-limit': '254',
- 'snap-max-soft-limit': '231',
- 'effective-hard-limit': '254'
+ 'snap-max-hard-limit': '254'
}
},
'system': {
diff --git a/vdsm/gluster/cli.py b/vdsm/gluster/cli.py
index 9a3e594..37a18a8 100644
--- a/vdsm/gluster/cli.py
+++ b/vdsm/gluster/cli.py
@@ -1351,9 +1351,7 @@
'snap-max-soft-limit': 'softLimit',
'auto-delete': 'enable/disable'},
'volume':{'name' :
- {'snap-max-hard-limit: 'hardlimit',
- 'effective-hard-limit: 'EffectiveHardlimit',
- 'snap-max-soft-limit': 'softLimit'}
+ {'snap-max-hard-limit: 'hardlimit'}
}
}
"""
@@ -1369,9 +1367,7 @@
for el in tree.findall('snapConfig/volumeConfig/volume'):
config = {}
volumeName = el.find('name').text
- config['snap-max-hard-limit'] = el.find('hardLimit').text
- config['effective-hard-limit'] =
el.find('effectiveHardLimit').text
- config['snap-max-soft-limit'] = el.find('softLimit').text
+ config['snap-max-hard-limit'] =
el.find('effectiveHardLimit').text
volumeConfig[volumeName] = config
return {'system': systemConfig, 'volume': volumeConfig}
diff --git a/vdsm/rpc/vdsmapi-gluster-schema.json b/vdsm/rpc/vdsmapi-gluster-schema.json
index 2ecf8cc..c370578 100644
--- a/vdsm/rpc/vdsmapi-gluster-schema.json
+++ b/vdsm/rpc/vdsmapi-gluster-schema.json
@@ -1752,15 +1752,10 @@
#
# @snap-max-hard-limit: Snapshot max hard limit
#
-# @effective-hard-limit: Effective snapshot Hard limit for that volume
-#
-# @snap-max-soft-limit: Snapshot soft limit for that volume
-#
# @Since: 4.17.0
##
{'type': 'VolumeSnapConfig',
- 'data': {'snap-max-hard-limit': 'str',
'effective-hard-limit': 'str',
- 'snap-max-soft-limit': 'str'}}
+ 'data': {'snap-max-hard-limit': 'str'}}
##
#
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
3:54 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
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/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
3:55 a.m.
Darshan N has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
4:49 a.m.
Shubhendu Tripathi has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
4:55 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 1:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16704/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16876/ : SUCCESS
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
3:26 p.m.
Piotr Kliczewski has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 1: Verified+1
Schema change looks good.
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:11 a.m.
Dan Kenigsberg has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:11 a.m.
Dan Kenigsberg has submitted this change and it was merged.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
gluster: modified parsing logic in snapshot config list.
Modified snapshot config parsing logic to return only
effective-hard-limit as snapmax-hard-limit in volume
config. As the remaining two values are not relevant.
Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Signed-off-by: Darshan N <dnarayan(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38758
Reviewed-by: Shubhendu Tripathi <shtripat(a)redhat.com>
Tested-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M tests/glusterTestData.py
M vdsm/gluster/cli.py
M vdsm/rpc/vdsmapi-gluster-schema.json
3 files changed, 5 insertions(+), 18 deletions(-)
Approvals:
Piotr Kliczewski: Verified
Shubhendu Tripathi: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Darshan N: Verified
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9:11 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:11 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (1/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/804/
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:13 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (2/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64...
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:22 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (3/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/776/
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:26 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (4/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el6-x86_64_...
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:27 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (5/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/809/
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:28 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (6/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_...
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:37 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (7/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64...
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:47 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (8/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/810/
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
10:17 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Started (9/9) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4785/
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
10:20 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: modified parsing logic in snapshot config list.
......................................................................
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/804/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4785/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/776/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el6-x86_64_... :
FAILURE
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/809/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/810/ : SUCCESS
--
To view, visit https://gerrit.ovirt.org/38758
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3c56c7448f6316835978577e4514568ddeb4a5ea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
3323
days inactive
3333
days old
vdsm-patches@lists.fedorahosted.org
18 comments
6 participants
participants (6)
-
automation＠ovirt.org -
Dan Kenigsberg -
dnarayan＠redhat.com -
Jenkins CI RO -
Piotr Kliczewski -
shtripat＠redhat.com