| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-10-26 | |||
| 16:45:49 | kashyap | bauzas: Yes, here | |
| 16:45:51 | bauzas | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-October/025500.html | |
| 16:45:54 | kashyap | bauzas: Sorry for "talking over" earlier | |
| 16:46:08 | bauzas | is this^ more a FYI ? | |
| 16:46:12 | kashyap | Yep | |
| 16:46:26 | kashyap | bauzas: In short: DevStack now defults to "Nehalem" CPU model for reasons outlined there. | |
| 16:46:43 | bauzas | kashyap: devstack/rhel you mean ? | |
| 16:46:47 | kashyap | No | |
| 16:46:50 | kashyap | DevStack upstream | |
| 16:47:06 | kashyap | I worked w/ Clark to get the patch merged in DevStack | |
| 16:47:20 | bauzas | I mean, devstack all distros or only upstream devstack with rhel ? | |
| 16:47:40 | kashyap | For all. | |
| 16:47:44 | bauzas | ack | |
| 16:47:54 | sean-k-mooney | for all on x86 | |
| 16:47:57 | kashyap | Everything is summarized and resolved in that thread. | |
| 16:48:23 | kashyap | Yes, x86 is "implied architecture", when we don't name an arch :) | |
| 16:48:47 | bauzas | okay, do we need further actions, like documenting it ? | |
| 16:48:55 | bauzas | in our relnotes or in the install docs ? | |
| 16:49:07 | sean-k-mooney | no we dont need to document anythign in nova | |
| 16:49:15 | kashyap | Yep; nothing needs documentation in Nova | |
| 16:49:27 | bauzas | my confusion comes from the fact we changed default for all distros, not only rhel | |
| 16:49:27 | sean-k-mooney | rhel have documented there new compilation target already in a blog | |
| 16:49:37 | kashyap | bauzas: No further action; just an FYI here ... if anyone in case notices "why is DevStack giving me Nehalem by default" | |
| 16:49:48 | kashyap | Yep; linked to in the post to the upstream list | |
| 16:50:05 | bauzas | kashyap: little chances that people notice this change by looking at our meeting minutes, honestly | |
| 16:50:24 | bauzas | but okay, noted. | |
| 16:50:31 | kashyap | bauzas: I've notified the Whitebox / CI folks to change their scripts too | |
| 16:50:45 | kashyap | So, the right parties are notified. I'm not expecting anyone to read these minutes here :) | |
| 16:50:54 | sean-k-mooney | bauzas: well honestly its a better default then qemu64 and its so old at this poitn everythign supprots it | |
| 16:51:02 | sean-k-mooney | bauzas: which is basicaly why it was done for all | |
| 16:51:30 | kashyap | bauzas: The change is a better default too. Yes. (Besides solving the switch to a new x86 baseline) | |
| 16:51:33 | sean-k-mooney | the jobs also might get slictly faster as a restult but that would just be a nice benift and not the intent | |
| 16:51:48 | kashyap | Yep; there will be slight improvement | |
| 16:52:15 | bauzas | OK, nice to hear | |
| 16:52:22 | bauzas | are we done ? | |
| 16:52:26 | kashyap | bauzas: That's all. I don't want to take up more time. But those interested / affected in CI/QE/Infra are notified, and I'm working w/ them. | |
| 16:52:31 | kashyap | Yep. | |
| 16:52:40 | bauzas | kashyap: excellent, thanks for providing guidance | |
| 16:52:54 | gibi | kashyap++ | |
| 16:53:08 | bauzas | I think we can reasonably call this meeting a wrap | |
| 16:53:24 | bauzas | I was expecting it to be shorter, but meh, good talks | |
| 16:53:33 | bauzas | #endmeeting | |
| 16:53:33 | opendevmeet | Meeting ended Tue Oct 26 16:53:33 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:53:33 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-10-26-16.00.html | |
| 16:53:33 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-10-26-16.00.txt | |
| 16:53:33 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-10-26-16.00.log.html | |
| 16:53:41 | gibi | bauzas: thanks! | |
| 16:53:55 | bauzas | yeah me too | |
| 16:54:44 | sean-k-mooney | fyi like bauzas i will be taking some pto around the weekend | |
| 16:55:07 | bauzas | I'll be Parisian for the weekend | |
| 16:55:22 | bauzas | although I'm already grumbling for nothing. | |
| 16:55:32 | sean-k-mooney | i need to book it still but firday and monday for sure and maybe ill exened it from thursday-turuseday inclusive | |
| 16:55:45 | bauzas | sean-k-mooney: ack, noticed. | |
| 16:56:08 | sean-k-mooney | thanks for doing the PTG summary by the way | |
| 16:56:21 | sean-k-mooney | i have not read it yet but ill try and skim over it later | |
| 16:57:29 | elodilles | sean-k-mooney: ping we if you updated the libvirt/os-vif related patches that you mentioned for the ussuri release, and I'll try to review them ASAP | |
| 16:58:31 | sean-k-mooney | elodilles: i need to redo it for the newer reelase first | |
| 16:59:17 | sean-k-mooney | elodilles: https://review.opendev.org/c/openstack/nova/+/790447 is missing one of the 3 patches that were ment to be squashed together | |
| 16:59:45 | elodilles | sean-k-mooney: ack | |
| 16:59:55 | sean-k-mooney | so i need to fix that one first and then propose it we have had some asks to backport this downstream recently too so i need to fix these up | |
| 17:00:11 | sean-k-mooney | if it end up missing the final release its proably ok but would be nice to include | |
| 17:00:20 | elodilles | hmm, so it needs to be merged in wallaby then victoria then ussuri :S | |
| 17:00:37 | sean-k-mooney | yep which is why im not sure it will make it in 2 weeks | |
| 17:00:53 | sean-k-mooney | so this is really just a nice to have not something to hold the release for | |
| 17:01:37 | elodilles | OK, no problem, let's try then. just ping me and I'll review them :) | |
| 17:01:47 | sean-k-mooney | ack thanks | |
| 17:38:54 | gmann | gibi: 1 comment on this otherwise lgtm https://review.opendev.org/c/openstack/tempest/+/809168 | |
| 17:40:21 | gmann | gibi: sorry for not mentioning about that method to use in my previous comment | |
| 18:33:32 | sean-k-mooney | gmann: i know this is a little out of your normal review spcace but feel like taking a look at https://review.opendev.org/c/openstack/nova/+/811947 | |
| 18:34:16 | sean-k-mooney | gmann: ill ping some other folks tomorow if you dont have time to look but it would be nice to get the last +2 on that | |
| 18:34:25 | sean-k-mooney | anyway going to call it a night o/ | |
| 19:24:41 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 20:22:56 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 20:31:30 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with free resources first https://review.opendev.org/c/openstack/nova/+/805649 | |
| #openstack-nova - 2021-10-27 | |||
| 08:56:40 | gibi | good day nova | |
| 09:17:47 | bauzas | hola folks | |
| 09:20:41 | gibi | o/ | |
| 09:20:54 | gibi | -another day in downstream land- | |
| 09:34:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/813419 | |
| 10:58:44 | gibi | stephenfin: do you recall why you needed to configure both db in a single Database fixture in https://review.opendev.org/c/openstack/nova/+/799526/5/nova/tests/fixtures/nova.py#612 ? | |
| 10:59:29 | gibi | I think our tests are using a separate fixture instance for each db (api, main) | |
| 11:00:18 | stephenfin | gibi: I'm not 100% sure but my guess is that I don't, and that was simply for expediency/laziness :) If SESSION_CONFIGURED became a mapping of DB type to "is configured" bool, we probably wouldn't need that | |
| 11:00:28 | stephenfin | *we don't | |
| 11:01:00 | gibi | I see so we had a single global but with two dbs to configure | |
| 11:01:07 | stephenfin | Yeah, I think so | |
| 11:02:48 | gibi | OK, if that is the only reason then I think I have a way to remove that global flag (based on melwitt's idea) with patch_factory from oslo_db | |
| 11:03:15 | gibi | it is no pretty confusing that we have to Database fixture intantiated one for main and one for api but the first one configures both db | |
| 11:03:38 | gibi | s/no/now | |
| 11:03:47 | gibi | /to/two | |
| 11:03:50 | gibi | /o\ | |
| 11:04:20 | stephenfin | yeah, tbc it could be more complicated than that but I really doubt it | |
| 11:08:07 | gibi | yeah, lets see if my idea works | |
| 11:12:21 | sean-k-mooney | stephenfin: since your about here an easy one for you https://review.opendev.org/c/openstack/nova/+/811947 think we can get that over the line | |
| 11:13:43 | stephenfin | sure, will look now | |
| 11:14:05 | sean-k-mooney | thanks :) | |
| 11:19:44 | sean-k-mooney | stephenfin: based on the ptg discussion woudl you mind removing your -2 on https://review.opendev.org/c/openstack/nova/+/804292 im going to rebase that and the autopep8 one shortly | |
| 13:05:29 | frickler | kashyap: I didn't make progress with reproduction without nova yet, so I created https://gitlab.com/qemu-project/qemu/-/issues/693 for now. let me know if you need additional data there | |
| 13:08:38 | kashyap | frickler: Thanks for the report. A quick one is: were you using nested setup, or was this DevStack instance on a baremetal host (<shudder>)? | |
| 13:09:07 | kashyap | A thumb-rule is to always explicitly state so if you're using a nested setup | |
| 13:09:47 | kashyap | frickler: Can you edit the report to state that "deploy DevStack in a VM?" So that an unsuspecting dev won't run it on their baremetal laptop and wreak havoc... | |