| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 07:04 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 07:05 |
| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 07:11 |
| tkajinam | tobias-urdin, I might need some help to look into the broken job in gnocchi... | 07:26 |
| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 08:16 |
| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 08:19 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Load definition files from config directory https://review.opendev.org/c/openstack/ceilometer/+/931198 | 08:21 |
| opendevreview | Takashi Kajinami proposed openstack/aodh master: Always load api-paste.ini from config directory https://review.opendev.org/c/openstack/aodh/+/931196 | 08:22 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Load definition files from config directory https://review.opendev.org/c/openstack/ceilometer/+/931198 | 08:26 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ceilometer/+/931185 | 09:09 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ceilometer/+/931185 | 09:11 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ceilometer/+/931185 | 09:13 |
| tobias-urdin | tkajinam: sure, in ceilometer or gnocchi ci? | 09:20 |
| tkajinam | tobias-urdin, in gnocchi. it seems posgresql-file-upgrade-from-4.4 is broken for me | 10:12 |
| tkajinam | eg. https://github.com/gnocchixyz/gnocchi/actions/runs/11121454005/job/30900605292 | 10:12 |
| tkajinam | if you know a good way to run that test in local (maybe using a container ?) then I can try it. my current blocker is that I'm struggling to run that job in my local for debugging | 10:13 |
| tobias-urdin | tkajinam: ERROR: Can not execute `setup.py` since setuptools is not available in the build environment. | 10:20 |
| tkajinam | where did you see that error ? | 10:21 |
| tobias-urdin | if we hack the GNOCCHI_TEST_DEBUG env variable and get it to rebuild the docker image it will spit out more debug logs | 10:23 |
| tobias-urdin | can also run the action manually and check the "debug" log on that action specifically and it will set GNOCCHI_TEST_DEBUG=1, more hacky to get changes in a PR tested setting it manually | 10:25 |
| tobias-urdin | https://github.com/gnocchixyz/gnocchi/actions/runs/11160105546/job/31019896515 | 10:26 |
| tobias-urdin | annoying is that if i add setuptools it will try to install it in same transaction, it needs to be installed before the other things | 10:26 |
| tkajinam | ah, that's a good tech | 10:27 |
| tkajinam | > GNOCCHI_TEST_DEBUG | 10:27 |
| tobias-urdin | yeah we capture all output because github actions (iirc previous ci had same) only allows a certain amount of log lines or it kills the job | 10:30 |
| tkajinam | ah, ok | 10:32 |
| tkajinam | I was about to ask if we can enable it by default :-( | 10:32 |
| tobias-urdin | https://github.com/gnocchixyz/gnocchi/actions/runs/11160306162/job/31020502925 – see related PR | 10:33 |
| tobias-urdin | if you have any ideas on how to order it | 10:33 |
| tobias-urdin | "equirement already satisfied: setuptools" | 10:35 |
| tobias-urdin | must be something else or is it that it's in the same "pip install" why it fails | 10:35 |
| tobias-urdin | perhaps version is too old? | 10:36 |
| tobias-urdin | needs to be fixed in stable/4.4 to for upgrade jobs to be green – but since py39-mysql-file fails with same error need to figure out how to solve it first https://github.com/gnocchixyz/gnocchi/actions/runs/11160306162/job/31020499772 | 10:40 |
| tkajinam | I wonder if we can try installing it in all python versions. https://github.com/gnocchixyz/gnocchi/pull/1413/files | 10:41 |
| tobias-urdin | another thing could be collision between dist-packages and site-packages for setuptools | 10:43 |
| tobias-urdin | tkajinam: lets see if it works :) | 10:43 |
| tkajinam | hmmm seems not | 10:44 |
| tobias-urdin | tkajinam: your latest patch looks better | 10:57 |
| tobias-urdin | upgrade jobs is expected to fail until that is first landed in stable/4.4 where it should become green if ok, since all stable branches and master install previous version from the stable branch of that version | 10:57 |
| tobias-urdin | i think we do from version 4.4 -> current, on all stable branches | 10:58 |
| tobias-urdin | and master | 10:58 |
| tkajinam | Idk if it is picked up but let me add a commit to disable upgrade jobs | 11:01 |
| tkajinam | to run the other jobs canceled | 11:01 |
| tkajinam | I wonder if master should do upgrade from 4.5 instead of 4.4 | 11:03 |
| tobias-urdin | not sure which openstack release 4.4 was included in, but yes | 11:05 |
| tobias-urdin | might be time to eol 4.4 and change upgrade testing from 4.5 | 11:06 |
| tkajinam | yoga: | 11:06 |
| tkajinam | source-branch: stable/4.4 | 11:06 |
| tkajinam | zed: | 11:06 |
| tkajinam | source-branch: stable/4.5 | 11:06 |
| tkajinam | this is what rdopkg says | 11:06 |
| tkajinam | hmm it seems 22.04 uses 4.4 https://launchpad.net/ubuntu/+source/gnocchi | 11:07 |
| tkajinam | though they ship yoga so we might be able to ignore it now | 11:07 |
| tobias-urdin | we can drop upgrade testing from 4.4 and change to 4.5 | 11:13 |
| tobias-urdin | and just leave the 4.4 branch as-is, i can eol it when we do next major release | 11:13 |
| tobias-urdin | not sure if ubuntu has forked 4.4 or if they rely on upstream branch | 11:13 |
| tkajinam | ok | 11:17 |
| tkajinam | I've disable fail-fast to see results from all jobs. we can see if the additional env for virtualenv helps by comparing results between https://github.com/gnocchixyz/gnocchi/pull/1413 and https://github.com/gnocchixyz/gnocchi/pull/1414 | 11:18 |
| tkajinam | hmm. I'll take some more time to dig into it but it seems the issue is seen in upgrade from 4.5 but is not seen in jobs in 4.5 | 11:39 |
| tobias-urdin | sorry – i'm in the middle of something else | 11:45 |
| tobias-urdin | i will look at helping you fix 4.5 asap or first thing tomorrow | 11:45 |
| tobias-urdin | then master should be ok with your patch when 4.5 is ok and might need same patch | 11:45 |
| tkajinam | tobias-urdin, thanks. and please take your time. I already bother you too much for such gnocchi things | 11:51 |
| tobias-urdin | tkajinam: lets see if this solves CI for stable/4.5 https://github.com/gnocchixyz/gnocchi/actions/runs/11162640724 | 13:11 |
| * tobias-urdin goes afk for today | 13:11 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!