Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-15
18:52:55 sdague hmmm... ok
18:53:44 mriedem yeah i'm good with extra specs
18:53:52 mriedem the admin wants to recreate an instance using a similar flavor
18:54:00 mriedem so they'll want the extra specs too
18:55:24 openstackgerrit Chris Friesen proposed openstack/nova master: fix InvalidSharedStorage exception message https://review.openstack.org/464726
18:56:20 cfriesen sdague: yeah, it would suck if we didn't have extra-specs for the reasons given
18:56:33 sdague ok, no prob
18:56:48 sdague alright, +2
18:57:30 sdague cfriesen: so how do you propose the array of extra specs be documented at the api-ref level?
18:58:15 dansmith oh
18:58:17 dansmith so
18:58:26 dansmith users can't see extra specs on regular flavors by default I guess
18:58:35 dansmith so we'd be exposing them here, which would be bad I guess
18:58:50 dansmith I still think this is almost useless if we don't, we just have to separate that out
18:59:02 cfriesen dansmith: the current proposed code uses the same policy as we have now for displaying extra-specs. if the user doesn't have permission the extra-specs key isn't included in the dict
18:59:08 dansmith and then the answer to the api-ref question is "however it's documented in the flavor api"
18:59:14 dansmith cfriesen: ah, perfect
19:00:08 sdague dansmith: well, it's documented poorly there, but keys showing up in the servers doc typically try to have a higher bar of documentation
19:00:22 dansmith okay
19:00:41 cfriesen sdague: dansmith: the proposed nova change is https://review.openstack.org/#/c/434514/
19:00:52 cfriesen that one doesn't have the rxtx_factor removal though, need to update it
19:01:14 sdague cfriesen: so, you are going to need to break each of those out
19:01:33 sdague I'll leave a comment
19:02:11 cfriesen sdague: in parameters.yaml? I talked about it with mriedem before the summit, we weren't sure how to handle it because each of them is already defined, but we didn't know how to link to them
19:02:54 sdague yeh
19:03:52 mriedem i can confirm that that conversation did in fact happen and i even have tapes of it
19:04:53 sdague ok, the pattern we were working on was https://review.openstack.org/#/c/434514/7/api-ref/source/servers.inc@614
19:08:08 cfriesen sdague: there are three api calls in servers.inc that return this dict...does that mean I need to describe the dict three times?
19:09:06 sdague cfriesen: it needs to be described every time it's part of the response, but that should just be the same block of field refs
19:09:09 cfriesen sdague: and what about the existing flavor dict fields that are already in parameters.yaml? (flavor_server, flavor_swap, etc)
19:09:42 sdague cfriesen: these are going to have min_microversion on them, so they'll need new entries
19:10:44 sdague the parameters.yaml is sort of 3rd normal form for this data, things aren't hyper normalized, means there is sometimes some copy paste, but it means fixes are more straight forward because they don't unrelatedly change something else you weren't expecting
19:11:00 cfriesen okay, will give it a shot
19:14:27 cfriesen sdague: mriedem: dansmith: on a totally different note, what do you think of https://review.openstack.org/464726 ? This is currently mostly an issue if you use horizon since it defaults to specifying "block_migration=False".
19:15:43 sdague cfriesen: i'm fine with that change
19:23:29 mriedem i don't know what there is not to not like about the original wording
19:23:57 dansmith it doesn't sound unfine to me when it's not inapplicable due to zero or less other conditionals
19:45:06 cfriesen thanks guys
20:10:15 openstackgerrit Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636
20:10:16 openstackgerrit Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634
20:10:18 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537
20:10:18 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513
20:10:18 openstackgerrit Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635
20:10:19 openstackgerrit Dan Smith proposed openstack/nova master: Target cell on local delete https://review.openstack.org/464059
21:16:30 mriedem mdbooth: https://review.openstack.org/#/c/443951/ was rebased but isn't failing on your test that checks to see if connect_volume and disconnect_volume are taking an instance parameter
22:18:36 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: expand checks for SubclassSignatureTestCase https://review.openstack.org/464765
22:18:37 mriedem mdbooth: fixed here ^
22:28:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951
23:37:13 gmann alex_xu: m back to work. ll update/follow up on policy doc and extension patches.
#openstack-nova - 2017-05-16
00:33:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
00:33:33 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
00:34:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
00:34:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
00:39:10 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.46 - List/Show all server migration types https://review.openstack.org/430839
00:40:18 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.46 - Enable cold migration with target host https://review.openstack.org/406707
00:41:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
00:41:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
00:41:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
02:41:17 openstackgerrit Zhaokun Fu proposed openstack/nova-specs master: fix typos https://review.openstack.org/464906
03:21:47 openstackgerrit jichenjc proposed openstack/nova-specs master: WIP: add z/VM Driver https://review.openstack.org/464915
03:33:58 openstackgerrit jichenjc proposed openstack/nova-specs master: WIP: add z/VM Driver https://review.openstack.org/464915
05:28:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
06:54:13 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
08:03:12 bauzas p!spy
08:23:53 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Attach and detach encryptors during swap_volume https://review.openstack.org/460244
08:23:53 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243
08:23:58 lyarwood mdbooth: ^ if you have any review time today your thoughts on these would be appreciated
08:51:34 openstackgerrit falseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.openstack.org/464991
09:37:19 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore https://review.openstack.org/460813
09:40:35 openstackgerrit jichenjc proposed openstack/nova master: Add missing rootwrap filter for cryptsetup https://review.openstack.org/462348
09:45:45 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847
09:47:36 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
09:47:44 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830
09:49:13 openstackgerrit jichenjc proposed openstack/nova master: [placement] Add api-ref for /resource_classes/{name} https://review.openstack.org/462837
10:36:14 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Use SubclassSignatureTestCase on all volume drivers https://review.openstack.org/465022
11:20:16 openstackgerrit melanie witt proposed openstack/nova master: WIP Cache database and message queue connection objects https://review.openstack.org/465042
11:56:10 openstackgerrit Monty Taylor proposed openstack/nova master: WIP Add description of server addresses objects https://review.openstack.org/464331
12:18:13 alex_xu gmann: thanks!
13:19:00 openstack Launchpad bug 1579876 in OpenStack Compute (nova) "os-resetState should return a 204 rather than a 202" [Low,Triaged]
13:19:00 alex_xu sdague: johnthetubaguy are we real going to fix something like change status from 202 to 204? like this bug https://bugs.launchpad.net/nova/+bug/1579876. I'm think whether I should mark it as 'won't fix'
13:19:22 sdague alex_xu: right now it seems really low priority to fix
13:20:30 alex_xu sdague: yea, agree that it is super low. and I actually think about we are never going to fix it.
13:20:35 sdague yeh
13:20:48 sdague honestly, Opinion / Wishlist is how I would file the bug
13:21:47 alex_xu sdague: ok, got it
13:31:07 bauzas dansmith: edleafe: sorry for not yet uploading my series, but I discovered some possible issues that I'm not wanting
13:32:02 dansmith ...alright
13:41:13 dansmith sdague: so, setting ulimit so mysql can actually use the number of connections we have configured seems to have made the test pass
13:41:21 dansmith sdague: I'm running again to get another sample
13:46:19 bauzas dansmith: for example, now that we will pass N nodes back to the conductor for rescheduling, we still want to only consume the first one in the scheduler
13:47:57 dansmith okay
14:15:13 openstackgerrit dane-fichter proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711
14:27:20 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
14:28:39 mriedem o/
14:29:08 mriedem someone want to put this veritas libvirt volume driver through so we can close the blueprint? https://review.openstack.org/#/c/443951/
14:33:18 mdbooth mriedem: The connection_info munging in there is evil and undocumented, btw
14:34:09 mriedem mdbooth: i realize the connection_info dict from cinder is a vendor-specific wild west
14:34:36 mriedem i flirted with the idea of trying to formalize that into a versioned object in cinder/os-brick years ago, but never had enough energy to really work on that
14:34:41 mdbooth Right, but in general we erect a 'wild west fence' around it with os-brick, right?
14:34:56 mriedem mdbooth: the same can be said for port binding details in neutron

Earlier   Later