Craton's Python Client

CLI requires region id for commands

Bug #1659428 reported by Jim Baker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Craton's Python Client
Fix Released
Critical
Ian Cordasco
Craton's Python Client v0.1.0 "cmdb"

Bug Description

$ craton host-list
usage: craton host-list -r <region> [-c <cell>] [--detail] [--limit <limit>]
                        [--sort-key <field>] [--sort-dir <direction>]
                        [--fields <fields> [<fields> ...]]
craton host-list: error: the following arguments are required: -r/--region

We have removed this requirement from the REST API and the Python client, so this should be removed from the CLI itself.

Revision history for this message
Jim Baker (jimbaker) wrote :

Initial fix in https://review.openstack.org/#/c/427032/

Sulochan Acharya (sulochan-acharya)
affects: craton → python-cratonclient
Jim Baker (jimbaker)
Changed in python-cratonclient:
importance: Undecided → Critical
assignee: nobody → Thomas Maddox (thomas-maddox)
milestone: none → v0.1.0
status: New → In Progress
Thomas Maddox (thomas-maddox)
Changed in python-cratonclient:
assignee: Thomas Maddox (thomas-maddox) → Ian Cordasco (icordasc)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-cratonclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/439629

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cratonclient (master)

Reviewed: https://review.openstack.org/439629
Committed: https://git.openstack.org/cgit/openstack/python-cratonclient/commit/?id=92133f29013663c4f5095185ce0ed353298fd64a
Submitter: Jenkins
Branch: master

commit 92133f29013663c4f5095185ce0ed353298fd64a
Author: Ian Cordasco <email address hidden>
Date: Wed Mar 1 08:25:36 2017 -0600

    Relax requirement of region-id for hosts and cells

    When listing hosts and cells, there is no need to require the user to
    provide a region-id. The API previously required region IDs to be
    present in the query string to list hosts and cells, but relaxed this
    constraint. The craton CLI encoded this constraint in its parameters but
    now needs to relax this as well.

    Change-Id: I973176a150c5d10de2144501dcc39cd6cfacbf9a
    Closes-bug: #1659428

Changed in python-cratonclient:
status: In Progress → Fix Released
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.