I did push a speculative patch which would address this by not exposing the lookup by int id backdoor - https://review.openstack.org/#/c/281277/ - the results were better than I expected.
Andrey, is this going to negatively impact the openstack/ec2 project at all if we do it? -Sean On 02/16/2016 06:49 AM, Sean Dague wrote: > This was needed originally for ec2 support (which requires an integer > id). It's not really the db index per say, just another id value which > is valid (though hidden) for the server. > > Before unwinding this issue we *must* make sure that the openstack/ec2 > project does not need access to it. > > On 02/15/2016 09:36 PM, Alex Xu wrote: >> I don't think our API supports get servers by DB index is good idea. So >> I prefer we remove it in the future with microversions. But for now, >> yes, it is here. >> >> 2016-02-16 8:03 GMT+08:00 少合冯 <lvmxhs...@gmail.com >> <mailto:lvmxhs...@gmail.com>>: >> >> I guess others may ask the same questions. >> >> I read the nova API doc: >> such as this API: >> http://developer.openstack.org/api-ref-compute-v2.1.html#showServer >> >> GET /v2.1/{tenant_id}/servers/{server_id} >> *Show server details* >> >> >> *Request parameters* >> Parameter Style Type Description >> tenant_id URI csapi:UUID >> >> The UUID of the tenant in a multi-tenancy cloud. >> >> server_id URI csapi:UUID >> >> The UUID of the server. >> >> >> But I can get the server by DB index: >> >> curl -s -H X-Auth-Token:6b8968eb38df47c6a09ac9aee81ea0c6 >> http://192.168.2.103:8774/v2.1/f5a8829cc14c4825a2728b273aa91aa1/servers/2 >> { >> "server": { >> "OS-DCF:diskConfig": "MANUAL", >> "OS-EXT-AZ:availability_zone": "nova", >> "OS-EXT-SRV-ATTR:host": "shaohe1", >> "OS-EXT-SRV-ATTR:hypervisor_hostname": "shaohe1", >> "OS-EXT-SRV-ATTR:instance_name": "instance-00000002", >> "OS-EXT-STS:power_state": 1, >> "OS-EXT-STS:task_state": "migrating", >> "OS-EXT-STS:vm_state": "error", >> "OS-SRV-USG:launched_at": "2015-12-18T07:41:00.000000", >> "OS-SRV-USG:terminated_at": null, >> ...... >> } >> } >> >> and the code really allow it use DB index >> https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1939 >> >> >> __________________________________________________________________________ >> OpenStack Development Mailing List (not for usage questions) >> Unsubscribe: >> openstack-dev-requ...@lists.openstack.org?subject:unsubscribe >> <http://openstack-dev-requ...@lists.openstack.org?subject:unsubscribe> >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> >> >> >> >> __________________________________________________________________________ >> OpenStack Development Mailing List (not for usage questions) >> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> > > -- Sean Dague http://dague.net __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev