| opendevreview | Cyril Roelandt proposed openstack/glance-specs master: Add support for flake8 https://review.opendev.org/c/openstack/glance-specs/+/973033 | 00:39 |
|---|---|---|
| opendevreview | Cyril Roelandt proposed openstack/glance-specs master: Add support for doc8 https://review.opendev.org/c/openstack/glance-specs/+/973034 | 00:39 |
| opendevreview | Merged openstack/glance master: Add glance/wsgi/__init__.py https://review.opendev.org/c/openstack/glance/+/971241 | 01:15 |
| *** mhen_ is now known as mhen | 02:57 | |
| opendevreview | Ivan Anfimov proposed openstack/glance stable/2025.1: Add glance/wsgi/__init__.py https://review.opendev.org/c/openstack/glance/+/973035 | 05:11 |
| opendevreview | Ivan Anfimov proposed openstack/glance stable/2025.2: Add glance/wsgi/__init__.py https://review.opendev.org/c/openstack/glance/+/973036 | 05:11 |
| opendevreview | Takashi Kajinami proposed openstack/glance_store master: Use ServiceCatalog class for identity v3 API https://review.opendev.org/c/openstack/glance_store/+/973041 | 05:53 |
| opendevreview | Takashi Kajinami proposed openstack/glance_store master: Use ServiceCatalog class for identity v3 API https://review.opendev.org/c/openstack/glance_store/+/973041 | 05:57 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use ServiceCatalog class for identity v3 API https://review.opendev.org/c/openstack/glance/+/973044 | 05:59 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use ServiceCatalog class for identity v3 API https://review.opendev.org/c/openstack/glance/+/973044 | 10:34 |
| abhishekk | croelandt: Looks like some issue with upstream promote-openstack-specs jobs, specs are merged but it does not show in master | 12:51 |
| abhishekk | https://review.opendev.org/c/openstack/glance-specs/+/971188 e.g. this one | 12:51 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use ServiceCatalog from python-keystoneclient. https://review.opendev.org/c/openstack/glance/+/973083 | 12:52 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use ServiceCatalog from python-keystoneclient. https://review.opendev.org/c/openstack/glance/+/973083 | 13:40 |
| frickler | abhishekk: croelandt: that job ran before we refreshed the AFS secret. do you have another change to merge on that repo? otherwise we could maybe rerun that buildset (you cannot do it via the UI) https://zuul.opendev.org/t/openstack/build/7ceb5dbb12f24daa94f6cd1228414958 | 14:02 |
| abhishekk | frickler: let me check | 14:03 |
| abhishekk | we have couple of changes, but gate is failing on that, I have added recheck, if it works then croelandt can merge it | 14:04 |
| croelandt | hm seems to me the changes are available in https://opendev.org/openstack/glance-specs but not on the GitHub mirror | 14:06 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use ServiceCatalog from python-keystoneclient. https://review.opendev.org/c/openstack/glance/+/973083 | 14:19 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove remaining logic for identity v1/v2 API https://review.opendev.org/c/openstack/glance/+/973102 | 14:19 |
| frickler | ah, github mirroring was affected too until earlier today and the same holds there. but I see the CI still failing without logs, I need to dig into zuul to investigate further https://zuul.opendev.org/t/openstack/build/554af2ba47c44614b993f3346c0a4250 | 14:27 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Replace ServiceCatalog from python-keystoneclient https://review.opendev.org/c/openstack/glance/+/973083 | 14:27 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove remaining logic for identity v1/v2 API https://review.opendev.org/c/openstack/glance/+/973102 | 14:27 |
| frickler | abhishekk: clarkb: openstack-tox-py37 is trying to run on bionic, which afaict is no longer supported. any chance you could switch to use some reasonably modern python instead? | 14:34 |
| abhishekk | croelandt: ^^^ | 14:36 |
| frickler | oh, wrong tab completion, sorry :D | 14:37 |
| croelandt | frickler: on what project? | 14:40 |
| croelandt | on ok glance-specs | 14:41 |
| frickler | croelandt: glance-specs, see https://zuul.opendev.org/t/openstack/build/7ceb5dbb12f24daa94f6cd1228414958 | 14:42 |
| croelandt | not sure why we even run that | 14:42 |
| croelandt | hm ok our test does nothing | 14:54 |
| * croelandt is looking into cleaning all this | 14:54 | |
| opendevreview | Cyril Roelandt proposed openstack/glance-specs master: Update Python3 from 3.7 to 3.13 for our CI https://review.opendev.org/c/openstack/glance-specs/+/973108 | 15:07 |
| croelandt | abhishekk: ^ this should be a good first step | 15:08 |
| abhishekk | croelandt: ack | 15:08 |
| croelandt | frickler: ok, we have a modern setup now \o/ | 15:26 |
| frickler | ack, that should also fix the github sync as a side effect :) feel free to ping me if you notice further issues | 15:33 |
| opendevreview | Merged openstack/glance-specs master: Update Python3 from 3.7 to 3.13 for our CI https://review.opendev.org/c/openstack/glance-specs/+/973108 | 15:34 |
| abhishekk | ^^ this fixed the issue I mentioned earlier, thanks frickler and croelandt | 15:38 |
| croelandt | I don't understand how this prevented the mirror from being updated | 15:39 |
| frickler | croelandt: it didn't, the mirror update was broken after the zuul incident last week, we needed to rotate secrets which happened earlier today. so now just any other merge was needed to trigger the mirror process once more | 15:44 |
| croelandt | oh I see | 15:47 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Drop unused logic to detect management_url https://review.opendev.org/c/openstack/glance/+/973112 | 16:01 |
| croelandt | abhishekk: I'm reading https://review.opendev.org/c/openstack/glance/+/972643 | 16:09 |
| croelandt | Now I can't remember why FunctionalTest relied on eventlet | 16:09 |
| croelandt | MinimalIPv6TestBase looks so much like FunctionalTest that I'm worried we might bring back a dependency on eventlet | 16:09 |
| abhishekk | no it's not dependent on eventlet any more | 16:39 |
| abhishekk | croelandt: neither of it is using eventlet | 16:41 |
| croelandt | But FunctionalTest is no longer used after that patch, right? | 16:42 |
| abhishekk | no | 16:43 |
| abhishekk | That's was our goal, remove functional test which was using eventlet, right? | 16:45 |
| abhishekk | FYI, it was using from glance.common import wsgi which uses eventlet | 16:46 |
| croelandt | abhishekk: ok so see my review | 16:52 |
| croelandt | I just wonder whether we should mention FunctionalTest at all | 16:52 |
| croelandt | since I think there will be one more patch cleaning up all references to it | 16:53 |
| abhishek_ | there is already | 16:57 |
| abhishek_ | https://review.opendev.org/c/openstack/glance/+/972822/1 | 16:57 |
| abhishek_ | this one | 16:57 |
| abhishek_ | and I have removed that wording in this patch | 16:59 |
| croelandt | oh ok | 17:03 |
| croelandt | ok I missed the second patch, hence my confusion :) | 17:03 |
| abhishek_ | no problem, there is 3rd one as well, but it's independent | 17:08 |
| abhishek_ | https://review.opendev.org/c/openstack/glance/+/972826 :) | 17:08 |
| opendevreview | Merged openstack/glance master: Use Ubuntu Noble to run py312 job https://review.opendev.org/c/openstack/glance/+/970053 | 17:18 |
| opendevreview | Merged openstack/glance master: Migrate py313 job to Debian Trixie https://review.opendev.org/c/openstack/glance/+/970054 | 17:18 |
| opendevreview | Merged openstack/glance master: Migrate functional tests to SynchronousAPIBase https://review.opendev.org/c/openstack/glance/+/972643 | 19:13 |
| opendevreview | Merged openstack/glance master: Remove unused functional test classes https://review.opendev.org/c/openstack/glance/+/972822 | 19:14 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!