Bug 1044267 - [RFE][python-novaclient]: using floating-ip-associate/disassociate
Summary: [RFE][python-novaclient]: using floating-ip-associate/disassociate
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: RFEs
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: RHOS Maint
QA Contact:
URL: https://blueprints.launchpad.net/pyth...
Whiteboard: upstream_milestone_none upstream_stat...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-18 02:21 UTC by RHOS Integration
Modified: 2015-03-19 16:57 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-03-19 16:57:00 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description RHOS Integration 2013-12-18 02:21:03 UTC
Cloned from launchpad blueprint https://blueprints.launchpad.net/python-novaclient/+spec/commands-floating-ip.

Description:

This blueprint is to add more consistent into nova commands, especially for the subcommand "floating-ip-*"

Currently when we want to associate or disassociate an ip with nova we have to use add-floating-ip and remove-floating-ip. This is not consitent with the actual schem of the subcommands:

example:
  nova image-*
  nova flovor-*
  nova floating-ip-*


I suggest:
In the client v1.1 to add a deprecated comment when using add-floating-ip or remove-floating-ip. In the client v3 to replace add-floating-ip with associate-floating-ip and remove-floating-ip with disassociate-floating-ip.


nova help | grep floating
    add-floating-ip     Add a floating IP address to a server.
    floating-ip-bulk-create
                        Bulk create floating ips by range.
    floating-ip-bulk-delete
                        Bulk delete floating ips by range.
    floating-ip-bulk-list
                        List all floating ips.
    floating-ip-create  Allocate a floating IP for the current tenant.
    floating-ip-delete  De-allocate a floating IP.
    floating-ip-list    List floating ips for this tenant.
    floating-ip-pool-list
                        List all floating ip pools.
    remove-floating-ip  Remove a floating IP address from a server.

I suggest to rename:
   add-floating-ip to floating-ip-associate
   remove-floating-ip to floating-ip-disassociate

Specification URL (additional information):

None


Note You need to log in before you can comment on or make changes to this bug.