| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-25 | |||
| 13:12:04 | sdague | alex_xu: did we ever end up with the swagger docs for nova committed anywhere? to show as an example? | |
| 13:12:21 | johnthetubaguy | mriedem: vladikr: I am really just waiting on reviews on that second one | |
| 13:13:01 | vladikr | johnthetubaguy, ah, sure. I just saw it on the list, | |
| 13:13:24 | vladikr | mriedem, cool, I'll continue with the first one then | |
| 13:13:25 | johnthetubaguy | vladikr: I believe the code was up for the live-migrated-rescued-instances one, the original paul murray patch was tidied up | |
| 13:13:49 | mriedem | mtreinish: looks like you want to own https://blueprints.launchpad.net/nova/+spec/remove-nova-cert now? | |
| 13:14:00 | mriedem | paul carlton | |
| 13:14:10 | johnthetubaguy | mriedem: you are right, my pad | |
| 13:14:14 | johnthetubaguy | bad | |
| 13:14:25 | mriedem | british HPE guys named paul are a dime a dozen | |
| 13:14:35 | johnthetubaguy | mriedem: was looking a policy this morning, made a lis on the spec | |
| 13:14:40 | johnthetubaguy | mriedem: apparently | |
| 13:14:51 | vladikr | ok, I see ... | |
| 13:15:11 | mriedem | vladikr: i've assigned https://blueprints.launchpad.net/nova/+spec/live-migrate-rescued-instances to you | |
| 13:15:12 | mriedem | thanks | |
| 13:15:21 | vladikr | mriedem, thanks | |
| 13:15:22 | mriedem | there is also a tempest test to test the series, but that probably also needs updating | |
| 13:15:31 | vladikr | ok | |
| 13:15:34 | johnthetubaguy | yeah, that needs work I suspect | |
| 13:16:07 | vladikr | I'll take a look. I haven't seen what has been done so far | |
| 13:18:10 | sfinucan | mriedem: Could you look at [1] at some point. It's low priority, but I want to get it in now so we can stick to a six month (/one release) deprecation cycle [1] https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/move-nova-cmds-to-cliff | |
| 13:19:21 | mriedem | sfinucan: i'll start the series, but probably can't start on it this morning | |
| 13:19:24 | mriedem | *star | |
| 13:19:47 | sfinucan | mriedem: Fine by me. Once it's looked at some time in the next week or two | |
| 13:19:49 | sfinucan | ta :) | |
| 13:20:03 | johnthetubaguy | mriedem: I made a list of the policy files that still need fixing up: https://blueprints.launchpad.net/nova/+spec/policy-docs | |
| 13:20:42 | johnthetubaguy | mriedem: in the process I implemented https://blueprints.launchpad.net/nova/+spec/remove-discoverable-policy-rules | |
| 13:23:31 | mriedem | johnthetubaguy: ah ok | |
| 13:24:56 | mriedem | dane-fichter: with AES and aes be handled the same way in castellan? | |
| 13:25:54 | dane-fichter | mriedem: I believe there's code in Castellan to handle that. Hold on, let me check. | |
| 13:28:44 | bauzas | mriedem: oh, I missed that bug cleaning day :( | |
| 13:29:17 | bauzas | mriedem: is macsz not impacted by the OSIC issue ? | |
| 13:29:24 | mriedem | bauzas: he is | |
| 13:30:22 | mriedem | dane-fichter: if using barbican it's just passed through https://github.com/openstack/castellan/blob/master/castellan/key_manager/barbican_key_manager.py#L224 | |
| 13:31:17 | dane-fichter | mriedem: Yeah it looks like all the unit tests use 'AES' | |
| 13:31:30 | alex_xu | sdague: this is a poc https://review.openstack.org/#/c/233446/ | |
| 13:31:50 | sdague | alex_xu: yeh, I found some files checked in on fairy-slipper | |
| 13:32:14 | sdague | I was trying to find the x-actions bit | |
| 13:32:21 | sdague | though, that review is a good reminder | |
| 13:32:32 | dane-fichter | mriedem: I'm asking Kaitlin Farr about this. I recall some code handling this aes/AES issue | |
| 13:33:27 | mriedem | dane-fichter: ok | |
| 13:33:34 | mriedem | i left questions in the review | |
| 13:33:55 | dane-fichter | mriedem: Thanks for taking a look, I'll follow-up on this and get back to you | |
| 13:35:48 | alex_xu | sdague: yea, probably fairy-slipper have something related to action. I don't remember we discuss the action in the poc or nova-spec, since we say no the extended swagger in early stage | |
| 13:36:46 | sdague | alex_xu: yeh, I was just trying to dig up some post mortem on the effort for my talk about solving big issues in OpenStack | |
| 13:37:47 | mriedem | sdague: johnthetubaguy: would one of you like to play god and pick the first microversion for pike? https://review.openstack.org/#/c/456504/ https://review.openstack.org/#/c/457181/ | |
| 13:37:52 | alex_xu | sdague: you mean a talk in summit or somewhere? | |
| 13:39:17 | sdague | alex_xu: summit | |
| 13:40:14 | alex_xu | sdague: cool, i'm looking aforward to seeing the video :) | |
| 13:43:52 | alex_xu | s/aforward/forward/ | |
| 13:47:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: move createBackup to server-actions https://review.openstack.org/452971 | |
| 13:47:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: 2.43: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443 | |
| 13:48:22 | mriedem | alex_xu: ^ is unchanged except for adding the 2.43 response examples to the api-ref as gmann pointd out | |
| 13:48:25 | mriedem | *pointed | |
| 13:48:52 | alex_xu | mriedem: got it, that is good point | |
| 13:51:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 13:53:55 | sdague | mriedem: I picked os-hosts | |
| 13:54:04 | alex_xu | mriedem: done | |
| 13:54:05 | sdague | alex_xu: you want to rev https://review.openstack.org/#/c/457181 to go on top of it ? | |
| 13:54:14 | mriedem | sdague: excellent choice | |
| 13:54:15 | alex_xu | sdague: yea, sure | |
| 13:54:37 | alex_xu | sdague: you need another chooice between my one and mriedem one :) | |
| 13:54:47 | sdague | alex_xu: oh, what's the next choice? | |
| 13:54:54 | mriedem | https://review.openstack.org/#/c/455443/ | |
| 13:54:59 | mriedem | but i'll have to rebase too | |
| 13:55:22 | sdague | so, just based on initial submit time, I'd take alex_xu's first | |
| 13:55:29 | sdague | and mriedem you stack up after tat | |
| 13:55:30 | sdague | that | |
| 13:56:03 | mriedem | ok | |
| 13:56:05 | alex_xu | ok :) | |
| 13:56:39 | sdague | so, given rebasing all those like that, we could probably get them both merged by tomorrow | |
| 13:57:43 | mriedem | alex_xu: i'll rebase mine once yours is up | |
| 13:57:49 | alex_xu | mriedem: thanks | |
| 14:02:45 | mriedem | sdague: i see our functional tests fail with this requests stuff a lot lately, is there some retry magic we should be building in? http://logs.openstack.org/20/459420/1/check/gate-nova-tox-functional-ubuntu-xenial/c0bdfa6/console.html#_2017-04-25_13_55_17_705409 | |
| 14:02:49 | mriedem | to the functional test api client? | |
| 14:07:25 | sdague | mriedem: hmmm... that's weird | |
| 14:07:45 | sdague | mriedem: so that whole BadStatusLine remains a mystery to me | |
| 14:08:08 | sdague | I wonder if it's further eventlet fallout? | |
| 14:08:14 | sdague | mriedem: how frequent is it? | |
| 14:08:21 | cdent | sdague: I seem to recall that having shown up in the past as an artifact of weird eventlet collisions | |
| 14:08:23 | mriedem | 11 hits in 7 days on that job | |
| 14:08:28 | cdent | but I may be remembering something else | |
| 14:08:55 | sdague | I remember looking at it in the gate, and assuming it was load / proxy related | |
| 14:12:26 | openstackgerrit | Alex Xu proposed openstack/nova master: Deprecate Multinic, floatingip action and os-virtual-interface API https://review.openstack.org/457181 | |
| 14:13:24 | alex_xu | mriedem: sdague ^ done | |
| 14:13:35 | mriedem | alex_xu: thanks | |
| 14:13:42 | alex_xu | np | |
| 14:13:46 | jaypipes | alex_xu: pls see my request on https://review.openstack.org/#/c/445864. If you can push that change quickly, I can re-review and +2/W it with johnthetubaguy. | |
| 14:14:01 | openstack | Launchpad bug 1528743 in OpenStack Compute (nova) "HostState in Scheduler can be incorrect" [Undecided,In progress] - Assigned to Zhenyu Zheng (zhengzhenyu) | |
| 14:14:01 | mriedem | bauzas: you might be interested in the oldest stale in progress bug (which isn't actually in progress): https://bugs.launchpad.net/nova/+bug/1528743 | |
| 14:14:13 | cdent | sdague: from some reading just now it appears to happen when either the server side has closed the connection at the exact time the client is trying to read the status line, or is sending a response (because it is evil) without a status line | |
| 14:14:33 | bauzas | mriedem: was absorbed into a team meeting, but will do as soon as it finishes | |
| 14:14:34 | alex_xu | jaypipes: ah, thanks, I will push that change quickly | |
| 14:15:04 | cdent | sdague: TLS isn't invovled in these is it? | |
| 14:15:59 | mriedem | cdent: for the functional tests it shouldn't be | |
| 14:16:40 | cdent | do these functional tests spawn off a different process for the api server, or do it in process somehow? | |
| 14:17:29 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrate_server.py https://review.openstack.org/459711 | |
| 14:18:02 | mriedem | cdent: they use OSAPIFixture which says "This spawns an OS API server as a fixture in a new greenthread in the current test." | |
| 14:18:20 | cdent | ah, then yeah, it could very well be eventlet | |
| 14:18:35 | mtreinish | mriedem: yeah, it seemed simple enough. Assuming I can figure out how to make the api-ref job pass | |