need to change compute-api incorrect info

Bug #1515222 reported by jichenjc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-api-site
Fix Released
Low
jichenjc

Bug Description

https://etherpad.openstack.org/p/nova-v2.1-api-doc
has a bunch of to do items, so use this bug to track it

Tags: nova
jichenjc (jichenjc)
Changed in openstack-api-site:
assignee: nobody → jichenjc (jichenjc)
Changed in openstack-api-site:
status: New → In Progress
Revision history for this message
Atsushi SAKAI (sakaia) wrote :

Need to write the problem point.
Currently it is too verbose.

tags: added: nova
Revision history for this message
Atsushi SAKAI (sakaia) wrote :

In my impression, you are checking aggregation related problem.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/244038
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=c794d179d4117b20427ea136c8165be5ad8741cf
Submitter: Jenkins
Branch: master

commit c794d179d4117b20427ea136c8165be5ad8741cf
Author: jichenjc <email address hidden>
Date: Fri Nov 6 07:53:09 2015 +0800

    Change nova-api-create-aggregate return code

    conceptually the create-aggregate should return 201,
    however current code returns 200 so in order to make
    them consistent, make this change and later by using
    microversion, the value can be switch back to 201.

    Partial-Bug: #1515222

    Change-Id: I02c3c57174e19e4ef604a2b840d76652840cdc13

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/244047
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=684bacefeeed448853a8fba6d9db407ba1f7ce4b
Submitter: Jenkins
Branch: master

commit 684bacefeeed448853a8fba6d9db407ba1f7ce4b
Author: jichenjc <email address hidden>
Date: Fri Nov 6 08:30:14 2015 +0800

    Aggregate id is a integer, not a uuid

    Id in Aggregate is defined as
    id = Column(Integer, primary_key=True, autoincrement=True)
    and it's input from API layer, so it's not a uuid format,
    it's a integer.

    Partial-Bug: #1515222

    Change-Id: I75e9aee7a720b69bf5157f6dc1fab9dc655ac54a

Revision history for this message
jichenjc (jichenjc) wrote :

sorry, it's a whole bunch of changes to be made , aggregate is the first one
I expect more will come if you refer to the etherpad, thanks

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
Atsushi SAKAI (sakaia) wrote :

Would you point the etherpad at least?
In my understanding, fixation list is not available on any etherpad.

> I expect more will come if you refer to the etherpad, thanks

Revision history for this message
jichenjc (jichenjc) wrote :

it's in the original bug report , here :)
https://etherpad.openstack.org/p/nova-v2.1-api-doc

but yes, we are doing this step by step, so find some and submit patch and on the same time, do the analysis and check ,thanks~

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/244312
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=b09370ca28821f256b367aced45270fa0daf3b40
Submitter: Jenkins
Branch: master

commit b09370ca28821f256b367aced45270fa0daf3b40
Author: jichenjc <email address hidden>
Date: Fri Nov 6 12:27:19 2015 +0800

    change return code for add-host and remove-host

    aggregate's add-host and remove-host action currently
    returns 200 instead of 202.

    Change-Id: Ia0ff940ae5af84e4d683540b094ab74b09971848
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/244310
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=b772750ef8c90de8b164adafa336d6b11c03f40c
Submitter: Jenkins
Branch: master

commit b772750ef8c90de8b164adafa336d6b11c03f40c
Author: jichenjc <email address hidden>
Date: Fri Nov 6 12:21:59 2015 +0800

    update aggregate is a PUT method

    update aggregate is a PUT instead of GET, fix the typo.

    Partial-Bug: #1515222

    Change-Id: Icaae83b50041e5f236b4f03c2057bc65171f6b28

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/244324
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=eb9526456714090a5b09f416c2f82fe44d37dc07
Submitter: Jenkins
Branch: master

commit eb9526456714090a5b09f416c2f82fe44d37dc07
Author: jichenjc <email address hidden>
Date: Fri Nov 6 12:54:58 2015 +0800

    make policies param mandatory

    the policies param in create server group function is not
    optional, it's mandatory, remove 'optional' for this param.

    Change-Id: I606e7ba42669c351058334f07d166b3845ae91f6
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/244326
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=f626020667191b6e7dc6813646f3f0355ea6a8ca
Submitter: Jenkins
Branch: master

