| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-08 | |||
| 12:40:43 | gibi | efried: thanks for the review | |
| 12:40:58 | efried | sean-k-mooney: ack meeting agenda | |
| 12:43:54 | efried | alex_xu: Is there more that needs to be done with https://review.opendev.org/#/c/685180/ (test case for unauthed versioned discovery) and its successor (the fix for same)? | |
| 13:02:34 | bauzas | stephenfin: dammit, just saw your courtesy ping now :) | |
| 13:02:52 | bauzas | stephenfin: that's what happens when you ping people when they are on gym :) | |
| 13:03:56 | bauzas | and FWIW, just clicked on the 4 changes :) | |
| 13:12:03 | mriedem | efried: +2 on https://review.opendev.org/#/c/686414/ now, just let dansmith know when you want to pull the trigger for rc2 | |
| 13:23:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: link to user/index from main home page https://review.opendev.org/687292 | |
| 13:24:02 | bauzas | I wonder, does the US already shifted from summer times ? | |
| 13:24:15 | bauzas | (that's in general 2 weeks before Europe hence my wonders) | |
| 13:24:25 | mriedem | not yet | |
| 13:24:38 | mriedem | november 3 | |
| 13:24:47 | bauzas | ack ok | |
| 13:25:17 | bauzas | then Europe is one week *before* this time | |
| 13:25:34 | bauzas | efried: left a comment on https://review.opendev.org/#/c/685857/ | |
| 13:26:12 | bauzas | efried: tl;dr while I'm basically OK with the idea, I just want to clarify the fact that we can't *guarantee* mandatory things | |
| 13:26:20 | bauzas | it's more or less best effort | |
| 13:26:36 | bauzas | although I'm trying hard to stick with this... | |
| 13:33:21 | efried | bauzas: Appreciate the reviews, responding now. We can't guarantee anything in this community, as you well know. I can try to make that clearer I suppose. | |
| 13:33:39 | bauzas | efried: yeah, I know you know | |
| 13:33:50 | bauzas | efried: I just want to make it clear by the documentation, that's it | |
| 13:33:55 | efried | ++ | |
| 13:34:57 | efried | aspiers: We're about to pull the trigger on the "SEV limitations" documentation for Train so we can get it into RC2. Unless the fix is, like, completely ready to go right now and you're just about to push it... | |
| 13:35:43 | sean-k-mooney | bauzas: isnt the eu ment to be stoping daylight saving time this year | |
| 13:36:10 | sean-k-mooney | e.g. the clocks will not change next summer | |
| 13:37:15 | bauzas | sean-k-mooney: well, that will be different between countries AFAICU | |
| 13:37:33 | stephenfin | efried: Think you could look at https://review.opendev.org/#/c/682746/ today, before it goes into merge conflict again? | |
| 13:37:40 | sean-k-mooney | bauzas: each contry will pick to keep either standard time or daylight savings time | |
| 13:37:49 | bauzas | yup this | |
| 13:37:53 | sean-k-mooney | but across the eu it will nolnger change | |
| 13:38:18 | bauzas | but for example, France and Germany would no longer have same times | |
| 13:38:19 | sean-k-mooney | also if the UK dont pick GMT/UTC i will just give up | |
| 13:42:03 | stephenfin | bauzas: Cool, thanks :) | |
| 13:43:11 | efried | stephenfin: looking | |
| 13:43:54 | gibi | mriedem: I have a comment about qos port support in https://review.opendev.org/#/c/633293/49/nova/compute/manager.py@4742 | |
| 13:47:36 | mriedem | gibi: tl;dr add request_spec to the prep_snapshot_based_resize_at_dest method yes? | |
| 13:47:43 | gibi | mriedem: yest | |
| 13:47:51 | mriedem | and finish_snapshot_based_resize_at_dest | |
| 13:47:55 | gibi | mriedem: and for the finish_snapshot_based_resize | |
| 13:47:58 | gibi | yepp | |
| 13:48:05 | mriedem | ok | |
| 13:49:05 | gibi | mriedem: and leave a todo in https://review.opendev.org/#/c/633293/49/nova/compute/manager.py to call _update_pci_request_spec_with_allocated_interface_name() to support qos ports | |
| 13:49:21 | gibi | qos sriov ports | |
| 13:51:37 | gibi | mriedem: btw efried is +2 on the ussuri qos spec so I think it is ready for a +A https://review.opendev.org/#/c/686779/ | |
| 13:54:51 | mriedem | yeah, i wasn't planning on implementing the qos port support in the first round so i'm glad you're ok with todos | |
| 13:55:01 | mriedem | because it's hard enough getting this off the ground with just basic use cases | |
| 13:55:39 | gibi | mriedem: I'm totally OK with TODOs. I can even volunteer to help fixing those TODOs at some point | |
| 13:55:59 | gibi | mriedem: but I want to avoid an extra RPC bump if possible | |
| 13:57:23 | mriedem | .yeah | |
| 13:57:27 | mriedem | +W on the qos move spec | |
| 13:57:39 | gibi | mriedem: thanks | |
| 14:04:28 | alex_xu | efried: nothing more, I just want to pick a time go through the patch again | |
| 14:05:02 | efried | alex_xu: okay, thanks. Any idea who we can get to be a second +2 there? I haven't seen johnthetubaguy in a while... | |
| 14:05:42 | alex_xu | efried: good question, and we just lost ken'ichi | |
| 14:06:09 | alex_xu | efried: maybe cdent and gmann can give a review on it | |
| 14:08:24 | efried | stephenfin: +A doc restructure patch | |
| 14:09:00 | cdent | alex_xu: which? | |
| 14:09:17 | alex_xu | cdent: this one https://review.opendev.org/685181 | |
| 14:09:25 | cdent | thanks | |
| 14:09:36 | efried | cdent: you +1ed an earlier version of the test patch underneath it | |
| 14:09:53 | efried | And I was looking for your help on the fix itself before you went on vacay | |
| 14:10:00 | stephenfin | efried: Thanks (y) | |
| 14:10:22 | cdent | yeah, I've gone back to that a few different times, wandered around vaguely over the paste.ini stuff, got a bit sad about that, and then got distracted again. will gaze again now | |
| 14:10:30 | gmann | alex_xu: efried sure, I will check that | |
| 14:11:09 | efried | cdent: yeah, the paste stuff is pretty arcane, but it turns out we're handling the URL mapping bit in nova, so I was able to hack into it a bit. | |
| 14:11:24 | efried | thanks gmann | |
| 14:11:51 | efried | If this flies, there are other projects with the same problem that may need to take similar approaches. | |
| 14:11:57 | openstackgerrit | Merged openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779 | |
| 14:11:59 | efried | neutron and I think cinder | |
| 14:12:40 | cdent | during the future times of tech debt cleanup we should consider adding a wsgi entry point that doesn't use paste.ini (while keeping (perhaps deprecating) for a while the paste way) | |
| 14:12:59 | efried | cdent: I talked with lbragstad about how they cut keystone over to flask | |
| 14:13:03 | cdent | I say this as the maintainer of pastedeploy paste and pastescript: I am not actually maintaining it | |
| 14:13:08 | efried | took them a couple releases | |
| 14:13:17 | efried | but... yes. | |
| 14:14:43 | lbragstad | we piecemeal'd it because we knew we weren't going to be able to make the complete transition in a single release (but YMMV) | |
| 14:15:23 | cdent | switchign to flask would be awesome, but there are smaller steps to take if getting rid of paste.ini is the intermerdiate goal | |
| 14:15:28 | efried | dansmith: Please merge the "sev busted docs" patch at 9am your time. That's 5pm for aspiers I think, a reasonable approximation of EOD. | |
| 14:16:13 | efried | cdent: intermediate steps that would actually move the ball? Or just replace one icky thing with another icky thing? | |
| 14:16:14 | dansmith | efried: ack | |
| 14:16:21 | lbragstad | cdent yeah - i'd be curious to know how nova tries to break that work up | |
| 14:16:57 | cdent | efried: actually move the ball. all paste.ini does it assemble middleware in a configurable way. If you choose to make it not configurable, you can do the same thing in code, like what placement/deploy.py does | |
| 14:17:15 | cdent | the interior apps remain the same | |
| 14:17:26 | efried | that makes sense. Do we have any idea whether deployments count on the configurability? | |
| 14:17:34 | lbragstad | we implemented some basic plumbing that introduced flask and incrementally moved APIs over from the old paste routers to using flask | |
| 14:17:59 | efried | like, do deployments inject security sniffers or whatever? | |
| 14:18:23 | cdent | a few years ago there was some discussion of paste.ini being an official config file, but I don't know if any hard numbers were available. that's why I was suggesting we have two diferent entry points for some duration | |
| 14:18:52 | efried | oic, with a conf switch that decides which one you're using? | |
| 14:19:17 | mriedem | efried: there are deployments that inject their own middleware yes | |
| 14:19:24 | efried | cdent: community goal: "stop using paste"? | |
| 14:19:53 | efried | btw, why is it called "paste"? | |
| 14:19:56 | cdent | that will go as well as stop using wsme (which is even less maintained) | |
| 14:20:01 | mriedem | one example i know of is a middleware from inap (i think?) that translated a request to like a volume-backed flavor | |
| 14:20:11 | cdent | it's pastes things (wsgis) together | |
| 14:20:25 | cdent | but it doesn't fix my typing | |
| 14:21:21 | dansmith | I think there was also another thing called paste that did a similar thing that this was modeled after, IIRC | |
| 14:21:25 | dansmith | it's been a loong time | |
| 14:21:51 | dansmith | or maybe a "paste function" in some other middleware stack | |
| 14:22:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pull up compute node queries to init_host https://review.opendev.org/682680 | |
| 14:22:05 | cdent | or zope | |
| 14:22:19 | dansmith | I think removing it would be a big deal, meaning removing a big plug point for people | |