| opendevreview | Brian Rosmaita proposed openstack/devstack-plugin-ceph master: Use debian-trixie https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/976279 | 00:09 |
|---|---|---|
| *** mhen_ is now known as mhen | 02:23 | |
| frickler | stephenfin: sean-k-mooney: so devstack stable branches seem fixed for now, but master is still waiting for horizon iiuc? do we want to drop horizon as temporary workaround? it doesn't look to me like the horizon team is near to a solution | 09:15 |
| sean-k-mooney | frickler: there solution should be to stop using xstaic and just use yarn or npm or bun or any other native js tool to manage there js deps like zuul does | 11:06 |
| sean-k-mooney | frickler: but yes we can disable horizon by default in the base jobs if its not already | 11:06 |
| sean-k-mooney | many project are unblocked i think becausae they dont have horizon enabled in the gates but obvioulys devstack itslef is not one of them | 11:07 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Do not enable horizon by default https://review.opendev.org/c/openstack/devstack/+/976357 | 11:36 |
| frickler | ok, let's see whether ^^ is enough | 11:36 |
| opendevreview | Stephen Finucane proposed openstack/hacking master: Drop flake8-docstrings https://review.opendev.org/c/openstack/hacking/+/976368 | 11:50 |
| opendevreview | Stephen Finucane proposed openstack/hacking master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/hacking/+/976371 | 12:07 |
| sean-k-mooney | locally it should be | 12:10 |
| sean-k-mooney | the standard devstack job also has horizon disabled https://github.com/openstack/devstack/blob/master/.zuul.yaml#L723 | 12:10 |
| sean-k-mooney | i belvie that is ture of tempest-full-py3 as well | 12:11 |
| sean-k-mooney | so most just shoudl be passing only the ones directly testing horizon are still broken | 12:11 |
| sean-k-mooney | frickler: are you a reviewer on the requriemetn repo? any opion on https://review.opendev.org/c/openstack/requirements/+/976259 | 12:12 |
| sean-k-mooney | that backport the remove of pkg_resouces form that repo so if folks are happy with that ill backport it to the ohter stable branches to unblock it once its merged | 12:13 |
| stephenfin | sean-k-mooney: https://pypi.org/project/standard-pkg-resources/1.0.0/ | 13:16 |
| *** iurygregory_ is now known as iurygregory | 13:26 | |
| *** haleyb_ is now known as haleyb | 13:36 | |
| sean-k-mooney | ack here is my version but i have not pushed it to pypi | 13:37 |
| sean-k-mooney | https://github.com/seanMooney/oslo-pkg-resources | 13:37 |
| sean-k-mooney | so one of the questions we need to resolve is do we want to have somthing under oslo or not and in etiher case do we want to invest any time in publishing/packaigng this beyond what we have done and pulling one fo them into our repos unitl we get a resolution to the upstream setuptools converstaoin | 13:42 |
| sean-k-mooney | ^ is kind fo a tc/oslo/comunity wide topic | 13:43 |
| opendevreview | Takashi Kajinami proposed openstack/devstack master: Horizon: Use built-in manage.py https://review.opendev.org/c/openstack/devstack/+/976086 | 14:23 |
| opendevreview | Brian Rosmaita proposed openstack/devstack-plugin-ceph master: Use debian-trixie https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/976279 | 14:27 |
| frickler | looks like tempest jobs need more overriding | 15:48 |
| tkajinam | frickler, I guess enabled services are defined in the base devstack job, and we should patch it instead | 16:09 |
| sean-k-mooney | we disable all services in the base job | 16:09 |
| sean-k-mooney | and then we selectivly enable them | 16:09 |
| sean-k-mooney | horizon is not in devstack-base devstck-minimal or devstack jobs | 16:09 |
| sean-k-mooney | and i tought tempet-full-py3 also did not have it | 16:09 |
| sean-k-mooney | the stack rc defaults are intentiolly not used in the jobs for enabeld services | 16:10 |
| sean-k-mooney | thats just for local deployment | 16:10 |
| sean-k-mooney | oh no tempest-full-py3 is expclity enabling it | 16:11 |
| sean-k-mooney | https://github.com/openstack/tempest/blob/master/zuul.d/integrated-gate.yaml#L104 | 16:12 |
| tkajinam | yeah I'm looking at the same | 16:12 |
| tkajinam | https://codesearch.opendev.org/?q=horizon%3A%20true&i=nope&literal=nope&files=.*zuul.*&excludeFiles=&repos= | 16:13 |
| sean-k-mooney | nova is not runing that in check we have our own replacmeent which si hwy its now green | 16:14 |
| sean-k-mooney | our plathform jobs all inthereit form tempest-full-py3 so we need to disable it ther i think in the short term but im not sure how gmaan feels about that? | 16:16 |
| sean-k-mooney | obviously the plan is to revert that once we have a working solution ideally in the next week or two before the release period | 16:16 |
| sean-k-mooney | but that will obviously be a very challangign time frame for affect projects to reconsile with | 16:17 |
| gmaan | sean-k-mooney: tkajinam frickler I think we can disable it in tempest-full-py3 job, that was added to run a horizon base tempest test. | 16:26 |
| gmaan | I am in downstream meerting if either of you can propose it with note of enabling it later, I can approve it | 16:26 |
| gmaan | or I can propose after meeting | 16:26 |
| sean-k-mooney | i can do it now quickly | 16:27 |
| sean-k-mooney | and then submit a revert patch on top | 16:27 |
| gmaan | perfect, thanks | 16:27 |
| sean-k-mooney | oh lol i missed the eventlet sync this week oh well :) | 16:27 |
| opendevreview | sean mooney proposed openstack/tempest master: disable horizon in tempest-full-py3 https://review.opendev.org/c/openstack/tempest/+/976501 | 16:36 |
| opendevreview | sean mooney proposed openstack/tempest master: Revert "disable horizon in tempest-full-py3" https://review.opendev.org/c/openstack/tempest/+/976506 | 16:39 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add AMD Sev instance creation test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/809441 | 16:57 |
| frickler | sean-k-mooney: more fails :( https://zuul.opendev.org/t/openstack/build/25fa44503ada4d1982df5dbc9f1e81b5 | 16:59 |
| sean-k-mooney | oh i didnt run tox but looks like tempst is using flake8-import-order | 17:09 |
| sean-k-mooney | time to swap to ruff with isort.... | 17:09 |
| opendevreview | Brian Rosmaita proposed openstack/devstack-plugin-ceph master: Migrate to bindep https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/976515 | 17:10 |
| sean-k-mooney | i think ill just disbale that extion for now as well | 17:10 |
| sean-k-mooney | but let me double check fi that is the only failure | 17:10 |
| sean-k-mooney | frickler: its fixed in 0.19.0 | 17:12 |
| sean-k-mooney | https://github.com/PyCQA/flake8-import-order/blob/82fe2dbb47a69bc171e52b5d0fdd4e24d8e43a59/CHANGELOG.rst#0190-2025-06-12 | 17:12 |
| sean-k-mooney | flake8-import-order should not be limited by uc so ill see if i can figure out why thats still failing | 17:13 |
| sean-k-mooney | ah its pinned in test-requiremetns | 17:16 |
| opendevreview | sean mooney proposed openstack/tempest master: disable horizon in tempest-full-py3 https://review.opendev.org/c/openstack/tempest/+/976501 | 17:20 |
| opendevreview | sean mooney proposed openstack/tempest master: Revert "disable horizon in tempest-full-py3" https://review.opendev.org/c/openstack/tempest/+/976506 | 17:20 |
| sean-k-mooney | frickler: stephen has started fixing all the xstatic stuff so im goign to -w my patch for now | 17:32 |
| sean-k-mooney | if they get through the gate in the next day or so there is no point in turnign it off | 17:32 |
| gmaan | sean-k-mooney: thanks. I think they will take time as they are many and some of them need reviews. | 17:37 |
| gmaan | I think we can proceed with the tempest disable horizon which only impact one tempest horizon test to skip | 17:37 |
| gmaan | it will unblock the oslo.service which I am waiting for some change to merge and then release | 17:38 |
| sean-k-mooney | ok we can alwasy revert quickly | 17:39 |
| gmaan | because 'Remove unnecessary setup.cfg file' are not reviewed which are first in line before 'Bump package version' | 17:39 |
| sean-k-mooney | its currenly pending ci results so we can see | 17:39 |
| gmaan | yeah | 17:40 |
| gmaan | cool | 17:40 |
| *** dmellado47 is now known as dmellado4 | 20:20 | |
| gmaan | sean-k-mooney: tempest-tox-plugin-sanity-check is failing for some plugin installation, can you skip that also in this fix? rest all looks good https://review.opendev.org/c/openstack/tempest/+/976501/1..2 | 20:23 |
| gmaan | I will take a look into that | 20:23 |
| sean-k-mooney | ah sure | 20:24 |
| gmaan | thanks | 20:24 |
| sean-k-mooney | patrole_tempest_plugin/ | 20:26 |
| sean-k-mooney | that using pkg_resouces | 20:26 |
| sean-k-mooney | will i make tempest-tox-plugin-sanity-check non-voting or just comment it out | 20:26 |
| gmaan | it is coming from airship-tempest-plugin so I am going to make that plugin inactive | 20:27 |
| gmaan | comment it out as it will fail 100% and I will propose fix on top of your fix to see if any more plugin failing | 20:27 |
| sean-k-mooney | wasnt patrole retired | 20:27 |
| gmaan | yes, very very long back and airship-tempest-plugin still using it confirm that airship-tempest-plugin is also not active | 20:28 |
| sean-k-mooney | well its been archived for 3 years now so ya its definlty inactive https://github.com/openstack-archive/patrole | 20:33 |
| gmaan | yah, its been 7 years it has not updated so I am going to remove it from this job to install https://opendev.org/airship/tempest-plugin | 20:34 |
| opendevreview | Merged openstack/hacking master: Drop flake8-docstrings https://review.opendev.org/c/openstack/hacking/+/976368 | 20:43 |
| sean-k-mooney | damit i messed up the rebase one sec | 20:43 |
| opendevreview | sean mooney proposed openstack/tempest master: disable horizon in tempest-full-py3 https://review.opendev.org/c/openstack/tempest/+/976501 | 20:49 |
| opendevreview | sean mooney proposed openstack/tempest master: Revert "disable horizon in tempest-full-py3" https://review.opendev.org/c/openstack/tempest/+/976506 | 20:49 |
| opendevreview | sean mooney proposed openstack/tempest master: re-enable tempest-tox-plugin-sanity-check https://review.opendev.org/c/openstack/tempest/+/976545 | 20:49 |
| sean-k-mooney | ok those look sane ^ i added an extra one for the sanity check job so you can reenable that seperately form horizon | 20:51 |
| gmaan | ack, thanks | 20:53 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Mark airship/tempest-plugin inactive https://review.opendev.org/c/openstack/tempest/+/976547 | 21:03 |
| gmaan | yes | 21:07 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Mark airship/tempest-plugin inactive https://review.opendev.org/c/openstack/tempest/+/976545 | 21:07 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Only use parent RPs for selection https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/975225 | 21:14 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!