commit f626020667191b6e7dc6813646f3f0355ea6a8ca
Author: jichenjc <email address hidden>
Date: Fri Nov 6 13:00:34 2015 +0800

    change comments for hypervisor statistics

    change from 'Shows statistics for all hypervisors. ' to
    'Shows statistics of the summary of all hypervisors'

    Change-Id: I8a4ca2b4063d0e8ba6e3c8291104e10e603552d7
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in openstack-api-site:
importance: Undecided → Low
milestone: none → mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/246319
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=0cd513b353e8960c1e8a4d94e8a9ee9db7e7a62a
Submitter: Jenkins
Branch: master

commit 0cd513b353e8960c1e8a4d94e8a9ee9db7e7a62a
Author: jichenjc <email address hidden>
Date: Tue Nov 17 03:36:28 2015 +0800

    Remove description for 413 as exceed quota error

    we used to return 413 for quota exceed, now we return
    403 instead (HTTPForbidden)

    Partial-Bug: #1515222

    Change-Id: I177683efba6b15b54320285b2f57140b86b699bc

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/246297
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=e94448aef0bc8b2fc32fe506af5be1ab2febf4a7
Submitter: Jenkins
Branch: master

commit e94448aef0bc8b2fc32fe506af5be1ab2febf4a7
Author: jichenjc <email address hidden>
Date: Tue Nov 17 03:11:33 2015 +0800

    Remove duplication of response on images

    images has duplicated response output, remove one of them.

    Partial-Bug: #1515222

    Change-Id: Ic5137843f4ab0f5fc268da09a94b84738b5fe8ba

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/246321
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=fb6c5554a339e75cab251d1292d823344d695e30
Submitter: Jenkins
Branch: master

commit fb6c5554a339e75cab251d1292d823344d695e30
Author: jichenjc <email address hidden>
Date: Tue Nov 17 03:44:31 2015 +0800

    Remove duplication of X-Compute-Request-ID

    X-Compute-Request-ID duplicated in image metadata, remove it

    Partial-Bug: #1515222

    Change-Id: Ib5f4c78b00f938e4dbfa02b8ae32a7bbb5cbc24c

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/246288
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=4711362abef48d077ab6f27abd379cbf1fc94624
Submitter: Jenkins
Branch: master

commit 4711362abef48d077ab6f27abd379cbf1fc94624
Author: jichenjc <email address hidden>
Date: Tue Nov 17 02:47:00 2015 +0800

    Add description for metadata is empty

    server-group return data's metadata field is always empty
    https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/server_groups.py#L58
    we need to mark this to the end user.

    Change-Id: If6166ac3f79cdf03ebd4c75f2ba568f842d85c17
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/247206
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=3a6299e3d1abcc3eca16a44e7e20303c712c99f0
Submitter: Jenkins
Branch: master

commit 3a6299e3d1abcc3eca16a44e7e20303c712c99f0
Author: jichenjc <email address hidden>
Date: Tue Nov 17 19:23:16 2015 +0800

    network-vif-deletedplugged is a typo

    change network-vif-deletedplugged to network-vif-deleted

    Partial-Bug: #1515222

    Change-Id: I99998a85839d27131eb828bcfeb8e75e5b495eb9

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/246312
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=9d5ef6b75f79cb043e78f88ed3a86b500e65d431
Submitter: Jenkins
Branch: master

commit 9d5ef6b75f79cb043e78f88ed3a86b500e65d431
Author: jichenjc <email address hidden>
Date: Tue Nov 17 03:33:45 2015 +0800

    Add minRam and minDisk as filter

    minRam and minDisk filter is missing, add it.

    (df) Cleaned up language for query (filter) parameters

    Change-Id: I092bd9a592e74b6a80ca9e46469a8997b06e5080
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/246851
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=3b6bbcf0be1aa87b75eb5a757f5e0a3a2873debd
Submitter: Jenkins
Branch: master

commit 3b6bbcf0be1aa87b75eb5a757f5e0a3a2873debd
Author: jichenjc <email address hidden>
Date: Tue Nov 17 13:03:25 2015 +0800

    Remove exceed quota will return 413 notification

    Nova now don't return HTTP code 413 in case the quota
    exceed. so no need to mention this in the doc any more.

    Partial-Bug: #1515222

    Change-Id: I5b851f01192acfd4bfec7559e38db610a9028c9e

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/246845
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=ed28e4cb449619af0fbe7b8471abd3ea27e85c49
Submitter: Jenkins
Branch: master

