Bug 1044267

Summary: [RFE][python-novaclient]: using floating-ip-associate/disassociate
Product: Red Hat OpenStack Reporter: RHOS Integration <rhos-integ>
Component: RFEsAssignee: RHOS Maint <rhos-maint>
Status: CLOSED UPSTREAM QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: markmc, yeylon
Target Milestone: ---Keywords: FutureFeature
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: https://blueprints.launchpad.net/python-novaclient/+spec/commands-floating-ip
Whiteboard: upstream_milestone_none upstream_status_implemented upstream_definition_approved
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-03-19 16:57:00 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

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