Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-05-23
16:46:44 Uggla ?
16:53:01 bauzas Uggla: your spec got approved
16:53:08 bauzas which means "hands on deck now"
16:53:15 Uggla oh cool
16:59:18 gibi dansmith, bauzas: thanks
17:04:23 sean-k-mooney gmann: o/ hi looks like you are also tryinng to use a newer version of tempest on ussuri
17:04:25 sean-k-mooney https://review.opendev.org/c/openstack/devstack/+/838051
17:04:55 sean-k-mooney im trying to do the same with https://review.opendev.org/c/openstack/requirements/+/842991 andhttps://review.opendev.org/c/openstack/devstack/+/843006
17:07:06 sean-k-mooney ok im going to call it a day but ill take a look at those again tomorrow
17:14:03 opendevreview Merged openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506
17:16:15 melwitt sean-k-mooney: have you seen this re: upper-constraints? https://review.opendev.org/c/openstack/nova/+/842813
17:16:46 melwitt oh, nvm, I guess that's what you just talking about the last few messages
17:18:02 sean-k-mooney no i had not
17:18:16 sean-k-mooney but i dont think we want to do that
17:18:52 sean-k-mooney i dont really agree with pinnint tempest but if we do im not conviced we shoudl do it via devstack
17:19:37 sean-k-mooney we could but i think we need to contol that form the project side unless the QA team want to support all the brances we care about including the EM ones
17:20:32 melwitt it's a nova patch
17:20:44 sean-k-mooney i know but devstack does nto seam to use the right version
17:20:47 sean-k-mooney either
17:20:58 sean-k-mooney at least on the stable branch of the requirement repo it uses master
17:21:06 sean-k-mooney not the patch under test
17:22:00 sean-k-mooney it looks like on stable its using the checked out version of the requiremetns repo
17:22:05 sean-k-mooney well on train https://github.com/openstack/devstack/blob/stable/train/stackrc#L320
17:22:28 sean-k-mooney that would be fine i guess if tha tis the correct branch
17:23:15 sean-k-mooney melwitt: https://github.com/openstack/devstack/blob/stable/ussuri/stackrc#L315=
17:23:23 sean-k-mooney on ussui it woudl end up using master
17:23:44 sean-k-mooney melwitt: so i guess that patch is fine for train
17:23:54 sean-k-mooney but not for ussuri
17:24:17 melwitt hm ok
17:25:41 sean-k-mooney look like gmann pinned train with https://github.com/openstack/devstack/commit/8a22f7380c7029e931fe9103320f24a223b619d1
17:26:03 sean-k-mooney so i guess the train patch is fine to merge
17:26:15 sean-k-mooney but that would not help with ussuri
17:26:19 gibi bauzas: thanks for the review, replied in https://review.opendev.org/c/openstack/nova-specs/+/791047
17:26:49 sean-k-mooney melwitt: i kind of would have epected the newer stable branches to be fix before train
17:27:00 sean-k-mooney but maybe its because train is EM already
17:27:15 opendevreview ribaudr proposed openstack/nova-specs master: libvirt: Allow Manila shares to be directly attached to instances https://review.opendev.org/c/openstack/nova-specs/+/833669
17:27:31 sean-k-mooney melwitt: mind if we pick this up tomorrow the train nova patch is likely ok to proceed with
17:27:36 melwitt sean-k-mooney: it might be an oversight? gmann ^
17:27:47 melwitt sean-k-mooney: sure
17:28:18 sean-k-mooney i would think using the locally checked out requiremetn repo would always be valid
17:28:31 sean-k-mooney and that instead of a stable only change we coudl have done this on master and backported
17:28:48 sean-k-mooney at least on the nova side
17:29:24 melwitt yeah, that does make sense
17:29:24 sean-k-mooney anyway got to run and pick up food for dinner
17:30:01 melwitt o/
17:59:36 elodilles melwitt: about the TEMPEST_VENV thing: afaik that is intentional: tempest should run always against master in 'maintained' stable branches (that's why it is in venv)
18:00:23 melwitt elodilles: I replied on the review, I see what you mean, it was my misunderstanding. I see now that tempest is pinned on train but not elsewhere
18:00:40 melwitt thanks for the quick reply :)
18:01:21 elodilles melwitt: oh, sorry :) haven't seen your answer yet o:)
18:01:36 melwitt no worry, I posted it only a minute ago :)
18:02:08 elodilles :)
18:02:25 melwitt I should have said "I also replied on the review"
18:08:12 elodilles no problem :) i really wonder how these new problems crept into ussuri and train... i mean, the SSHABLE thing was not needed before, so why now? did we merge something to nova? or did it come with some other dependencies? :/
18:12:38 melwitt elodilles: same. looking at a CI run from today on stable/ussuri, one thing it's failing on the same oslo.log dep issue as on stable/train https://zuul.opendev.org/t/openstack/build/571459b47ccb4ed1854f61ce99597eca
18:14:20 melwitt ERROR: full: could not install deps [-chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/tempest/requirements.txt]; v = InvocationError('/opt/stack/tempest/.tox/tempest/bin/pip install -chttps://releases.openstack.org/constraints/upper/master -r/opt/stack/tempest/requirements.txt', 1)
18:29:41 elodilles melwitt: oslo.log 5.0.0 dropped py36 & py37
18:30:31 elodilles so yes it seems it is used from master's upper-constraints.txt :S
18:33:05 elodilles but on ussuri we have zuulv3. so you are right, we need a fix for ussuri then, too :S (though it will be probably a different fix, because the hook does not exist there anymore)
18:35:23 melwitt yeah :/
18:35:25 melwitt sean-k-mooney has been looking at it
18:46:20 elodilles i guess from zed till victoria now things are running with py38 and now that during zed some project dropped py36 support we need to pin things where still py36 is used only
18:46:45 elodilles i.e. on ussuri and train
20:24:12 sean-k-mooney elodilles: the sshable thing is needed partly because of a qemu change and partly because it was always a race
20:24:37 sean-k-mooney elodilles: it was previoulsy undefined behaivor to retry detach
20:25:15 sean-k-mooney we have always had races in those tests too with the kernel attachiting the device so it was technially always need but it was not an issue in the past
20:25:44 sean-k-mooney i dont know if canonical have backproted the change or centos have in the stable brances to qemu
20:26:20 sean-k-mooney but it could jsut be that the ci is runing slightly slower and now we are lossing rahter then wining the race
20:27:06 sean-k-mooney elodilles: using master requirement with master tempest makes sesnse but on any branch where we pin tempest we have to also pinn the upper constraits
20:27:53 sean-k-mooney we also need to deal with py36 by either using py38 to run tempest
20:28:22 sean-k-mooney on stable ussuri or using older requirements to work around oslo droping support
20:30:13 sean-k-mooney basiclaly not that master does not use py36 we cant use master in any job the uses py36 for the upper constraits so we need to cap to yoga at the latest
21:17:04 opendevreview Merged openstack/nova stable/train: [stable-only] Use Tempest's run upper constraints from devstack https://review.opendev.org/c/openstack/nova/+/842813
23:43:03 gmann sean-k-mooney: I did not get about your 'unpined' things. For all EM, we pin Tempest in devstack and so does compatible upper constraints to use. for ussuri, I am trying with tempest 30 but I have not finished that yet.
23:43:46 gmann sean-k-mooney: note, project can always override the pin tempest version and constraints via devstack variable in job so projects side choice still available
23:45:08 gmann I am not clear what exactly we miss to pin the Tempest for EM branch. All good there right?
23:50:24 melwitt gmann: I put the ussuri issue on the etherpad https://etherpad.opendev.org/p/nova-stable-branch-ci basically it's failing the same way train was, failing to install oslo.log version coming from upper-constraints master
23:54:45 gmann melwitt: thanks. I started working on this but then forget to figure out the failure and release - https://review.opendev.org/q/topic:ussuri-last
23:54:50 gmann I will work on that
23:58:17 melwitt ah k cool
23:59:01 gmann due to some constraint mismatch in Tempest master and stable/ussuri, it is little complicated and I am trying to see what combination of Tempest and constraint will work
#openstack-nova - 2022-05-24
01:53:35 opendevreview Ghanshyam proposed openstack/nova stable/ussuri: DNM: Testing stable/ussuri with tempest fix for constraints mismatch https://review.opendev.org/c/openstack/nova/+/843046
07:07:21 sean-k-mooney[m] gmann is ussuri EM or supported currently
07:07:35 sean-k-mooney[m] because it seams to be useing master uc
07:07:48 sean-k-mooney[m] but has pinned tempest
07:08:28 sean-k-mooney[m] so i think we either need to have it use python 3.8 so that master uc works
07:08:59 sean-k-mooney[m] or we need to chanve uc to be <= instead of ===
07:11:05 sean-k-mooney[m] gmann: if we dont pin i think we canc get it to install but i was actully using 3.9 to test i was going to try deplying ussuri today
07:11:32 sean-k-mooney[m] i was using one of my centos 9 vms to test yesterday which was too new to test properly
07:18:03 opendevreview Rico Lin proposed openstack/nova-specs master: Add vIOMMU device support for libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/840310
07:19:44 ricolin stephenfin: I need your feedback on comment https://review.opendev.org/c/openstack/nova-specs/+/840310/7..10/specs/zed/approved/libvirt-viommu-device.rst#b52 thanks :)
07:20:53 ricolin Also for aw_bits, I propose we set it to 48 (which at least will cover both 39 and 48 width option ) and don't expose it to end user
07:38:24 sean-k-mooney[m] ricolin if 48 is supported on our min version of qemu im ok to hard code to that
07:38:56 sean-k-mooney[m] at least for now until we have a need for something higher
07:55:26 whoami-rajat hi #openstack-nova , wanted to add a topic to today's meeting agenda, do we have an etherpad for the meeting?
07:55:48 sean-k-mooney[m] we have a wiki page
07:56:10 whoami-rajat ack, can i add topics there?
07:56:49 sean-k-mooney[m] yep just add it to the adgenda here https://wiki.openstack.org/wiki/Meetings/Nova https://wiki.openstack.org/wiki/Meetings/Nova
08:04:49 whoami-rajat done, thanks!
10:31:22 opendevreview Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127
11:08:58 ricolin sean-k-mooney[m]: aw_bit is introduced since libvirt 6.5.0 is that means I should propose in spec to bump min version for libvirt/qemu as well?

Earlier   Later