| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-23 | |||
| 16:27:02 | dansmith | oh yeah I saw this ack | |
| 16:27:05 | dansmith | oops | |
| 16:27:33 | dansmith | that was intended to be two statements to two different people, but it worked out this time :) | |
| 16:28:19 | sean-k-mooney | oh i have debated if that was useful in the past | |
| 16:28:39 | sean-k-mooney | i sometimes start replyint in the wrong tab if misclick | |
| 16:28:49 | sean-k-mooney | but more often then not i assuem that causes issues | |
| 16:29:18 | gibi | :) | |
| 16:29:37 | dansmith | I'm not a fan.. end up with too many "that guy is an ass okay sure buddy" things where you don't remember to hit send in a /query and then switch | |
| 16:33:01 | sean-k-mooney | https://review.opendev.org/c/openstack/requirements/+/842991 is going to fail but even using an old upper-constriats i think wont work | |
| 16:33:13 | sean-k-mooney | its been resolving for ~5 mins locally | |
| 16:34:13 | sean-k-mooney | realistically if i want to bump the tempest version i think i would need to either do this differently via devstack or bump other constraits | |
| 16:41:22 | sean-k-mooney | actully the best apptoch might be to just not use an upper-constratits file for the venv | |
| 16:44:03 | dansmith | gibi: I have one typo gripe plus bauzas | |
| 16:44:12 | dansmith | can we fix and re-ack real quick? | |
| 16:44:23 | dansmith | I can edit inline and then +2 if you'll be around to +W | |
| 16:44:47 | bauzas | dansmith: sure | |
| 16:45:14 | opendevreview | Dan Smith proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506 | |
| 16:45:22 | sean-k-mooney | ok i think the best way to make this work is just to remove using upper-constraits from the venv in devstack ill try that and see if that work | |
| 16:45:22 | bauzas | gibi: I'll have comments on the PCI spec, mostly on the config options | |
| 16:45:30 | dansmith | bauzas: gibi done | |
| 16:45:35 | bauzas | dansmith: saw | |
| 16:46:30 | bauzas | Uggla: typey typey now | |
| 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 | |