Bug 769179

Summary: [VDSM] - vdsClient getSessionList gives error (method "getSessionList" is not supported')
Product: [Retired] oVirt Reporter: David Botzer <dbotzer>
Component: vdsmAssignee: Dan Kenigsberg <danken>
Status: CLOSED CURRENTRELEASE QA Contact: David Botzer <dbotzer>
Severity: low Docs Contact:
Priority: low    
Version: unspecifiedCC: abaron, acathrow, bazulay, hateya, iheim, pstehlik, yeylon, ykaul
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: v4.9.3 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-17 07:49:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description David Botzer 2011-12-20 07:43:39 UTC
Description of problem:
running the command vdsClient -s 0 getSessionList returns error message.

Version-Release number of selected component (if applicable):
IC153.1
rhevm-3.0.1_0001-3.el6.x86_64
vdsm-4.9-112.1.el6.x86_64

How reproducible:
always

Steps to Reproduce:
1.setup rhevm with host and storage
2.run the command "vdsClient -s 0 getSessionList "
  
Actual results:
gives error message

Expected results:
should output the following command's output -> iscsiadm -m session -R

Additional info:

Haim:
------
     def getSessionList(self, args):
        sessions = self.s.getSessionList()
        for session in sessions['sessions']:
            print session
        return 0, ''

added a print for session and its an empty dict, args is 0 as well and not required by optparse
-------------------------

vdsClient -s 0 getSessionList
and got error,
I got the same on 10.35.70.13

Please advise,

Traceback (most recent call last):
  File "/usr/share/vdsm/vdsClient.py", line 1987, in <module>
    code, message = commands[command][0](commandArgs)
  File "/usr/share/vdsm/vdsClient.py", line 558, in getSessionList
    sessions = self.s.getSessionList()
  File "/usr/lib64/python2.6/xmlrpclib.py", line 1199, in __call__
    return self.__send(self.__name, args)
  File "/usr/lib64/python2.6/xmlrpclib.py", line 1489, in __request
    verbose=self.__verbose
  File "/usr/lib64/python2.6/site-packages/M2Crypto/m2xmlrpclib.py", line 64, in request
    return self.parse_response(h.getfile())
  File "/usr/lib64/python2.6/xmlrpclib.py", line 1361, in parse_response
    return self._parse_response(file, None)
  File "/usr/lib64/python2.6/xmlrpclib.py", line 1392, in _parse_response
    return u.close()
  File "/usr/lib64/python2.6/xmlrpclib.py", line 838, in close
    raise Fault(**self._stack[0])
Fault: <Fault 1: '<type \'exceptions.Exception\'>:method "getSessionList" is not supported'>

Comment 1 Haim 2011-12-20 09:37:30 UTC
http://gerrit.ovirt.org/#change,659

Comment 2 David Botzer 2012-02-19 12:17:35 UTC
Hi Dan,

I checked with Haim and this bug is only fixed Upstream,
What about Downstream ?
David,

Comment 3 Dan Kenigsberg 2012-02-19 13:00:12 UTC
(In reply to comment #2)
> Hi Dan,
> 
> I checked with Haim and this bug is only fixed Upstream,
> What about Downstream ?
> David,

The bug is not important enough to backport into rhev-3.0.z (vdsClient is not supported at all). Wait for rhev-3.1 and you'll have it downstream.

Comment 4 David Botzer 2012-02-20 10:56:29 UTC
So Should I close this bug / re-assign or else ?

Comment 5 Haim 2012-02-20 11:21:04 UTC
(In reply to comment #4)
> So Should I close this bug / re-assign or else ?

Dan is right (as always) however, this bug should move to assigned till backport is done.

Comment 6 Dan Kenigsberg 2012-02-20 12:04:31 UTC
Haim, no. This is an oVirt bug. It is now solved in oVirt.

If it was a RHEV bug, it should have been backported. But it is won't ever be bacported, as it is not important enough.

Check it upstream, verify it there, and close it.

If you REALLY want, you can move the bug to RHEL-6.3, check it there, verify and close it.

In any case, it does not worth our worries.

Comment 7 Haim 2012-02-20 12:21:36 UTC
(In reply to comment #6)
> Haim, no. This is an oVirt bug. It is now solved in oVirt.
> 
> If it was a RHEV bug, it should have been backported. But it is won't ever be
> bacported, as it is not important enough.
> 
> Check it upstream, verify it there, and close it.
> 
> If you REALLY want, you can move the bug to RHEL-6.3, check it there, verify
> and close it.
> 
> In any case, it does not worth our worries.

sorry !!! i thought that the bug was opened on RHEL, my bad.