commit ed28e4cb449619af0fbe7b8471abd3ea27e85c49
Author: jichenjc <email address hidden>
Date: Tue Nov 17 12:45:06 2015 +0800

    change 'server_uuid' to 'instance_uuid'

    nova actually returns instance_uuid instead of server_uuid.

    Partial-Bug: #1515222

    Change-Id: Ib6118c8611349619c7b4337211e449682afc9452

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/248012
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=394f809c255bed2bf4acf9f23a60418f43643b4b
Submitter: Jenkins
Branch: master

commit 394f809c255bed2bf4acf9f23a60418f43643b4b
Author: jichenjc <email address hidden>
Date: Fri Nov 20 18:37:15 2015 +0800

    Change return value from 202 to 200

    os-tenant-networks create returns 200 if it succeed.
    so we need to change it from 202 to 200.

    Change-Id: I0457251c1b81f42c7ff78d003a4369f7403ff508
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/247768
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=f79f0e7af4fbdde4ac86a16aae0f8a8d0518085a
Submitter: Jenkins
Branch: master

commit f79f0e7af4fbdde4ac86a16aae0f8a8d0518085a
Author: jichenjc <email address hidden>
Date: Fri Nov 20 04:14:15 2015 +0800

    Change output from metadata to meta

    /v2.1/images/{image_id}/metadata/{key}
    returns 'meta' instead of 'metadata', so change it

    Partial-Bug: #1515222

    Change-Id: I7a7047da1bfe072913045bf4ba78711003db2268

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/246850
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=e84cc55c8255580768286b9e32ffafe63c3da36c
Submitter: Jenkins
Branch: master

commit e84cc55c8255580768286b9e32ffafe63c3da36c
Author: jichenjc <email address hidden>
Date: Tue Nov 17 12:59:15 2015 +0800

    request_id is missing in the request param

    request_id is missing in the request param of instance-action
    add this to the param list.

    Partial-Bug: #1515222

    Change-Id: I5d502e3e9571f7d3f34909c70e0e3baa5a94f60e

107 comments hidden view all 187 comments
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/275076
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=95af7c5db5e78a15ccbde451ab1d4517880d6ccd
Submitter: Jenkins
Branch: master

commit 95af7c5db5e78a15ccbde451ab1d4517880d6ccd
Author: jichenjc <email address hidden>
Date: Mon Jan 25 18:10:13 2016 +0800

    change param from user to user_id

    user_id is the correct param , also it's accepted by GET request.

    blueprint fix-compute-api-ref
    Partial-Bug: #1515222

    Change-Id: Ia8e541adb572dad6161dc5895c297949409048f8

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/271969
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=74701951205d12a6d6491e0a985e97f8e63a9577
Submitter: Jenkins
Branch: master

commit 74701951205d12a6d6491e0a985e97f8e63a9577
Author: jichenjc <email address hidden>
Date: Sat Jan 23 09:09:47 2016 +0800

    Add description for os-hosts

    some virt driver code don't support all host functions

    Change-Id: Ia6d448706ca33809f155d8ab66df3ce737f51f0f
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/275068
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=692e2e60bb7d2d8df21587c7082d8642f2fe2a90
Submitter: Jenkins
Branch: master

commit 692e2e60bb7d2d8df21587c7082d8642f2fe2a90
Author: jichenjc <email address hidden>
Date: Mon Jan 25 17:45:44 2016 +0800

    The quota ID is UUID instead of int

    change the type of quota ID from int to UUID

    blueprint fix-compute-api-ref
    Partial-Bug: #1515222

    Change-Id: I1779702f374d4c6fc5328ed3cf0a1fa267459ba6

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/275629
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=4489ff57e4b0b4cee4299a9c67321a4bdd06efaf
Submitter: Jenkins
Branch: master

commit 4489ff57e4b0b4cee4299a9c67321a4bdd06efaf
Author: jichenjc <email address hidden>
Date: Tue Jan 26 06:14:22 2016 +0800

    Change descriptions of default policy of networks

    for list, show and create action of networks, the doc is different
    to default code policy, this patch changed them

    blueprint fix-compute-api-ref

    Change-Id: Id846480172c546514c63c1d55b60bf557f10a78b
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/277329
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=d06bb34b5dc27150559121500572c48ac47b8f16
Submitter: Jenkins
Branch: master

