OS API 1.1 has id and uuid in server details
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Medium
|
Brian Waldon |
Bug Description
For example, in this response:
{
"server": {
},
"hostId": "",
"id": 33,
"imageRef": 3,
"links": [
{
},
{
},
{
}
],
"metadata": {
"key1": "value1",
"key2": "value2"
},
"name": "testserver",
"status": "BUILD",
"uuid": "97ce15ba-
}
}
I believe there should only be the 'id' field, and its value should be the value of the 'uuid' field.
Related branches
- Nova Core security contacts: Pending requested
-
Diff: 10619 lines (+2448/-1783)58 files modifiedbin/nova-manage (+1/-1)
nova/api/ec2/__init__.py (+2/-2)
nova/api/ec2/cloud.py (+45/-23)
nova/api/ec2/ec2utils.py (+18/-1)
nova/api/openstack/contrib/volumes.py (+8/-8)
nova/api/openstack/images.py (+4/-1)
nova/api/openstack/schemas/v1.1/server.rng (+0/-1)
nova/api/openstack/servers.py (+107/-67)
nova/api/openstack/views/servers.py (+18/-8)
nova/compute/api.py (+234/-215)
nova/compute/manager.py (+203/-220)
nova/console/manager.py (+4/-4)
nova/console/vmrc.py (+2/-2)
nova/db/api.py (+91/-93)
nova/db/sqlalchemy/api.py (+121/-142)
nova/db/sqlalchemy/migrate_repo/versions/031_fk_fixed_ips_virtual_interface_id.py (+0/-1)
nova/db/sqlalchemy/migrate_repo/versions/050_renames_for_instance_uuids.py (+117/-0)
nova/db/sqlalchemy/migrate_repo/versions/050_sqlite_downgrade.sql (+266/-0)
nova/db/sqlalchemy/migrate_repo/versions/050_sqlite_upgrade.sql (+265/-0)
nova/db/sqlalchemy/migration.py (+5/-2)
nova/db/sqlalchemy/models.py (+38/-28)
nova/exception.py (+6/-9)
nova/network/api.py (+9/-9)
nova/network/manager.py (+64/-69)
nova/network/quantum/manager.py (+11/-11)
nova/network/quantum/nova_ipam_lib.py (+1/-1)
nova/notifier/api.py (+1/-1)
nova/scheduler/api.py (+22/-42)
nova/scheduler/driver.py (+6/-6)
nova/scheduler/zone.py (+2/-2)
nova/tests/api/ec2/test_cloud.py (+66/-49)
nova/tests/api/ec2/test_middleware.py (+1/-1)
nova/tests/api/openstack/contrib/test_volumes.py (+0/-1)
nova/tests/api/openstack/test_images.py (+8/-0)
nova/tests/api/openstack/test_server_actions.py (+24/-19)
nova/tests/api/openstack/test_servers.py (+185/-223)
nova/tests/api/openstack/test_zones.py (+6/-6)
nova/tests/fake_network.py (+5/-5)
nova/tests/integrated/test_volumes.py (+1/-1)
nova/tests/scheduler/test_scheduler.py (+28/-41)
nova/tests/test_compute.py (+274/-299)
nova/tests/test_console.py (+14/-14)
nova/tests/test_db_api.py (+9/-9)
nova/tests/test_libvirt.py (+34/-31)
nova/tests/test_metadata.py (+1/-0)
nova/tests/test_network.py (+20/-20)
nova/tests/test_quantum.py (+4/-4)
nova/tests/test_test_utils.py (+1/-1)
nova/tests/test_volume.py (+18/-18)
nova/tests/test_xenapi.py (+22/-15)
nova/tests/xenapi/stubs.py (+1/-1)
nova/utils.py (+1/-1)
nova/virt/fake.py (+1/-1)
nova/virt/libvirt/connection.py (+4/-4)
nova/virt/libvirt/firewall.py (+9/-9)
nova/virt/vmwareapi_conn.py (+0/-1)
nova/virt/xenapi/vmops.py (+39/-39)
nova/virt/xenapi_conn.py (+1/-1)
Changed in nova: | |
assignee: | nobody → Brian Waldon (bcwaldon) |
status: | Incomplete → In Progress |
Changed in nova: | |
importance: | Undecided → Medium |
tags: | added: osapi-v1.1 |
Changed in nova: | |
assignee: | Brian Waldon (bcwaldon) → nobody |
status: | In Progress → Triaged |
Changed in nova: | |
assignee: | nobody → Brian Waldon (bcwaldon) |
status: | Triaged → In Progress |
Changed in nova: | |
milestone: | none → essex-1 |
Changed in nova: | |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | essex-1 → 2012.1 |
Looking at the spec, the uuid field is certainly superfluous, but I'm not sure its value should replace the one under id.