| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-08 | |||
| 10:08:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Improve PDF document structure https://review.opendev.org/682746 | |
| 10:10:08 | stephenfin | bauzas: Could I ask you to push through a few trivial patches for me, please? I've a lot of stuff in flight /o\ | |
| 10:11:04 | stephenfin | If you could hit https://review.opendev.org/686211 https://review.opendev.org/686901 https://review.opendev.org/684774 and https://review.opendev.org/#/c/684775/ that's be great | |
| 10:11:19 | stephenfin | https://imgflip.com/s/meme/That-Would-Be-Great.jpg | |
| 10:17:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Take an instance.uuid lock when rebooting https://review.opendev.org/673463 | |
| 10:38:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779 | |
| 10:39:53 | gibi | efried: fixed your comments in ^^ | |
| 10:42:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove 'adv-config', 'system-admin' subdocs https://review.opendev.org/684402 | |
| 10:42:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Replacing underscores with dashes https://review.opendev.org/685929 | |
| 10:42:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Strip '.rst' suffix https://review.opendev.org/687264 | |
| 10:45:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove @safe_connect from put_allocations https://review.opendev.org/686579 | |
| 10:55:23 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: refactor volume bdm rollback error handling https://review.opendev.org/656500 | |
| 10:55:24 | lyarwood | stephenfin: ^ finally updated that if you have time this wekk. | |
| 10:55:29 | lyarwood | week* | |
| 11:20:36 | sean-k-mooney | stephenfin: dumb question but how to i add blueprint to the nova team meeting agenda again. there is a wiki page i think right? | |
| 11:20:55 | sean-k-mooney | https://wiki.openstack.org/wiki/Meetings/Nova? | |
| 11:23:22 | sean-k-mooney | im not sure if that is the agenda for last weeks meetin or thi seeks meting | |
| 11:29:06 | gibi | sean-k-mooney: add your stuff to that agenda with a timestamp so when efried goes and updates the agenda for the next meeting he will see that your item is for the upcoming meeting | |
| 11:29:59 | sean-k-mooney | thanks | |
| 11:33:34 | sean-k-mooney | gibi: i bacically want to know if https://blueprints.launchpad.net/nova/+spec/inplace-rebuild-of-numa-instances can proceed as a specless blueprint or if i should write a spec | |
| 11:33:57 | sean-k-mooney | but i have updated the agenda with the timestamp as you suggesteded | |
| 11:36:09 | gibi | sean-k-mooney: after reading the bp, for me it seems OK to have this as specless as it is only fixing bugs | |
| 11:42:28 | sean-k-mooney | gibi: that is what i was hoping. do you think it makes sense to group them as a blueprint as i have | |
| 11:43:29 | sean-k-mooney | i intend to fix the bug as seperate patches in the order i suggested (prevent broken behavior then skip filter if safe) | |
| 11:51:53 | gibi | sean-k-mooney: it is easier to track multiple bugs as a single bp so I agree with your syle | |
| 11:51:57 | gibi | style | |
| 11:52:38 | kashyap | stephenfin: Just noticed that (WIP) change on skipping CPU comparison check on AArch64. Will comment there. | |
| 11:52:54 | sean-k-mooney | cool i also just fixed all the spelling issue so it should be eaiser for others to parse | |
| 11:54:48 | gibi | sean-k-mooney: thanks for the spelling fix :) | |
| 11:57:12 | sean-k-mooney | i am trying to get back in to using grammerly more at least of non irc communications | |
| 12:14:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refine comments about move_allocations https://review.opendev.org/686182 | |
| 12:14:40 | openstackgerrit | Merged openstack/nova master: Update contributor guide for Ussuri https://review.opendev.org/685630 | |
| 12:40:30 | efried | gibi: +2, thanks for the updates | |
| 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. | |