commit d06bb34b5dc27150559121500572c48ac47b8f16
Author: jichenjc <email address hidden>
Date: Wed Jan 27 10:55:39 2016 +0800

    rules is a dict, not a string

    the response of secgroup list include a rules list, so
    it's a dict , not a string.

    fix-compute-api-ref
    Partial-Bug: #1515222

    Change-Id: I286dfedbb01d0da29e7d42c4ac82c9e7c4ae52dd

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/273999
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=397d4f88e62b4c9e45db0408ce37463bef879905
Submitter: Jenkins
Branch: master

commit 397d4f88e62b4c9e45db0408ce37463bef879905
Author: jichenjc <email address hidden>
Date: Mon Jan 25 01:21:27 2016 +0800

    Add descriptions for server groups policy

    add descriptions for valid input of server group policy
    also, the type of the policy field is list ,not a dict.

    Change-Id: I7a1af19dbb2a5924aee70b10a465d38fb8521497
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/278980
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=a664294670bbae6f0e94b632584c090b65d0f1d6
Submitter: Jenkins
Branch: master

commit a664294670bbae6f0e94b632584c090b65d0f1d6
Author: jichenjc <email address hidden>
Date: Fri Jan 29 01:01:17 2016 +0800

    Detach failed will keep the volume in in-use

    currently we say it's 'available' but this is for success
    case, for fail caes, it will be kept in in-use state

    Change-Id: I604a0ca10af3d4b0fdb2f213cf1f3ccff97b2b7d
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/272955
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=a04f2352dde632912ec556dba5a905ed71400850
Submitter: Jenkins
Branch: master

commit a04f2352dde632912ec556dba5a905ed71400850
Author: jichenjc <email address hidden>
Date: Sun Jan 24 07:56:48 2016 +0800

    Add missing hypervisor_idTemplateParameter

    missing this param so the request param is not complete

    Change-Id: I8cc613043981b0f2e834eaa65241998f87f0b543
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/278973
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=4d89b226aedea0a0be8f3a8f959feac3d90c5998
Submitter: Jenkins
Branch: master

commit 4d89b226aedea0a0be8f3a8f959feac3d90c5998
Author: jichenjc <email address hidden>
Date: Fri Jan 29 00:44:24 2016 +0800

    Adjust the request param sequence

    the volumeAttachment contains volumeId and device
    so it should be on top of them as convention of whole book.

    Change-Id: Ie16cc25b371dedc775520ab0db79e14129ecbe2d
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/275622
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=3c70c7d5b88f7cb2b0606b658726d178f0726fba
Submitter: Jenkins
Branch: master

commit 3c70c7d5b88f7cb2b0606b658726d178f0726fba
Author: jichenjc <email address hidden>
Date: Tue Jan 26 06:08:42 2016 +0800

    Change creates network return from 202 to 200

    currently the nova-network returns 200 instead of 202

    blueprint fix-compute-api-ref

    Change-Id: Ib7cc354759d022300b085139b15f630ea6562323
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master)

Reviewed: https://review.openstack.org/281743
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=b6238d8704a4c7de53dcfb797cf0fe63de4d908b
Submitter: Jenkins
Branch: master

commit b6238d8704a4c7de53dcfb797cf0fe63de4d908b
Author: jichenjc <email address hidden>
Date: Thu Feb 18 18:18:28 2016 +0800

    Remove duplicated error code

    remove duplicated error code in interface

    Change-Id: I3d6214630f696e3f92a631ac6200c65e8f381108
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/280609
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=e0070b02ac92e741b5aacf9f9175ffeacee0d039
Submitter: Jenkins
Branch: master

commit e0070b02ac92e741b5aacf9f9175ffeacee0d039
Author: jichenjc <email address hidden>
Date: Sat Jan 30 01:47:00 2016 +0800

    Add show command for os-quota-sets

    show is a valid command and valid path for os-quota-sets
    we missed it now and this patch added it.

    Change-Id: I640ce5d5a8349f6cd96ec2461e5d0556755ef761
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/281736
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=553e5b198eb4aae97baf0f17bfa68cad5f684c9b
Submitter: Jenkins
Branch: master

