| opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866155 | 09:09 |
|---|---|---|
| opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | 09:10 |
| bauzas | sean-k-mooney: could you please look at this small PTL doc change ? https://review.opendev.org/c/openstack/nova/+/875730 | 09:17 |
| sean-k-mooney | bauzas: sure ill review it now. for what its worth i never really found the nova wiki useful | 09:46 |
| sean-k-mooney | so i prefer having this in the main schdule | 09:46 |
| 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:48 |
| sean-k-mooney | * Run the `count-blueprints`__ script daily to gather data for the cycle | 09:49 |
| sean-k-mooney | burndown chart | 09:49 |
| sean-k-mooney | do you really think that has value ? | 09:49 |
| sean-k-mooney | i really dont | 09:50 |
| sean-k-mooney | bauzas: soft -1 i have two questions inline | 09:54 |
| sean-k-mooney | over all the update looks good | 09:55 |
| bauzas | sean-k-mooney: sorry I was afk as I need to taxi my daughter | 10:17 |
| opendevreview | Merged openstack/nova stable/xena: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866155 | 10:18 |
| *** blarnath is now known as d34dh0r53 | 12:14 | |
| opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | 12:22 |
| 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 |
| dansmith | because of the silly tox behavior where it will run any missing testenv | 13:40 |
| bauzas | dansmith: hem, yeah, I guess we can now support py 3.11 by the TC ? | 14:14 |
| 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:14 |
| dansmith | bauzas: ^ | 14:20 |
| bauzas | dansmith: because you run tox -efunctional by default ? | 14:21 |
| 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 |
| bauzas | sorry, I need to understand the problem | 14:22 |
| dansmith | tox has always done | 14:22 |
| bauzas | dansmith: hah, ok | 14:22 |
| dansmith | that | 14:22 |
| dansmith | tox -enot-a-real-thing will pass | 14:22 |
| bauzas | dansmith: yeah because you pin the python version when calling the target | 14:22 |
| dansmith | so I just want to put up a tox.ini modification to catch 311 as well | 14:22 |
| dansmith | not really related to the python version | 14:23 |
| dansmith | well, it is in the sense that functional-310 won't work | 14:23 |
| bauzas | dansmith: okay, then you already had a patch ? | 14:23 |
| dansmith | maybe that's what you mean | 14:23 |
| dansmith | bauzas: locally, I'll push | 14:23 |
| bauzas | dansmith: okay, then upload it | 14:24 |
| opendevreview | Dan Smith proposed openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | 14:25 |
| 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 |
| dansmith | right, like I said, I don't think that matters :) | 14:28 |
| bauzas | dansmith: so, +1 to your change but you could add a non-voting job if you want | 14:28 |
| dansmith | I was just asking because I was surprised nobody had done it | 14:29 |
| dansmith | bauzas: I don't want a job | 14:29 |
| dansmith | all I want is to be able to run it locally | 14:29 |
| dansmith | without this, it will run unit tests instead of functional if you try | 14:29 |
| bauzas | ack, if this is only for local testing, gtm | 14:29 |
| 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:29 |
| bauzas | all cool then | 14:30 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Update to the PTL guide https://review.opendev.org/c/openstack/nova/+/875730 | 14:43 |
| opendevreview | Dan Smith proposed openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | 15:28 |
| opendevreview | Dan Smith proposed openstack/nova master: Add compute_id column to instances table https://review.opendev.org/c/openstack/nova/+/879499 | 15:28 |
| opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | 15:28 |
| opendevreview | Merged openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | 15:54 |
| opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821980 | 15:55 |
| gmann | dansmith: bauzas yes, that is what we did in past also to test the py version in advance so that we will make things compatible when it is in testing runtime | 17:26 |
| gmann | dansmith: bauzas: do you know if any distro support it in their official release? we can add unit test job also as non voting in general template | 17:27 |
| dansmith | fedora has it | 17:27 |
| dansmith | I don't think we need to add a job yet, personally | 17:27 |
| dansmith | I just don't want to run functional-py311, see it pass (because it's running unit tests), submit my patch and then see ALL the functionals have failed :P | 17:27 |
| gmann | ok, I was waiting for debian to release it and we can add job that time | 17:28 |
| gmann | true, adding functional tests run also on that is good idea | 17:28 |
| dansmith | gmann: to be clear, I can run 'tox -epy311' and get unit tests on 3.11 as expected, | 17:29 |
| dansmith | but I can't run functionals locally because there is no testenv | 17:29 |
| dansmith | so tox just makes up a fake functional-py311 based on the base testenv (which is unit tests) and then lies to me :) | 17:29 |
| gmann | dansmith: ah yeah due to default tox env https://review.opendev.org/c/openstack/nova/+/879559/2/tox.ini#3 | 17:30 |
| dansmith | right | 17:30 |
| gmann | dansmith: +W on your patch | 17:30 |
| dansmith | thanks | 17:30 |
| sean-k-mooney | i think 3.11 is also alredy in fedora 37 and will be in ubuntu 23.04 | 17:37 |
| sean-k-mooney | so it should be in the cloud archve ocne that releases | 17:37 |
| sean-k-mooney | 3.11.0~rc1 shoudl be in jammy-updates/universe packages already | 17:38 |
| sean-k-mooney | so it wont be long before we have it on 22.04 | 17:38 |
| sean-k-mooney | dansmith: oh while i think of it i appoved the schduler lazy loading patch thanks for the reminder | 17:41 |
| dansmith | thanks | 17:41 |
| sean-k-mooney | are we going to backport that | 17:41 |
| sean-k-mooney | i assume so but not sure how far | 17:41 |
| dansmith | upstream? I wouldn't think so | 17:41 |
| sean-k-mooney | ok i would at least cherry pick it upstream to antelope if we are going to backpot it downstream and see what elodilles thinks | 17:42 |
| sean-k-mooney | if nothign else it will give use a ci run and we can abandon it | 17:42 |
| dansmith | there's a bug for it, but it doesn't really resolve an issue, it's more a serviceability feature :) | 17:43 |
| dansmith | but yeah whatever | 17:43 |
| dansmith | it can't go back any further than the conductor on | 17:43 |
| dansmith | *one | 17:43 |
| clarkb | dansmith: tox lying is one of the things I don't like about it. Nox is a lot more explicit | 17:43 |
| dansmith | clarkb: I know, I've never understood that behavior | 17:43 |
| sean-k-mooney | well its because we removed the config that froce it to use the specifed version since it "fixed" | 17:44 |
| sean-k-mooney | dansmith: were you geting a non default python version locally | 17:44 |
| clarkb | re python3.11 we're successfully using the rc package on jammy for unittests in zuul then rely on functional testing with python:3.11-bullseye based images to sanity check it | 17:44 |
| sean-k-mooney | i.e. something other then 3.11? | 17:44 |
| dansmith | no | 17:45 |
| sean-k-mooney | clarkb: good to know but we should get teh full released version in like 2 or 3 weeks in 22.04 right? | 17:45 |
| dansmith | sean-k-mooney: run this: 'tox -esnarglepuss9000' | 17:45 |
| sean-k-mooney | ok that should create a default env sicne it wont match any of our default ones | 17:46 |
| sean-k-mooney | and run whatever we have as the default | 17:46 |
| dansmith | tox -e'failopotomus2000' | 17:46 |
| dansmith | exactly | 17:46 |
| dansmith | as does functional-py311 | 17:46 |
| sean-k-mooney | yes that more or less what i expect | 17:46 |
| sean-k-mooney | but if you did tox -e functional | 17:46 |
| sean-k-mooney | it would have used 3.11 | 17:46 |
| sean-k-mooney | if that is your system default | 17:46 |
| sean-k-mooney | i know its not intuitive | 17:47 |
| sean-k-mooney | but that was the behvior i observed when we looked at using the generitive envs | 17:47 |
| dansmith | yeah, you understand that I understand, right? I just wanted it to be updated so I can use my aliases, which all specify the version | 17:47 |
| sean-k-mooney | yep | 17:47 |
| sean-k-mooney | no issue with the patch | 17:47 |
| sean-k-mooney | i just avoid using the versioned ones ot not have that problem | 17:48 |
| clarkb | sean-k-mooney: I don't know if they will update the full version in jammy | 17:48 |
| clarkb | they might | 17:48 |
| sean-k-mooney | the yave in the past (not the default) but ya not sure | 17:49 |
| sean-k-mooney | i dont really mind using debian instead for the unit/functional tests if that is what makes sense | 17:50 |
| sean-k-mooney | is 3.11 in the PTI runtimes this release | 17:50 |
| sean-k-mooney | looks like no | 17:50 |
| sean-k-mooney | https://github.com/openstack/governance/blob/master/reference/runtimes/2023.2.rst but it would still be nice to have some coverage | 17:51 |
| clarkb | its also just faster ime. On tumbleweed locally it is 10% quicker on my machine to run zuul's unittests under python3.11 + x86_64-v3 than python3.10 without x86_64-v3 (can't be sure how much of that is python version vs cpu stuff though) | 17:55 |
| sean-k-mooney | tumbleweed is compliing for x86_64-v3 | 17:56 |
| sean-k-mooney | i think rhel 9 just moved to -v1 | 17:56 |
| sean-k-mooney | oh actully v2 https://developers.redhat.com/blog/2021/01/05/building-red-hat-enterprise-linux-9-for-the-x86-64-v2-microarchitecture-level | 17:57 |
| sean-k-mooney | v3 gets avx2 which is nice but i woudl be surpised if that impacted unit tests | 17:58 |
| sean-k-mooney | perhaps it impoved the hash funciton for dictonaries but either way 10% is nice | 17:58 |
| clarkb | sean-k-mooney: they are doing overlay packages that do ldpreload magic or something | 18:03 |
| clarkb | basically old cpu support remains then zypper detects if you've got a newer cpu and it will automaticall install the overlay packages too | 18:03 |
| sean-k-mooney | ah ok so they are building for v3 and also for v2 or older | 18:03 |
| sean-k-mooney | and using ldpreload to load the correct lib based on cpu support | 18:04 |
| sean-k-mooney | ya i think rhel considerd that and tought it was too much work :) | 18:04 |
| opendevreview | Merged openstack/nova master: Allow running functional-py311 https://review.opendev.org/c/openstack/nova/+/879559 | 19:15 |
| opendevreview | Dan Smith proposed openstack/nova master: Add compute_id columns to instances, migrations https://review.opendev.org/c/openstack/nova/+/879499 | 19:39 |
| opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | 19:39 |
| opendevreview | Dan Smith proposed openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | 19:39 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | 21:29 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | 21:30 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | 21:36 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM testing grenade object versions https://review.opendev.org/c/openstack/nova/+/879687 | 21:37 |
| opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | 23:37 |
| opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | 23:37 |
| opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | 23:37 |
| opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!