python-novaclient

novaclient.utils.print_dict should be replaced with novaclient.common.cliutils.print_dict

Bug #1503841 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Won't Fix
Low
Matt Riedemann
python-novaclient
Invalid
Low
Xiao Li Xu

Bug Description

The code in novaclient.utils.print_dict is very similar (and arguably better in some ways) to the code in novaclient.common.cliutils.print_dict which comes from oslo-incubator.

We should sync up the novaclient version with the oslo-incubator version in the o-i repo, then sync that back to novaclient and use it from novaclient.common.cliutils rather than the one in novaclient.utils.

Matt Riedemann (mriedem)
tags: added: low-hanging-fruit
Changed in python-novaclient:
status: New → Confirmed
Changed in oslo-incubator:
status: New → Confirmed
Revision history for this message
Matt Riedemann (mriedem) wrote :

A couple of changes here:

https://review.openstack.org/#/q/status:open+project:openstack/oslo-incubator+branch:master+topic:bug/1503841,n,z

Changed in oslo-incubator:
status: Confirmed → In Progress
assignee: nobody → Matt Riedemann (mriedem)
importance: Undecided → Low
Changed in python-novaclient:
importance: Undecided → Low
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

All code in oslo-incubator will be deleted soon. Please fix this up in nova-client itself.

http://markmail.org/message/kgdnukenoni3hbmw

thanks,
dims

Changed in oslo-incubator:
status: In Progress → Won't Fix
Xiao Li Xu (xiao-li-xu)
Changed in python-novaclient:
assignee: nobody → IBM-Cloud-SH (ibm-cloud-sh)
Xiao Li Xu (xiao-li-xu)
Changed in python-novaclient:
assignee: IBM-Cloud-SH (ibm-cloud-sh) → nobody
assignee: nobody → IBM-Cloud-SH (ibm-cloud-sh)
Revision history for this message
AMIT KUMAR (maurya0092) wrote :

Hi Matt Riedemann,

Is this bug still valid?
As I'm not able to locate "novaclient.common" module in master branch of novaclient on github
( https://github.com/openstack/python-novaclient/tree/master/novaclient ).

Revision history for this message
Matt Riedemann (mriedem) wrote :

No this is no longer valid, the novaclient.common (oslo-incubator) code was recently removed from novaclient.

Changed in python-novaclient:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.