Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-17 Thread Anne Gentle
On Wed, Feb 17, 2016 at 8:49 AM, Sean Dague wrote: > 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

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-17 Thread Sean Dague
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

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-16 Thread Sean Dague
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

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-16 Thread Alex Xu
HYAM MANN <ghanshyamm...@gmail.com> wrote: - > To: "OpenStack Development Mailing List (not for usage questions)" < > openstack-dev@lists.openstack.org> > From: GHANSHYAM MANN <ghanshyamm...@gmail.com> > Date: 02/16/2016 03:49AM > Subject: Re: [openstack-dev] [

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-16 Thread Chen CH Ji
.openstack.org>From: GHANSHYAM MANN <ghanshyamm...@gmail.com>Date: 02/16/2016 03:49AMSubject: Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?Yes, currently Nova support that for show/update/delete server APIs etc (both v2 and v2.1) and python-novaclient t

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-15 Thread 少合冯
File a bug. https://bugs.launchpad.net/openstack-api-site/+bug/1545922 Anne, Alex, and Ghanshyam Mann, Can this be raised in the API meeting for discussion? I'm very care my patch: https://review.openstack.org/#/c/258771/12/nova/api/openstack/compute/server_migrations.py Should I allow it as

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-15 Thread GHANSHYAM MANN
Yes, currently Nova support that for show/update/delete server APIs etc (both v2 and v2.1) and python-novaclient too. But I think that was old behaviour and for ec2 API mainly? I searched on ec2 repo [1] and they get the instance from nova using UUID, i did not find any place they are fetching

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-15 Thread Alex Xu
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 少合冯 : > I guess others may ask the same questions. > > I read the nova API doc: > such as

Re: [openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-15 Thread Anne Gentle
On Mon, Feb 15, 2016 at 6:03 PM, 少合冯 wrote: > 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

[openstack-dev] [Nova][API] Does nova API allow the server_id parem as DB index?

2016-02-15 Thread 少合冯
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* ParameterStyleTypeDescription tenant_id URI