Bug 1365007

Summary:	[RFE] dump_volume_chains: migrate to jsonrpcvdscli
Product:	[oVirt] vdsm	Reporter:	Sandro Bonazzola <sbonazzo>
Component:	Tools	Assignee:	Irit Goihman <igoihman>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Lukas Svaty <lsvaty>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	---	CC:	amureini, bugs, mperina, nsoffer, pkliczew, pstehlik, ybronhei
Target Milestone:	ovirt-4.1.0-alpha	Keywords:	CodeChange, FutureFeature, Improvement
Target Release:	4.19.2	Flags:	mperina: ovirt-4.1? pstehlik: testing_plan_complete- rule-engine: planning_ack? mperina: devel_ack+ pstehlik: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-02-01 14:55:19 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Sandro Bonazzola 2016-08-08 11:07:10 UTC

Description of problem:
# vdsm-tool configure --force
/usr/lib/python2.7/site-packages/vdsm/tool/dump_volume_chains.py:28: DeprecationWarning: vdscli uses xmlrpc. since ovirt 3.6 xmlrpc is deprecated, please use vdsm.jsonrpcvdscli
  from vdsm import vdscli


Version-Release number of selected component (if applicable):
vdsm-python-4.18.999-374.git6f033fe.fc24.noarch

Comment 1 Yaniv Bronhaim 2016-08-09 07:35:25 UTC

some details for the issue: dump_volume_chains is a tool that uses vdscli to retrieve the output. as part of the work to eliminate all xmlrpc usages we need to change all places that uses vdscli module to jsonrpcvdscli.  vdsm-tool loads dump_volume_chains as part of the commands and therefore we see the deprecation warning each call to the tool

Comment 2 Martin Perina 2016-08-09 08:36:26 UTC

Retargeting to 4.1, when patch is ready, we can decide if it's worth to backport.

Comment 3 Lukas Svaty 2016-11-23 13:44:35 UTC

adding CodeChange, no qa process necessary other than basic sanity

Comment 4 Sandro Bonazzola 2016-12-12 13:59:42 UTC

The fix for this issue should be included in oVirt 4.1.0 beta 1 released on December 1st. If not included please move back to modified.

Comment 5 Lukas Svaty 2017-01-31 14:11:43 UTC

verified in vdsm-4.19.2-2.el7ev.x86_64