| opendevreview | Brian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints https://review.opendev.org/c/openstack/devstack/+/710519 | 02:28 |
|---|---|---|
| frickler | dansmith: the only relevant change I found is https://review.opendev.org/c/openstack/requirements/+/844424/3/upper-constraints.txt#b600 | 04:35 |
| opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 06:11 |
| kopecmartin | dansmith: i haven't seen an error like that, lol, it's weird | 07:11 |
| kopecmartin | i'll try to have a look during the week, although it will be a bit problematic, i'm attending the openinfra summit this week | 07:11 |
| *** jpena|off is now known as jpena | 07:45 | |
| *** bhagyashris_ is now known as bhagyashris | 08:03 | |
| whoami-rajat | gmann, bkopilov, hey, can you take another look at this cinder-tempest-plugin patch, I've updated it as per gmann's comments https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/795753 | 08:10 |
| bkopilov | whoami-rajat, ack , checking | 08:10 |
| whoami-rajat | thanks | 08:10 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Don't install pinned setuptools with distro pip https://review.opendev.org/c/openstack/devstack/+/844922 | 08:15 |
| frickler | dansmith: kopecmartin: ^^ something for discussion (assuming that it will solve the issue), maybe time to drop setuptools from u-c completely. | 08:16 |
| frickler | or maybe someone takes up the courage to finally make devstack use only venvs, which I think would also help here | 08:17 |
| frickler | meh, installing projects with u-c still forces updating setuptools. need to add unpin | 09:34 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Don't install pinned setuptools with distro pip https://review.opendev.org/c/openstack/devstack/+/844922 | 09:39 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Don't install pinned setuptools with distro pip https://review.opendev.org/c/openstack/devstack/+/844922 | 11:43 |
| *** akahat is now known as akahat|afk | 12:03 | |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Update grep cmd when checking for vgpu device https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/844969 | 13:23 |
| dansmith | frickler: and that fixes it? | 13:29 |
| *** akahat|afk is now known as akahat | 14:30 | |
| dansmith | gmann: in case you didn't see, jammy jobs are hosed for some reason and frickler has this proposed: https://review.opendev.org/c/openstack/devstack/+/844922 | 15:28 |
| dansmith | I figure it's worth getting multiple eyes on that before we decide to go that route | 15:28 |
| dansmith | cc clarkb ^ | 15:29 |
| frickler | fyi I also proposed dropping setuptools from u-c to prometheanfire, who also doesn't remember why it was there in the first place, see #openstack-requirements | 15:41 |
| dansmith | frickler: that's a good sign :P | 16:44 |
| *** jpena is now known as jpena|off | 16:48 | |
| gmann | dansmith: frickler clarkb kopecmartin +2 on that approach. not +A to have more eyes in case we are missing anything. | 16:48 |
| gmann | frickler: agree on u-c, I think it was there to specially blacklist the some broken setuptools version but original ration behind having it in setuptools is not known to me too. I vote for removing it from there and see how it goes | 16:49 |
| gmann | * original rational | 16:49 |
| dansmith | it feels wrong to silently edit the requirements text on the fiy, which is likely to confuse someone I think | 16:54 |
| dansmith | so I assume the right fix is to just remove it from u-c? | 16:54 |
| gmann | agree | 16:59 |
| gmann | whoami-rajat__: thanks lgtm, +1 | 17:00 |
| whoami-rajat__ | gmann, thanks! | 17:01 |
| jparker | artom: I know its WIP but I'll transition over to review ready after I try it against 16, but if you have time can you take a look at this? https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/844969 | 17:43 |
| jparker | artom: the core of the change is just the grep check that has changed | 17:43 |
| jparker | artom: no rush btw I have a workaround downstream until its patched | 17:44 |
| artom | jparker, seems legit | 17:45 |
| jparker | artom: thanks I'll move it to review ready after I try it against 16 | 17:46 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Update NUMA Affinity tests for per port policy https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/832693 | 18:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!