| *** bhagyashris is now known as bhagyashris|ruck | 04:38 | |
| *** bhagyashris_ is now known as bhagyashris|ruck | 09:55 | |
| *** bhagyashris_ is now known as bhagyashris|ruck | 10:44 | |
| ade_lee | dansmith, rosmaita hey guys - can we get some of the needed backports for fips merged? | 19:53 |
|---|---|---|
| ade_lee | https://review.opendev.org/c/openstack/glance/+/828233 | 19:53 |
| ade_lee | https://review.opendev.org/c/openstack/glance/+/828232/2 | 19:54 |
| ade_lee | https://review.opendev.org/c/openstack/glance/+/828240 | 19:54 |
| ade_lee | https://review.opendev.org/c/openstack/glance/+/828239 | 19:54 |
| dansmith | ade_lee: I have no such powers | 20:12 |
| dansmith | I almost said ', sorry" at the end, but it's really you who are sorry about that :P | 20:12 |
| rosmaita | ade_lee: i hit the xena backports | 20:17 |
| rosmaita | dansmith: can you take a look at https://review.opendev.org/c/openstack/python-glanceclient/+/830824 | 20:17 |
| dansmith | rosmaita: can you link me to an example of it running the wrong thing? | 20:19 |
| dansmith | rosmaita: in glance it's TEST_PATH not OS_TEST_PATH.. maybe that's the problem? | 20:21 |
| rosmaita | don't know ... OS_TEST_PATH works fine for cinderclient | 20:22 |
| dansmith | okay, poking locally real quick | 20:22 |
| dansmith | ah, I got it | 20:23 |
| dansmith | may I revise your patch? | 20:23 |
| rosmaita | please do | 20:23 |
| rosmaita | dansmith: to answer your question, https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5a7/829155/1/check/glanceclient-dsvm-functional/5a7b76e/testr_results.html | 20:24 |
| rosmaita | from https://review.opendev.org/c/openstack/python-glanceclient/+/829155/ | 20:25 |
| opendevreview | Dan Smith proposed openstack/python-glanceclient master: Fix functional CI job https://review.opendev.org/c/openstack/python-glanceclient/+/830824 | 20:25 |
| dansmith | I repro'd locally so I saw | 20:25 |
| dansmith | rosmaita: ^ | 20:25 |
| rosmaita | jeez | 20:25 |
| rosmaita | freakin' dotfiles | 20:25 |
| dansmith | rosmaita: probably okay for you to +W that.. trivial and that way both of us are on it | 20:26 |
| rosmaita | i wonder whether we should just remove that line from .stestr.conf and only set it in tox.ini | 20:26 |
| dansmith | this matches glance | 20:26 |
| rosmaita | yeah, but i don't know that that's a good reason! | 20:27 |
| dansmith | haha | 20:27 |
| dansmith | well, up to you I guess, i think fixing it is probably the priority and not the purity, but whatever you want | 20:27 |
| rosmaita | but you're right, it is TEST_PATH in glance's .stestr.conf | 20:27 |
| rosmaita | looks like those tests have been not running for 3 years, 9 months | 20:38 |
| rosmaita | i fixed them 4 years, 2 months ago | 20:39 |
| rosmaita | so they had a good run of 5 months back in queens | 20:39 |
| ade_lee | dansmith, thanks anyways :/ | 21:20 |
| ade_lee | rosmaita, thanks - feel free to continue with the wallaby backports :) | 21:21 |
| opendevreview | Merged openstack/glance stable/xena: Update qcow test to use qcow2 instead https://review.opendev.org/c/openstack/glance/+/828233 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!