Tuesday, 2024-06-18

*** bauzas_ is now known as bauzas00:26
*** bauzas_ is now known as bauzas02:27
*** ykarel_ is now known as ykarel05:49
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction)  https://review.opendev.org/c/openstack/nova/+/83119408:20
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part)  https://review.opendev.org/c/openstack/nova/+/83309008:20
opendevreviewribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process  https://review.opendev.org/c/openstack/nova/+/88007508:20
opendevreviewribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion  https://review.opendev.org/c/openstack/nova/+/88147208:20
opendevreviewribaudr proposed openstack/nova master: Add metadata for shares  https://review.opendev.org/c/openstack/nova/+/85050008:20
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/85482308:20
opendevreviewribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/85482408:20
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/86028408:20
opendevreviewribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028508:20
opendevreviewribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares  https://review.opendev.org/c/openstack/nova/+/86028608:20
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares  https://review.opendev.org/c/openstack/nova/+/86028708:20
opendevreviewribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol  https://review.opendev.org/c/openstack/nova/+/88386208:20
opendevreviewribaudr proposed openstack/nova master: Check shares support (compute manager)  https://review.opendev.org/c/openstack/nova/+/88575108:20
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API)  https://review.opendev.org/c/openstack/nova/+/83683008:20
opendevreviewribaudr proposed openstack/nova master: Add helper methods to attach/detach shares  https://review.opendev.org/c/openstack/nova/+/88575308:20
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach notification  https://review.opendev.org/c/openstack/nova/+/85050108:20
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach notification  https://review.opendev.org/c/openstack/nova/+/85102808:20
opendevreviewribaudr proposed openstack/nova master: Add shares to InstancePayload  https://review.opendev.org/c/openstack/nova/+/85102908:20
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach_error notification  https://review.opendev.org/c/openstack/nova/+/86028208:20
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach_error notification  https://review.opendev.org/c/openstack/nova/+/86028308:20
opendevreviewribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working.  https://review.opendev.org/c/openstack/nova/+/85208608:20
opendevreviewribaudr proposed openstack/nova master: Add virt/libvirt error test cases  https://review.opendev.org/c/openstack/nova/+/85208708:20
opendevreviewribaudr proposed openstack/nova master: Docs about Manila shares API usage  https://review.opendev.org/c/openstack/nova/+/87164208:20
opendevreviewElod Illes proposed openstack/nova stable/2023.1: [ironic] Partition & use cache for list_instance*  https://review.opendev.org/c/openstack/nova/+/92215508:26
sahido/08:29
sahidhello gibi, any chance to have your eyes on that one? https://review.opendev.org/c/openstack/nova/+/921665 ?08:30
sahid:-)08:30
sahidI think you and _sean-k-mooney are the ones08:30
bauzassean-k-mooney: when you're up, I'm curious why FORCE=yes envvar wasn't used by ./stack.sh here https://review.opendev.org/c/openstack/nova/+/922140/3/.zuul.yaml#26708:55
bauzasthe job run failed because FORCE=yes wasn't around https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a38/922140/3/experimental/nova-mtty/a383bd1/job-output.txt08:55
bauzasoh, [[ True != \y\e\s ]]08:57
bauzashmmmm08:57
bauzassean-k-mooney: nevermind, I think I found the solution09:03
opendevreviewSylvain Bauza proposed openstack/nova master: WIP : Enable an experimental job for testing mtty  https://review.opendev.org/c/openstack/nova/+/92214009:03
opendevreviewSylvain Bauza proposed openstack/nova master: WIP : mtty: add devstack compilation for fedora builds  https://review.opendev.org/c/openstack/nova/+/91842009:03
fricklerbauzas: seems qa has finally found a volunteer to make devstack run on noble? ;)09:33
bauzasfrickler: yeah because I need to use a quite new libvirt version for testing mtty :) 09:35
sean-k-mooneybauzas: ya its proably not using the TrueOrFalse macro09:46
sean-k-mooneyfrickler: i have been meanign to try that too. i spun up a ubuntu vm when the beta was released but i dont think i got around to running devstack on it09:47
sean-k-mooneyfrickler: do we have images in ci yet. i was debating if i waould spend time on the dib patch or not09:48
bauzaslooks like using brackets with 'yes' worked09:52
bauzasthanks codesearch09:52
fricklersean-k-mooney: yes, images are there and I proposed to run py312 unittests on noble as next step, but I won't get to look into testing devstack myself https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/92213209:55
sean-k-mooneyfrickler: ack ya i think that makes more sense then using pyenv on debian 09:58
sean-k-mooneyfrickler: for what its worth i have no issue with useing debian or debian testing in our ci i just dont think we need to be compling our onw python even if we are doing it once in the image build when there is py312 aviabel in 24.04 and centos 10 stream already09:59
fricklersean-k-mooney: yes, using the default python version for each distro seems more sensible to me, too. but I haven't seen any work on centos 10 yet, is it out already?10:02
sean-k-mooneyyep its out for about a month10:02
sean-k-mooneyhttps://mirror.stream.centos.org/10-stream/10:02
sean-k-mooneythe generic cloud image is here https://composes.stream.centos.org/stream-10/production/latest-CentOS-Stream/compose/BaseOS/x86_64/images/CentOS-Stream-GenericCloud-10-20240617.0.x86_64.qcow210:03
sean-k-mooneyfrickler: i dont think there has been an offical release annouchment yet10:04
sean-k-mooneyim hoping we can get ubuntu 24.04 into ci this cycle and c10s next cycle as a testing runtime with the 2025.1 the last release with c9s10:05
sean-k-mooneyi.e 2025.1 will be the upgrade release for both ubuntu 22.04->24.04 and centos stream 9 -> 1010:06
fricklerIMO you'd better get both working at least with devstack this cycle, then, so the TC can add them to the support list for 2025.1 from the start10:07
sean-k-mooneythat woudl be ideal yes10:08
sean-k-mooneythe supported runtimes have generally been based on aviablity not devstack support in the past10:08
sean-k-mooneyso both are valid for inclustion and should be added by default based on our disto upgrade policy10:09
sean-k-mooneyassumign c10s has its offial release annouchment before the start of the next cycle10:09
sean-k-mooneypractically speakign however yes we need devstack to work10:10
sean-k-mooneythe main challange is likely the changes to pip isntalling 10:10
sean-k-mooneyubuntu should be in a good place but c9s is still using a venv10:10
sean-k-mooneyso we need to fix that this cycle10:10
sean-k-mooney*still not useing a venv10:10
sean-k-mooneyi have not chekced if c10s has blocked "sudo pip install" or not10:11
sean-k-mooneybut thats the direction most distor seam to be going so i would not be surpprised if they have, although fedora has not yet so im unsure10:11
opendevreviewFabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load  https://review.opendev.org/c/openstack/nova-specs/+/92220111:45
opendevreviewFabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load  https://review.opendev.org/c/openstack/nova-specs/+/92220112:16
*** bauzas_ is now known as bauzas12:23
opendevreviewFabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load  https://review.opendev.org/c/openstack/nova-specs/+/92220112:35
*** bauzas_ is now known as bauzas12:40
opendevreviewFabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load  https://review.opendev.org/c/openstack/nova-specs/+/92220112:55
bauzasfrickler: sean-k-mooney: fwiw, I got some distutils issue when stacking on noble https://zuul.opendev.org/t/openstack/build/9c2329f04a5344b4bbc9661e2360d282/log/job-output.txt#8152-816013:15
bauzasthanks py3.12 13:16
sean-k-mooney well distutils was remvoed13:17
sean-k-mooneyit wsa deprected for quite a while13:17
bauzasI know13:18
bauzasdeprecated in 3.10 13:18
sean-k-mooneyso this seams to be a limitaiton in the requirement repo that should be updated13:18
bauzaswith a removal on 3.1213:18
sean-k-mooneyits not an issue in openstack istslef13:18
bauzas(which is *again* a short time)13:18
sean-k-mooney2 years  is not short13:18
bauzasfor a language ?13:19
sean-k-mooneycorrect13:19
bauzasyou're kidding?13:19
sean-k-mooneyi think your postion on that is unreasonable13:19
sean-k-mooneybut i dont want to get into that now13:19
bauzasask the operators what their positions are on something like 2 years13:19
bauzasanyway, ship has sailed 13:20
fricklerah, nice, I noticed earlier today that also the use of pkg_resources in reqs repo needs replacing13:20
bauzasand I agree, this is a requirements repo13:20
sean-k-mooneyits the cadance we do downtream major releease on and that lts distos often release on13:20
sean-k-mooneyalso i belive as of 3.11 python are not giving 3 release or 3 years for removals13:21
frickleralso seems that there is no unit testing for edit-constraints. more fun13:21
sean-k-mooneyfrickler: i would expect it to be used in all the tox gets for the cross gating13:21
sean-k-mooneyfrickler:but probaly nothing direcltly testing it13:21
* bauzas thinks that Oracles takes popcorn while seeing the Python community rushing so hard for getting angst13:22
frickleryes, the cross tests all run the latest support python, which is 3.1113:22
sean-k-mooneyfrickler: since this was not seen on the debinian pyevn tests im guessign it was falling back to the dist utils module form the default versio13:22
fricklersean-k-mooney: iiuc the error is seen in devstack only, that didn't run on py312 yet13:23
bauzasfrickler: want me to report a bug against requirements ?13:23
sean-k-mooneymaybe we dont do the requriemts edit for tox because we are installing the package explcity 13:23
sean-k-mooneybauzas: can you try installing "sudo apt-get install python3-distutils"13:25
sean-k-mooneybauzas: it might still be shipped but not instlled by default on 24.0413:25
fricklerbauzas: I won't advise you to use storyboard and I very rarely look at it, but feel free to do so https://storyboard.openstack.org/#!/project/openstack/requirements13:25
bauzasI don't have a box13:25
bauzasI only use CI resources13:25
bauzasbut I can spin some noble instance (provided I can on our internal lab)13:25
fricklerI have a noble test node running, let me check that13:25
sean-k-mooneyoh ok thats fine ill see if the package exist and push a quick patch to devstack to add it13:26
bauzasdistutils is continued to be shipped thru setuptools13:26
bauzashttps://docs.python.org/3.12/whatsnew/3.12.html#distutils13:26
bauzasso I see a workaround here13:27
sean-k-mooneyok ya for the short term13:27
sean-k-mooneyubuntu dropped the seperate pacakage in 23.1013:27
sean-k-mooneyfunny that https://packages.ubuntu.com/noble/python3-distutils-extra is still a thing13:28
sean-k-mooneythat pulls in setuptools to provide the core supprot13:28
bauzasthat's call "washing our hands"13:28
bauzascalled*13:29
sean-k-mooneyhow, we are still going to need to fix requireemtns13:29
sean-k-mooneythis is just a short term workaround13:29
fricklerhmm, I don't get the failure on my noble node. and I didn't even run devstack there13:29
sean-k-mooneydo you have setuptools13:29
fricklerbut it does have python3-dev installed, which pulled a lot of deps13:29
frickleryes, python3-setuptools is one of them13:30
sean-k-mooneyya so thats what provides it13:30
sean-k-mooneywe are surly installing python3-dev in devstack however13:30
bauzashttps://zuul.opendev.org/t/openstack/build/9c2329f04a5344b4bbc9661e2360d282/log/job-output.txt#8152-816013:30
sean-k-mooneyi wonder is the a venv thing13:30
bauzasI can try to look at the installed packages13:30
sean-k-mooneywe dont allow site packages13:30
sean-k-mooneyso we might need to explcitly install setuptools form pip in the venv13:31
bauzashttps://0dcb2cbd4fa467880edd-6556216e0c08650db1b5109918fbb7f6.ssl.cf2.rackcdn.com/922140/4/experimental/nova-mtty/9c2329f/controller/logs/dpkg-l.txt13:31
sean-k-mooneyi dont think python -m venv isntalls setup tools by default and we used to clamp it partly becasue of our use of pbr and clamping pip version in the past13:31
sean-k-mooneymy guess is if setuptools is in the venv its too old to provide it13:31
opendevreviewElod Illes proposed openstack/nova master: [CI] Replace deprecated regex  https://review.opendev.org/c/openstack/nova/+/92221213:32
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex  https://review.opendev.org/c/openstack/nova/+/92221313:32
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex #2  https://review.opendev.org/c/openstack/nova/+/92221413:32
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex #3  https://review.opendev.org/c/openstack/nova/+/92221513:32
sean-k-mooneyah13:32
sean-k-mooneyhttps://docs.python.org/3.12/whatsnew/3.12.html#ensurepip13:32
sean-k-mooneyeasy_install, pkg_resources, setuptools and distutils are no longer provided by default in environments created with venv or bootstrapped with ensurepip, since they are part of the setuptools package. For projects relying on these at runtime, the setuptools project should be declared as a dependency and installed separately (typically, using pip).13:32
sean-k-mooneyso ya we need to expictly include setuptools in the venv13:33
bauzassean-k-mooney: frickler: python3-setuptools was installed thru a apt dependency https://0dcb2cbd4fa467880edd-6556216e0c08650db1b5109918fbb7f6.ssl.cf2.rackcdn.com/922140/4/experimental/nova-mtty/9c2329f/controller/logs/devstacklog.txt13:44
sean-k-mooneybauzas: yes but you missed the part i said above about the venv13:46
opendevreviewFabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load  https://review.opendev.org/c/openstack/nova-specs/+/92220113:46
sean-k-mooneybauzas: we install devstack in a venv without accss to site packages13:46
sean-k-mooneyso the setup tools package need to installed form pip in that venv 13:46
*** bauzas_ is now known as bauzas13:53
bauzassorry, got a fucking IRC blip due my LTE connection13:55
sean-k-mooneyno worries13:55
bauzassean-k-mooney: I just said I thought we were installing devstack in a venv if and only if GLOBAL_VENV was set13:55
bauzaswhich is not the case with that job13:55
sean-k-mooneybauzas: its the default13:56
bauzasoh, a venv per project then13:56
sean-k-mooneyand not installing in a venv is effectilvy deprecated13:56
bauzascompared to a larger global vencv13:56
sean-k-mooneyno the venv per project is even more deprecated13:56
bauzaswith shared venv for all projects13:56
bauzashum, I'm lost13:57
bauzaswhat's the difference between GLOBAL_VENV=True and False?13:57
sean-k-mooneyGLOBAL_VENV=True uses a single venv for all project 13:57
fricklerwe are using a dedicated venv in this case /opt/stack/requirements/.venv/bin/edit-constraints13:57
sean-k-mooneyGLOBAL_VENV=false does a site wide install 13:57
*** haleyb|out is now known as haleyb13:57
sean-k-mooneyusing sudo13:57
fricklerI guess because this runs before actual service installs are happening13:58
sean-k-mooneybauzas: the only reason we have not removed GLOBAL_VENV and made GLOBAL_VENV=True the only option is centos/fedroa were broken due to the way pbr generates wsgi files13:58
sean-k-mooneyfrickler: like not because of that but because of grenade13:59
sean-k-mooneyfrickler: like tempest the requiremet repo might be slightly diffent in terms of deps13:59
bauzasfrickler: okay, can you please confirm me that the stack wasn't done with a global venv ?13:59
* bauzas tries to untangle the pieces14:00
sean-k-mooneybauzas: the pice your mssing is enven with that set some sepical project use there own venv14:00
sean-k-mooneybauzas: tempest alwas uses its own venv becaue we uses master tempest on stable/*14:00
sean-k-mooneybauzas: the requirement repo apprently also does that14:00
sean-k-mooneyfrickler: the quick fix for now woudl be to add setuptools explcitly to the requirements.txt for the requiremets repo14:01
sean-k-mooneyuntil the distutils dep is remvoed14:01
opendevreviewElod Illes proposed openstack/nova master: [CI] Replace deprecated regex  https://review.opendev.org/c/openstack/nova/+/92221214:20
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex  https://review.opendev.org/c/openstack/nova/+/92221314:20
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex #2  https://review.opendev.org/c/openstack/nova/+/92221414:20
opendevreviewElod Illes proposed openstack/nova master: DNM: test zuul job regex #3  https://review.opendev.org/c/openstack/nova/+/92221514:20
*** bauzas_ is now known as bauzas14:43
*** ykarel is now known as ykarel|away15:39
bauzasnova meeting in ~4 mins here15:56
bauzas#startmeeting nova16:00
opendevmeetMeeting started Tue Jun 18 16:00:14 2024 UTC and is due to finish in 60 minutes.  The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'nova'16:00
bauzashey folks16:00
gibio/16:00
auniyalo/16:00
bauzassorry, maybe my bouncer will fail during this meeting16:00
elodilleso/16:00
bauzas#link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting16:01
bauzaslet's start16:02
bauzas#topic Bugs (stuck/critical) 16:02
bauzas#info No Critical bug16:02
bauzas#info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster16:02
bauzasany bugs to raise ?16:02
bauzaslooks not16:03
bauzas#topic Gate status 16:03
Ugglao/16:04
bauzas#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:04
bauzas#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:04
bauzas#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status16:04
bauzaseverything green except nova-emulation 16:04
bauzas#link https://zuul.openstack.org/build/c118533b8c7c4954904ef430e4597ecd16:04
bauzasthe usual suspect16:04
bauzasbut I think we discussed it last week16:05
bauzasso, moving on16:05
bauzas#info Please look at the gate failures and file a bug report with the gate-failure tag.16:05
bauzas#info Please try to provide meaningful comment when you recheck16:05
bauzasany gate things you want to discuss ?16:05
bauzas#topic Release Planning 16:06
bauzas#link https://releases.openstack.org/dalmatian/schedule.html16:06
bauzas#info Dalmatian-2 in 2 weeks16:06
bauzas#info Spec review day planned on July 1st https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/JBPORDTEELT5ZVPJAFCYD733IVSMXMKR/16:06
bauzasI slowly started to review some specs (due to my internet issue), but I'll continue to do this16:07
bauzasanything about this ?16:07
bauzaslooks not16:08
bauzas#topic Review priorities 16:08
bauzas#link https://etherpad.opendev.org/p/nova-dalmatian-status16:08
bauzas#topic Stable Branches 16:08
bauzaselodilles: floor is yours16:08
elodilles#info stable/202*.* gates seem to be OK16:08
elodilles#info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci16:09
bauzas++16:09
elodillesso nothing special16:09
bauzasack16:09
bauzasthanks then16:10
bauzas#topic vmwareapi 3rd-party CI efforts Highlights 16:10
bauzasfwiesel: no updates AFAICS16:10
fwieselCorrect.16:10
fwieselI do understand some failures where due to some changes in openstack/requirements?16:11
fwieselJust to cross-check.16:11
bauzaswhich failures ? :)16:11
fwieselGreat, or not so great, depending. Then nothing from my side. Any questions or remarks?16:13
fwieselbauzas: Then back to you16:13
fwiesel#info No updates16:14
fwieselMissed that one.16:14
bauzasok16:14
bauzas#topic Open discussion 16:14
bauzasnothing on the agenda, unless I'm blind16:14
bauzasso, anything here ?16:14
bauzasok cool16:16
bauzasgiving you a whole of time back16:16
bauzasthanks all16:16
bauzas#endmeeting16:16
opendevmeetMeeting ended Tue Jun 18 16:16:30 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:16
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-18-16.00.html16:16
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-18-16.00.txt16:16
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-18-16.00.log.html16:16
fwiesel\o16:16
gibio/16:16
sean-k-mooneyo/16:17
elodilleso/16:18
opendevreviewsean mooney proposed openstack/nova stable/2024.1: Revert "Always delete NVRAM files when deleting instances"  https://review.opendev.org/c/openstack/nova/+/92223516:26
bauzasfrickler: so, please lemme know once the requirements patch will support setuptools for devstack so I could update my own mtty patch16:29
bauzaswith a Depends-On16:29
opendevreviewJens Harbott proposed openstack/nova master: WIP : Enable an experimental job for testing mtty  https://review.opendev.org/c/openstack/nova/+/92214019:27
frickler^^ just because I'm curious ;)19:28
*** haleyb is now known as haleyb|out22:27

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!