Change in vdsm[master]: gluster: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 8: * 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 6: * 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Piotr Kliczewski has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 8: Code-Review+1 api/schema changes looks good -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
Piotr Kliczewski has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: Code-Review-1 (2 comments) http://gerrit.ovirt.org/#/c/31069/7/vdsm/rpc/vdsmapi-gluster-schema.json File vdsm/rpc/vdsmapi-gluster-schema.json: Line 1466: # @remoteHost: Is remote host name or ip Line 1467: # Line 1468: # @remoteVolumeName: Is an available existing volume name in the remote node Line 1469: # Line 1470: # @force: For pausing a geo-replication session forcefully Please mark as optional. In apiwrapper force flag has default value. Line 1471: # Line 1472: # Returns: Line 1473: # True if session is successfully paused Line 1474: # Line 1489: # @remoteHost: Is remote host name or ip Line 1490: # Line 1491: # @remoteVolumeName: Is an available existing volume name in the remote node Line 1492: # Line 1493: # @force: For resuming a geo-replication session forcefully As above. Line 1494: # Line 1495: # Returns: Line 1496: # True if session is successfully resumed Line 1497: # -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: * 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. 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 http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
automat...@ovirt.org has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 9: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: 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: geo-replication pause and resume session
Dan Kenigsberg has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 8: Code-Review+2 Raising Baka+Piotr score -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: 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: geo-replication pause and resume session
Dan Kenigsberg has submitted this change and it was merged. Change subject: gluster: geo-replication pause and resume session .. gluster: geo-replication pause and resume session This patch adds two new verbs to pause and resume geo-replication session between master and slave volume. Verbs: glusterVolumeGeoRepSessionPause This verb Pauses a geo-replication session between a local and a remote volume. It retruns boolean output. glusterVolumeGeoRepSessionResume This verb resumes a geo-replication session between a local and a remote volume. It retruns boolean output. Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Signed-off-by: Darshan N dnara...@redhat.com Reviewed-on: http://gerrit.ovirt.org/31069 Reviewed-by: Piotr Kliczewski piotr.kliczew...@gmail.com Reviewed-by: Dan Kenigsberg dan...@redhat.com --- M client/vdsClientGluster.py M vdsm/gluster/api.py M vdsm/gluster/apiwrapper.py M vdsm/gluster/cli.py M vdsm/gluster/exception.py M vdsm/rpc/vdsmapi-gluster-schema.json 6 files changed, 170 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Darshan N: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: 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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: (2 comments) http://gerrit.ovirt.org/#/c/31069/7/vdsm/rpc/vdsmapi-gluster-schema.json File vdsm/rpc/vdsmapi-gluster-schema.json: Line 1466: # @remoteHost: Is remote host name or ip Line 1467: # Line 1468: # @remoteVolumeName: Is an available existing volume name in the remote node Line 1469: # Line 1470: # @force: For pausing a geo-replication session forcefully Please mark as optional. In apiwrapper force flag has default value. Done Line 1471: # Line 1472: # Returns: Line 1473: # True if session is successfully paused Line 1474: # Line 1489: # @remoteHost: Is remote host name or ip Line 1490: # Line 1491: # @remoteVolumeName: Is an available existing volume name in the remote node Line 1492: # Line 1493: # @force: For resuming a geo-replication session forcefully As above. Done Line 1494: # Line 1495: # Returns: Line 1496: # True if session is successfully resumed Line 1497: # -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 8: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15466/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15297/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14509/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Bala.FA has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 7: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15454/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15285/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14497/ : FAILURE -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@redhat.com Gerrit-Reviewer: Darshan N dnara...@redhat.com Gerrit-Reviewer: Piotr Kliczewski piotr.kliczew...@gmail.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: geo-replication pause and resume session
Bala.FA has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 5: Code-Review-1 (2 comments) http://gerrit.ovirt.org/#/c/31069/5/vdsm/gluster/exception.py File vdsm/gluster/exception.py: Line 528: code = 4203 Line 529: message = Geo Rep status failed Line 530: Line 531: Line 532: class GlusterVolumeGeoRepSessionPauseFailedException(GlusterVolumeException): The parent class should be GlusterGeoRepException Line 533: code = 4204 Line 534: message = Volume geo-replication session pause failed Line 535: Line 536: Line 533: code = 4204 Line 534: message = Volume geo-replication session pause failed Line 535: Line 536: Line 537: class GlusterVolumeGeoRepSessionResumeFailedException(GlusterVolumeException): Same here Line 538: code = 4205 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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
Change in vdsm[master]: gluster: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 6: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15448/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15279/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14491/ : FAILURE -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15443/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15274/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14486/ : FAILURE -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 5: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15445/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15276/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14488/ : FAILURE -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15324/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/15155/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/14367/ : FAILURE -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/10746/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/11688/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/11531/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/10658/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/11600/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/11443/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@redhat.com Gerrit-Reviewer: Bala.FA barum...@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: geo-replication pause and resume session
Darshan N has posted comments on this change. Change subject: gluster: geo-replication pause and resume session .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/31069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Darshan N dnara...@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: geo-replication pause and resume session
Darshan N has uploaded a new change for review. Change subject: gluster: geo-replication pause and resume session .. gluster: geo-replication pause and resume session This patch adds two new verbs to pause and resume geo-replication session between master and slave volume. It returns a boolean output. Change-Id: I024bcee148bab1e713e1bc5c73d288613c466656 Signed-off-by: ndarshan dnara...@redhat.com --- M client/vdsClientGluster.py M vdsm/gluster/api.py M vdsm/gluster/cli.py M vdsm/gluster/exception.py M vdsm/gluster/vdsmapi-gluster-schema.json 5 files changed, 157 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/69/31069/1 diff --git a/client/vdsClientGluster.py b/client/vdsClientGluster.py index 421c853..bd7e107 100644 --- a/client/vdsClientGluster.py +++ b/client/vdsClientGluster.py @@ -430,6 +430,34 @@ pp.pprint(status) return status['status']['code'], status['status']['message'] +def do_glusterVolumeGeoRepPause(self, args): +params = self._eqSplit(args) +volumeName = params.get('volumeName', '') +remoteHost = params.get('remoteHost', '') +remoteVolumeName = params.get('remoteVolumeName', '') +force = (params.get('force', 'no').upper() == 'YES') + +status = self.s.glusterVolumeGeoRepPause(volumeName, + remoteHost, + remoteVolumeName, + force) +pp.pprint(status) +return status['status']['code'], status['status']['message'] + +def do_glusterVolumeGeoRepResume(self, args): +params = self._eqSplit(args) +volumeName = params.get('volumeName', '') +remoteHost = params.get('remoteHost', '') +remoteVolumeName = params.get('remoteVolumeName', '') +force = (params.get('force', 'no').upper() == 'YES') + +status = self.s.glusterVolumeGeoRepResume(volumeName, + remoteHost, + remoteVolumeName, + force) +pp.pprint(status) +return status['status']['code'], status['status']['message'] + def getGlusterCmdDict(serv): return \ @@ -731,4 +759,26 @@ ('volumeName=volume name', 'Returns total, free and used space(bytes) of gluster volume' )), + 'glusterVolumeGeoRepPause': ( + serv.do_glusterVolumeGeoRepPause, + ('volumeName=master_volume_name ' + 'remoteHost=slave_host_name ' + 'remoteVolumeName=slave_volume_name ' + '[force={yes|no}]\n\t' + 'master_volume_nameexisting volume name in the master node\n\t' + 'slave_host_nameis remote slave host name or ip\n\t' + 'slave_volume_nameexisting volume name in the slave node', + 'Pause the geo-replication session' + )), + 'glusterVolumeGeoRepResume': ( + serv.do_glusterVolumeGeoRepResume, + ('volumeName=master_volume_name ' + 'remoteHost=slave_host_name ' + 'remoteVolumeName=slave_volume_name ' + '[force={yes|no}]\n\t' + 'master_volume_nameexisting volume name in the master node\n\t' + 'slave_host_nameis remote slave host name or ip\n\t' + 'slave_volume_nameexisting volume name in the slave node', + 'Resume the geo-replication session' + )), } diff --git a/vdsm/gluster/api.py b/vdsm/gluster/api.py index 0e6c850..abe8ba0 100644 --- a/vdsm/gluster/api.py +++ b/vdsm/gluster/api.py @@ -322,6 +322,22 @@ data = self.svdsmProxy.glusterVolumeStatvfs(volumeName) return self._computeVolumeStats(data) +@exportAsVerb +def volumeGeoRepPause(self, volumeName, remoteHost, remoteVolumeName, + force=False, options=None): +self.svdsmProxy.glusterVolumeGeoRepPause(volumeName, + remoteHost, + remoteVolumeName, + force) + +@exportAsVerb +def volumeGeoRepResume(self, volumeName, remoteHost, remoteVolumeName, + force=False, options=None): +self.svdsmProxy.glusterVolumeGeoRepResume(volumeName, + remoteHost, + remoteVolumeName, + force) + def getGlusterMethods(gluster): l = [] diff --git a/vdsm/gluster/cli.py b/vdsm/gluster/cli.py index 2e1c9a9..7993b18 100644 --- a/vdsm/gluster/cli.py +++ b/vdsm/gluster/cli.py @@ -1054,3 +1054,33 @@