| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-04 | |||
| 16:51:56 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-04-16.00.log.html | |
| 16:51:56 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-04-16.00.txt | |
| 16:51:56 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-04-16.00.html | |
| 16:51:56 | opendevmeet | Meeting ended Tue Apr 4 16:51:56 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:51:56 | bauzas | #endmeeting | |
| 16:52:15 | elodilles | thanks o/ | |
| 16:53:47 | bauzas | that's also it for me today | |
| 16:53:50 | bauzas | see ya folks | |
| 16:54:21 | bauzas | for the fun, I want to test virtiofs on my laptop with a windows guest :) | |
| 16:57:31 | bauzas | waaah that works with winfcp | |
| 17:01:45 | dansmith | gibi: sean-k-mooney: can ya'll hit this at some point: https://review.opendev.org/c/openstack/nova/+/878238 | |
| 17:01:55 | dansmith | related to a recent conversation we had | |
| 17:04:16 | gibi | dansmith: added to my list | |
| 17:04:25 | dansmith | thanks | |
| 17:42:38 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id column to instances table https://review.opendev.org/c/openstack/nova/+/879499 | |
| 17:42:39 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 17:45:47 | dansmith | anyone else having trouble with the fast8 target? it's complaining about python not being in the list of allowed externals, presumably because of the install_command override | |
| 17:46:06 | dansmith | adding it causes it to fail install because it tries to install nova to my /usr/local instead of the venv | |
| 17:46:37 | dansmith | other targets don't seem to have any problems | |
| 17:48:11 | clarkb | python shouldn't be in externals because it is in the venv | |
| 17:48:25 | dansmith | I know that's how it should work | |
| 17:50:02 | dansmith | okay I blew away .tox and it may be working | |
| 17:50:35 | dansmith | I cleaned a(n apparently very old) .tox/fast8 before and it didn't fix it, but I think because flake8 now uses .tox/shared | |
| 17:50:41 | dansmith | so removing that seems to have fixed it up | |
| 17:51:05 | dansmith | I dunno how it got confused about that, but it was the python pip install on the nova package that it was failing with that error | |
| 17:51:14 | dansmith | so maybe something to do with me recently upgrading to py311 on my system | |
| 17:51:28 | dansmith | yeah, that worked | |
| 17:54:13 | dansmith | the pep8 target worked, but I guess it doesn't use the shared venv so I guess it got rebuilt and the one fast8 uses didn't trigger it or soemthing | |
| 17:59:34 | sean-k-mooney | it could be related to the fact we now dont set useddevelop | |
| 17:59:37 | sean-k-mooney | or what ever | |
| 18:01:20 | dansmith | we still do for all the regular venvs, AFAICT | |
| 18:01:44 | dansmith | I think it's more likely a stale bin/python link in that venv pointing to the old python3.10 on my system or something | |
| 18:01:52 | sean-k-mooney | as in i cant fine the sepcific thing i was likign for | |
| 18:02:02 | sean-k-mooney | it might be related to this https://github.com/openstack/nova/commit/5e5b6751d32aa50dc372c55b5c2044d0dade51f4 | |
| 18:02:13 | sean-k-mooney | but i toug we change how ew used sdigts | |
| 18:02:32 | sean-k-mooney | i tought we did somethign for tox 4 | |
| 18:02:39 | dansmith | yeah, that added isntall_command which was the problem | |
| 18:03:54 | sean-k-mooney | ok ya that was the tox 4 change https://review.opendev.org/c/openstack/nova/+/869545 | |
| 18:04:12 | sean-k-mooney | i tought thact actully bumped the min version to 4 but it did not | |
| 18:04:16 | sean-k-mooney | just made it compatible | |
| 18:55:01 | sean-k-mooney | dansmith: im going to call it a day but this is the youtube link i added to the vdpa spec https://www.youtube.com/watch?v=zyDSUU0TLB4 its not that long but it answered some of my questions | |
| 18:55:22 | dansmith | I saw the link but haven't watched it | |
| 18:55:24 | dansmith | cool | |
| 18:56:21 | sean-k-mooney | i think we are going to want two flags hw_disk_bus=vdpa-blk and then a second flag for if it requires hardware offload | |
| 18:56:48 | sean-k-mooney | if we want to supprot the software and hardware backends eventually | |
| 18:57:20 | dansmith | hmm okay I guess I don't know why we'd need two flags, but I'm sure it's ignorance.. I'll watch, we can chat tomorrow or later | |
| 18:57:55 | sean-k-mooney | well its vdpa-blk in the guest in either caes but ya watch the vid and we can chat about it whenever | |
| 18:57:57 | sean-k-mooney | o/ | |
| 18:58:04 | dansmith | o/ | |
| 19:34:30 | opendevreview | Merged openstack/nova stable/yoga: db: Resolve additional SAWarning warnings https://review.opendev.org/c/openstack/nova/+/874065 | |
| #openstack-nova - 2023-04-05 | |||
| 09:09:49 | opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866155 | |
| 09:10:41 | opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | |
| 09:17:09 | bauzas | sean-k-mooney: could you please look at this small PTL doc change ? https://review.opendev.org/c/openstack/nova/+/875730 | |
| 09:46:23 | sean-k-mooney | bauzas: sure ill review it now. for what its worth i never really found the nova wiki useful | |
| 09:46:54 | sean-k-mooney | so i prefer having this in the main schdule | |
| 09:48:07 | sean-k-mooney | i would almost prefer to remove the wiki entirly and move the meeting to etherpad instead but thats a seperate topic | |
| 09:49:24 | sean-k-mooney | * Run the `count-blueprints`__ script daily to gather data for the cycle | |
| 09:49:25 | sean-k-mooney | burndown chart | |
| 09:49:42 | sean-k-mooney | do you really think that has value ? | |
| 09:50:00 | sean-k-mooney | i really dont | |
| 09:54:54 | sean-k-mooney | bauzas: soft -1 i have two questions inline | |
| 09:55:07 | sean-k-mooney | over all the update looks good | |
| 10:17:09 | bauzas | sean-k-mooney: sorry I was afk as I need to taxi my daughter | |
| 10:18:02 | opendevreview | Merged openstack/nova stable/xena: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866155 | |
| 12:22:29 | opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | |
| 13:40:02 | dansmith | bauzas: gmann: can we enable functional-py311 in our tox? I submitted a patch yesterday I thought was passing functional, because when I ran it locally I got a passing tox run (of unit tests) | |
| 13:40:21 | dansmith | because of the silly tox behavior where it will run any missing testenv | |
| 14:14:09 | bauzas | dansmith: hem, yeah, I guess we can now support py 3.11 by the TC ? | |
| 14:14:36 | dansmith | I don't think we've made that call yet, no, but I don't see why we can't just have our tox not broken for people with 3.11-based dev machines | |
| 14:20:32 | dansmith | bauzas: ^ | |
| 14:21:33 | bauzas | dansmith: because you run tox -efunctional by default ? | |
| 14:22:05 | dansmith | bauzas: no, because if you run tox -efunctional-py311 right now, it will say "huh, there's no such testenv for that, so I'll just run the base one under that name and not say anything" ... which runs unit :) | |
| 14:22:07 | bauzas | sorry, I need to understand the problem | |
| 14:22:15 | bauzas | dansmith: hah, ok | |
| 14:22:15 | dansmith | tox has always done | |
| 14:22:16 | dansmith | that | |
| 14:22:22 | dansmith | tox -enot-a-real-thing will pass | |
| 14:22:48 | bauzas | dansmith: yeah because you pin the python version when calling the target | |
| 14:22:53 | dansmith | so I just want to put up a tox.ini modification to catch 311 as well | |
| 14:23:04 | dansmith | not really related to the python version | |
| 14:23:18 | dansmith | well, it is in the sense that functional-310 won't work | |
| 14:23:20 | dansmith | maybe that's what you mean | |
| 14:23:20 | bauzas | dansmith: okay, then you already had a patch ? | |
| 14:23:27 | dansmith | bauzas: locally, I'll push | |
| 14:24:43 | bauzas | dansmith: okay, then upload it | |
| 14:25:35 | opendevreview | Dan Smith proposed openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | |
| 14:28:33 | bauzas | dansmith: looks to me we don't need to wait for the TC to be saying we should support a python version, as we merged the same for 3.10 without this https://review.opendev.org/c/openstack/nova/+/839029 | |
| 14:28:51 | dansmith | right, like I said, I don't think that matters :) | |
| 14:28:54 | bauzas | dansmith: so, +1 to your change but you could add a non-voting job if you want | |
| 14:29:01 | dansmith | I was just asking because I was surprised nobody had done it | |
| 14:29:07 | dansmith | bauzas: I don't want a job | |
| 14:29:11 | dansmith | all I want is to be able to run it locally | |
| 14:29:33 | bauzas | ack, if this is only for local testing, gtm | |
| 14:29:33 | dansmith | without this, it will run unit tests instead of functional if you try | |
| 14:29:47 | dansmith | we can add a job when the TC moves us to a 3.11-based distro, this is just for local testing yes | |
| 14:30:35 | bauzas | all cool then | |
| 14:43:38 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update to the PTL guide https://review.opendev.org/c/openstack/nova/+/875730 | |
| 15:28:38 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id column to instances table https://review.opendev.org/c/openstack/nova/+/879499 | |
| 15:28:38 | opendevreview | Dan Smith proposed openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | |
| 15:28:39 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |