need to change compute-api incorrect info
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
openstack-api-site |
Fix Released
|
Low
|
jichenjc |
Bug Description
https:/
has a bunch of to do items, so use this bug to track it
Changed in openstack-api-site: | |
assignee: | nobody → jichenjc (jichenjc) |
Changed in openstack-api-site: | |
status: | New → In Progress |
Atsushi SAKAI (sakaia) wrote : | #2 |
In my impression, you are checking aggregation related problem.
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #3 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit c794d179d4117b2
Author: jichenjc <email address hidden>
Date: Fri Nov 6 07:53:09 2015 +0800
Change nova-api-
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: I02c3c57174e19e
OpenStack Infra (hudson-openstack) wrote : | #4 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 684bacefeeed448
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: I75e9aee7a720b6
jichenjc (jichenjc) wrote : | #5 |
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
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #6 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #7 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #8 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #9 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #10 |
Fix proposed to branch: master
Review: https:/
Atsushi SAKAI (sakaia) wrote : | #11 |
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
jichenjc (jichenjc) wrote : | #12 |
it's in the original bug report , here :)
https:/
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~
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #13 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit b09370ca28821f2
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: Ia0ff940ae5af84
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #14 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit b772750ef8c90de
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: Icaae83b50041e5
OpenStack Infra (hudson-openstack) wrote : | #15 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit eb9526456714090
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: I606e7ba42669c3
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #16 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit f626020667191b6
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: I8a4ca2b4063d0e
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #17 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #18 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #19 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #20 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #21 |
Fix proposed to branch: master
Review: https:/
Changed in openstack-api-site: | |
importance: | Undecided → Low |
milestone: | none → mitaka |
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #22 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 0cd513b353e8960
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: I177683efba6b15
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #23 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #24 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #25 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #26 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit e94448aef0bc8b2
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: Ic5137843f4ab0f
OpenStack Infra (hudson-openstack) wrote : | #27 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit fb6c5554a339e75
Author: jichenjc <email address hidden>
Date: Tue Nov 17 03:44:31 2015 +0800
Remove duplication of X-Compute-
X-Compute-
Partial-Bug: #1515222
Change-Id: Ib5f4c78b00f938
OpenStack Infra (hudson-openstack) wrote : | #28 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4711362abef48d0
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:/
we need to mark this to the end user.
Change-Id: If6166ac3f79cdf
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #29 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #30 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #31 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #32 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 3a6299e3d1abcc3
Author: jichenjc <email address hidden>
Date: Tue Nov 17 19:23:16 2015 +0800
network-
change network-
Partial-Bug: #1515222
Change-Id: I99998a85839d27
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #33 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #34 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 9d5ef6b75f79cb0
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: I092bd9a592e74b
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #35 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 3b6bbcf0be1aa87
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: I5b851f01192acf
OpenStack Infra (hudson-openstack) wrote : | #36 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit ed28e4cb449619a
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: Ib6118c86113496
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #37 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #38 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 394f809c255bed2
Author: jichenjc <email address hidden>
Date: Fri Nov 20 18:37:15 2015 +0800
Change return value from 202 to 200
os-
so we need to change it from 202 to 200.
Change-Id: I0457251c1b81f4
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #39 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit f79f0e7af4fbdde
Author: jichenjc <email address hidden>
Date: Fri Nov 20 04:14:15 2015 +0800
Change output from metadata to meta
/v2.
returns 'meta' instead of 'metadata', so change it
Partial-Bug: #1515222
Change-Id: I7a7047da1bfe07
OpenStack Infra (hudson-openstack) wrote : | #40 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit e84cc55c8255580
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: I5d502e3e9571f7
107 comments hidden Loading more comments | view all 187 comments |
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #148 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #149 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #150 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #151 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #152 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #153 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 95af7c5db5e78a1
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: Ia8e541adb572da
OpenStack Infra (hudson-openstack) wrote : | #154 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 74701951205d12a
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: Ia6d448706ca338
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #155 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 692e2e60bb7d2d8
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: I1779702f374d4c
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #156 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #157 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4489ff57e4b0b4c
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: Id846480172c546
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #158 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #159 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #160 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #161 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #162 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #163 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit d06bb34b5dc2715
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-
Partial-Bug: #1515222
Change-Id: I286dfedbb01d0d
OpenStack Infra (hudson-openstack) wrote : | #164 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 397d4f88e62b4c9
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: I7a1af19dbb2a59
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #165 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit a664294670bbae6
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: I604a0ca10af3d4
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #166 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #167 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #168 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit a04f2352dde6329
Author: jichenjc <email address hidden>
Date: Sun Jan 24 07:56:48 2016 +0800
Add missing hypervisor_
missing this param so the request param is not complete
Change-Id: I8cc613043981b0
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #169 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4d89b226aedea0a
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: Ie16cc25b371ded
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #170 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #171 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 3c70c7d5b88f7cb
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: Ib7cc354759d022
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Fix proposed to api-site (master) | #172 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #173 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #174 |
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Fix merged to api-site (master) | #175 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit b6238d8704a4c7d
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: I3d6214630f696e
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #176 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit e0070b02ac92e74
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: I640ce5d5a8349f
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #177 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 553e5b198eb4aae
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: Ia7d1d8c30bcd93
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #178 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit a6f9f07ec059865
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: I721bb7ba19f11d
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #179 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit c3844faa0363996
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: I834706150d69b3
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #180 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit b3a280befce4bc3
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: I5527b3389af8db
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #181 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 674381e55821631
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: Ice497246348ed5
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #182 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4bd302754e5db45
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: Iff9f0330d98bf6
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : | #183 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4c9452f92255454
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: Iac0f127ce5ac71
Partial-Bug: #1515222
OpenStack Infra (hudson-openstack) wrote : Change abandoned on api-site (master) | #184 |
Change abandoned by jichenjc (<email address hidden>) on branch: master
Review: https:/
Atsushi SAKAI (sakaia) wrote : | #185 |
Hello Jichenjc
Is this issue finished or still remain?
I guess this issue is finished, just for confirmation.
Anne Gentle (annegentle) wrote : | #186 |
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 |
OpenStack Infra (hudson-openstack) wrote : | #187 |
Change abandoned by KATO Tomoyuki (<email address hidden>) on branch: master
Review: https:/
Need to write the problem point.
Currently it is too verbose.