commit 553e5b198eb4aae97baf0f17bfa68cad5f684c9b
Author: jichenjc <email address hidden>
Date: Thu Feb 18 18:13:49 2016 +0800

    Change attachment_id to port_id

    there is no attachment_id concept in the nova, it's the port
    id that is used.

    Change-Id: Ia7d1d8c30bcd93eb092814019fbb895c326fbcd3
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/278969
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=a6f9f07ec0598652e03887e0e1c5f0073aca7e4a
Submitter: Jenkins
Branch: master

commit a6f9f07ec0598652e03887e0e1c5f0073aca7e4a
Author: jichenjc <email address hidden>
Date: Fri Jan 29 00:37:52 2016 +0800

    Remove duplicate error code in volume

    commandFault and getFault are duplicated, remove them

    Change-Id: I721bb7ba19f11dd149d54dfda4e83eb8358773f8
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/281746
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=c3844faa03639961b4b35dac754f7f76813d7767
Submitter: Jenkins
Branch: master

commit c3844faa03639961b4b35dac754f7f76813d7767
Author: jichenjc <email address hidden>
Date: Thu Feb 18 18:26:48 2016 +0800

    Change fixed_ip from string to list

    fixed_ip is list, not a string

    Change-Id: I834706150d69b3dfd9f427ae21fb312ac88c4ab4
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/280146
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=b3a280befce4bc357d9bc9617e57df277c479854
Submitter: Jenkins
Branch: master

commit b3a280befce4bc357d9bc9617e57df277c479854
Author: jichenjc <email address hidden>
Date: Fri Jan 29 08:56:47 2016 +0800

    defaults not accept a 'user id' as param

    defaults of os-quota-sets do not accept a user id as input
    param, so remove it.

    Change-Id: I5527b3389af8db09e1bcdafbb438359b24b98a9a
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/280150
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=674381e55821631c0fe39dc4a98aa9cf5c327ddf
Submitter: Jenkins
Branch: master

commit 674381e55821631c0fe39dc4a98aa9cf5c327ddf
Author: jichenjc <email address hidden>
Date: Fri Jan 29 09:11:19 2016 +0800

    Params are optional, not mandatory

    those changed request params are not mandotary.

    Change-Id: Ice497246348ed51df6c1f7eab63df42cd9815fdc
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/277752
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=4bd302754e5db4516931df696577372e0b4af555
Submitter: Jenkins
Branch: master

commit 4bd302754e5db4516931df696577372e0b4af555
Author: jichenjc <email address hidden>
Date: Wed Jan 27 23:22:07 2016 +0800

    Add version path for v2.1 and v2

    nova allow /v2 and /v2.1 as path to search version now
    so sync doc with code.

    Change-Id: Iff9f0330d98bf617250e2ad5d09c3c8ebac062e3
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/278968
Committed: https://git.openstack.org/cgit/openstack/api-site/commit/?id=4c9452f9225545499cb1fb11f45ffdcaf3d05c10
Submitter: Jenkins
Branch: master

commit 4c9452f9225545499cb1fb11f45ffdcaf3d05c10
Author: jichenjc <email address hidden>
Date: Fri Jan 29 00:32:12 2016 +0800

    Change return code of POST&GET for volume attach

    The return code is 200 instead of 202 according to code
    so ,change the return code.

    Change-Id: Iac0f127ce5ac7156a5fccae0154cdd2e9e853d91
    Partial-Bug: #1515222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on api-site (master)

Change abandoned by jichenjc (<email address hidden>) on branch: master
Review: https://review.openstack.org/273725

Revision history for this message
Atsushi SAKAI (sakaia) wrote :

Hello Jichenjc

Is this issue finished or still remain?
I guess this issue is finished, just for confirmation.

Revision history for this message
Anne Gentle (annegentle) wrote :

At the nova design summit session, Ken'ichi said he would review the etherpad and put new bugs in nova as needed. So we can close this one.

Changed in openstack-api-site:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by KATO Tomoyuki (<email address hidden>) on branch: master
Review: https://review.openstack.org/276648

Displaying first 40 and last 40 comments. View all 187 comments or add a comment.
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.