| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-25 | |||
| 13:01:39 | mriedem | it's national bug spring cleaning day | |
| 13:01:44 | mriedem | international i should say | |
| 13:02:19 | mriedem | bauzas: sfinucan: can we get this in to unblock using the latest os-brick? https://review.openstack.org/#/c/459420/ | |
| 13:02:48 | bauzas | mriedem: wait, what ? international bug sprint cleaning day ? orly? | |
| 13:02:52 | bauzas | mriedem: looking | |
| 13:04:05 | bauzas | mriedem: done | |
| 13:04:10 | bauzas | sfinucan: ^ | |
| 13:04:27 | dane-fichter | mriedem: I've got a bug that's breaking ephemeral storage encryption | |
| 13:04:32 | dane-fichter | mriedem: https://review.openstack.org/#/c/446072/ | |
| 13:04:35 | mriedem | bauzas: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115704.html | |
| 13:04:37 | sfinucan | mriedem: likewise | |
| 13:04:46 | dane-fichter | ^ that patch has been sitting for a while | |
| 13:10:05 | vladikr | https://blueprints.launchpad.net/nova/+spec/prep-for-network-aware-scheduling-pike | |
| 13:10:05 | vladikr | mriedem, hey, for some reason my emails to the dev list are being rejected .. but if no one else is interested, I could continue working on these two: https://blueprints.launchpad.net/nova/+spec/live-migrate-rescued-instances | |
| 13:11:17 | mriedem | vladikr: i'm good with you taking the live migrate rescued instances one, you might want to talk to johnthetubaguy about the network-aware scheduling one | |
| 13:12:02 | johnthetubaguy | yeah, I have a lot of the patches up for that second one | |
| 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 | |