Some v3 API extension names are not camelcase

Bug #1315597 reported by Ken'ichi Ohmichi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Ken'ichi Ohmichi

Bug Description

When operating "nova --os-compute-api-version 3 list-extensions" with a patch https://review.openstack.org/#/c/91942/ ,
the output is the following.
So 'extensions', 'flavors' and 'ips' are not camelcase. This seems inconsistent.

$ nova --os-compute-api-version 3 list-extensions
+--------------------------+-----------------------------------------------------------------------+-------------------------------+---------+
| Name | Summary | Alias | Version |
+--------------------------+-----------------------------------------------------------------------+-------------------------------+---------+
| Consoles | Consoles. | consoles | 1 |
| extensions | Extension information. | extensions | 1 |
| FlavorAccess | Flavor access support. | flavor-access | 1 |
| FlavorsExtraSpecs | Flavors Extension. | flavor-extra-specs | 1 |
| FlavorManage | Flavor create/delete API support. | flavor-manage | 1 |
| flavors | Flavors Extension. | flavors | 1 |
| ips | Server addresses. | ips | 1 |
| Keypairs | Keypair Support. | keypairs | 1 |
| AccessIPs | Access IPs support. | os-access-ips | 1 |
| AdminActions | Enable admin-only server actions... | os-admin-actions | 1 |
| AdminPassword | Admin password management support. | os-admin-password | 1 |
| Agents | Agents support. | os-agents | 1 |
| Aggregates | Admin-only aggregate administration. | os-aggregates | 1 |
| AttachInterfaces | Attach interface support. | os-attach-interfaces | 1 |
| AvailabilityZone | 1. Add availability_zone to the Create Server API.... | os-availability-zone | 1 |
| BlockDeviceMapping | Block device mapping boot support. | os-block-device-mapping | 1 |
| Cells | Enables cells-related functionality such as adding neighbor cells,... | os-cells | 1 |
| Certificates | Certificates support. | os-certificates | 1 |
| ConfigDrive | Config Drive Extension. | os-config-drive | 1 |
| ConsoleAuthTokens | Console token authentication support. | os-console-auth-tokens | 1 |
| ConsoleOutput | Console log output support, with tailing ability. | os-console-output | 1 |
| CreateBackup | Create a backup of a server. | os-create-backup | 1 |
| DeferredDelete | Instance deferred delete. | os-deferred-delete | 1 |
| Evacuate | Enables server evacuation. | os-evacuate | 1 |
| ExtendedAvailabilityZone | Extended Server Attributes support. | os-extended-availability-zone | 1 |
| ExtendedServerAttributes | Extended Server Attributes support. | os-extended-server-attributes | 1 |
| ExtendedStatus | Extended Status support. | os-extended-status | 1 |
| ExtendedVolumes | Extended Volumes support. | os-extended-volumes | 1 |
| FlavorRxtx | Support to show the rxtx status of a flavor. | os-flavor-rxtx | 1 |
| HideServerAddresses | Support hiding server addresses in certain states. | os-hide-server-addresses | 1 |
| Hosts | Admin-only host administration. | os-hosts | 1 |
| Hypervisors | Admin-only hypervisor administration. | os-hypervisors | 1 |
| LockServer | Enable lock/unlock server actions. | os-lock-server | 1 |
| MigrateServer | Enable migrate and live-migrate server actions. | os-migrate-server | 1 |
| Migrations | Provide data on migrations. | os-migrations | 1 |
| Multinic | Multiple network support. | os-multinic | 1 |
| MultipleCreate | Allow multiple create in the Create Server v3 API. | os-multiple-create | 1 |
| PauseServer | Enable pause/unpause server actions. | os-pause-server | 1 |
| PCIAccess | Pci access support. | os-pci | 1 |
| Quotas | Quotas management support. | os-quota-sets | 1 |
| RemoteConsoles | Interactive Console support. | os-remote-consoles | 1 |
| Rescue | Instance rescue mode. | os-rescue | 1 |
| SchedulerHints | Pass arbitrary key/value pairs to the scheduler. | os-scheduler-hints | 1 |
| SecurityGroups | Security group support. | os-security-groups | 1 |
| ServerActions | View a log of actions and events taken on an instance. | os-server-actions | 1 |
| ServerDiagnostics | Allow Admins to view server diagnostics through server action. | os-server-diagnostics | 1 |
| ServerExternalEvents | Server External Event Triggers. | os-server-external-events | 1 |
| ServerPassword | Server password support. | os-server-password | 1 |
| ServerUsage | Adds launched_at and terminated_at on Servers. | os-server-usage | 1 |
| Services | Services support. | os-services | 1 |
| Shelve | Instance shelve mode. | os-shelve | 1 |
| SuspendServer | Enable suspend/resume server actions. | os-suspend-server | 1 |
| Server Metadata | Server Metadata API. | server-metadata | 1 |
| Servers | Servers. | servers | 1 |
| Versions | API Version information. | versions | 1 |
+--------------------------+-----------------------------------------------------------------------+-------------------------------+---------+

Changed in nova:
assignee: nobody → Ken'ichi Ohmichi (oomichi)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/91943
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=523e52f2f754f35b1e2453190f935d1a990464fb
Submitter: Jenkins
Branch: master

commit 523e52f2f754f35b1e2453190f935d1a990464fb
Author: Ken'ichi Ohmichi <email address hidden>
Date: Sat May 3 14:22:02 2014 +0900

    Fix v3 API extension names for camelcase

    Most v3 API names are camelcase, but few APIs are not.
    This patch changes them to camelcase for the consistency.

    Change-Id: I0dc6c23ffcdcf97fc52d3575d521692805ffd9e3
    Closes-Bug: #1315597

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/93891

Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/93891
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=ba8d6c01d743a3d24739de1aecc2cf3cd22fa00a
Submitter: Jenkins
Branch: master

commit ba8d6c01d743a3d24739de1aecc2cf3cd22fa00a
Author: Ken'ichi Ohmichi <email address hidden>
Date: Fri May 16 19:47:14 2014 +0900

    Add tests to avoid inconsistent extension names

    On v2.1/v3 API, extension names should be camelcases.
    I0dc6c23ffcdcf97fc52d3575d521692805ffd9e3 has fixed inconsistent
    names already.
    This patch adds tests for blocking these inconsistent name/alias
    additions in the future.

    Change-Id: I3c9ee3f59fd17394b251fedf8df592e5d6304962
    Related-Bug: #1315597

Thierry Carrez (ttx)
Changed in nova:
milestone: juno-1 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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