| *** bhagyashris is now known as bhagyashris|ruck | 04:42 | |
| opendevreview | yatin proposed openstack/devstack master: Use upper-constraints from in review changes https://review.opendev.org/c/openstack/devstack/+/823127 | 07:06 |
|---|---|---|
| frickler | kopecmartin: gmann: yoctozepto: ^^ seems ykarel has found out how the broken u-c change got merged. I commented on https://review.opendev.org/c/openstack/requirements/+/823128 , but I'd prefer to have at least one further opinion on it before single-approving | 09:20 |
| frickler | I've actually wondered yesterday how it happened that the paramiko update was a couple of days old, but gate failures seemed more recent. somehow I didn't check reqs closely enough | 09:22 |
| yoctozepto | frickler: done both | 09:47 |
| opendevreview | Radosław Piliszek proposed openstack/devstack master: Use upper-constraints from in review changes https://review.opendev.org/c/openstack/devstack/+/823127 | 09:48 |
| opendevreview | Dr. Jens Harbott proposed openstack/tempest master: DNM: Test paramiko override for cirros https://review.opendev.org/c/openstack/tempest/+/823159 | 11:38 |
| frickler | yoctozepto: the devstack patch is still failing grenady things. not sure whether we should backport the blocks in requirements first or n-v the jobs in devstack? | 11:46 |
| frickler | I also found that possible workaround in tempest ^^ | 11:46 |
| frickler | hmm, wait, paramiko is capped in xena (which is a questionable thing in its own), so why exactly is grenade failing? maybe a similar thing to 823127 is needed inside grenade code | 11:49 |
| opendevreview | Dr. Jens Harbott proposed openstack/tempest master: DNM: Test paramiko override for cirros https://review.opendev.org/c/openstack/tempest/+/823159 | 12:32 |
| yoctozepto | frickler: that could be it | 13:02 |
| yoctozepto | I mean, grenade needing proper u-c handling | 13:02 |
| yoctozepto | frickler: xena also uses origin/master so it breaks grenade | 13:14 |
| yoctozepto | the failure is on smoke tests pre-upgrade | 13:14 |
| yoctozepto | frickler: can you merge https://review.opendev.org/c/openstack/requirements/+/823128 ? | 13:31 |
| yoctozepto | let's get the gates worky | 13:31 |
| frickler | kopecmartin: gmann: https://review.opendev.org/c/openstack/tempest/+/823159 seems to be working, not sure if we'd want to keep the option unconditional as it currently stands or add another config option | 15:17 |
| frickler | incidentally https://review.opendev.org/c/openstack/tempest/+/807465/22 would be another option, as the issue would only affect rsa keys and not ecdsa | 15:18 |
| frickler | I'll also try to get an updated cirros version done | 15:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!