Bug 906128

Summary: regression: Product.get(include_fields=[..., "components"]) doesn't return any data
Product: [Community] Bugzilla Reporter: Cole Robinson <crobinso>
Component: WebServiceAssignee: Simon Green <sgreen>
Status: CLOSED UPSTREAM QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 4.4CC: ebaak, jingwang
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-06 22:40:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 858932    

Comment 2 Simon Green 2013-02-06 22:21:52 UTC 
The URL has been modified in this post, since the original mentioned an internal server. This test was done with an internal 4.4 server.

$ cat xmlrpconeoff.py
#!/usr/bin/python

import pprint
import xmlrpclib

url = "https://_SECRET_/xmlrpc.cgi"
product = "Virtualization Tools"

bz = xmlrpclib.ServerProxy(url)
ret = bz.Product.get({"names": [product],
                      "include_fields": ["name", "components"]})
pprint.pprint(ret)

$ python xmlrpconeoff.py 
{'products': [{'components': [{},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {},
                              {}],
               'name': 'Virtualization Tools'}]}


Against [Bugzilla 4.2], same code returns results like

{'default_assigned_to': 'Daniel Berrange',
                               'default_qa_contact': '',
                               'description': 'graphical console viewer for virtual machines',
                               'id': 87950,
                               'is_active': True,
                               'name': 'virt-viewer',
                               'sort_key': 0}
Comment 3 Simon Green 2013-02-06 22:40:30 UTC 
This change comes from upstream. I agree that it doesn't seem intuitive. I've filed a bug upstream for their comment. As it stands though, you'll need to change the include_fields value.

  -- simon
Comment 4 Simon Green 2013-02-07 06:21:54 UTC 
Just an update. Not only do upstream think it is a problem, but they have marked it as a blocker for 4.4rc2. I'm working on fixing the bug upstream. As soon as it is committed to bazaar, we'll inherit the change.

  -- simon
Comment 5 Cole Robinson 2013-02-07 15:17:00 UTC 
Okay, thanks simon.