*** mhen_ is now known as mhen | 02:54 | |
opendevreview | yatin proposed openstack/devstack stable/2023.1: [Stable/Unmaintained Only] Pin to last compatible tag for py3.8 https://review.opendev.org/c/openstack/devstack/+/933704 | 07:09 |
---|---|---|
opendevreview | yatin proposed openstack/tempest master: Use stable branch upper-constraints till 2023.1 https://review.opendev.org/c/openstack/tempest/+/933716 | 10:36 |
ykarel | gmann, kopecmartin pushed ^ for py3.8 thing | 12:22 |
ykarel | both needs each other to pass | 12:22 |
ykarel | test job passes with both together https://review.opendev.org/c/openstack/neutron/+/933705 | 12:23 |
ykarel | can you check, may be can get 1 force merged if direction is ok or have to do some non-voting dance | 12:24 |
ykarel | frickler, ^ | 12:26 |
ykarel | kopecmartin, i see you rechecked devstack patch, but that will not pass without the tempest one, as said above both need each other | 12:38 |
ykarel | ironic-tempest-bios-ipmi-direct-tinyipa https://zuul.opendev.org/t/openstack/build/2787a42844f0499da99e54769bf56b6a : FAILURE in 30m 00s | 12:38 |
kopecmartin | ah, the node failure confused me | 12:40 |
ykarel | yeap that was non-voting | 12:40 |
opendevreview | yatin proposed openstack/devstack stable/2023.1: [Stable/Unmaintained Only] Pin to last compatible tag for py3.8 https://review.opendev.org/c/openstack/devstack/+/933704 | 13:42 |
opendevreview | Jason Grosso proposed openstack/tempest master: [WIP]test_boot_nova_vm_unadressed_port https://review.opendev.org/c/openstack/tempest/+/932919 | 18:01 |
opendevreview | yatin proposed openstack/tempest master: Use stable branch upper-constraints till 2023.1 https://review.opendev.org/c/openstack/tempest/+/933716 | 18:03 |
opendevreview | Ghanshyam proposed openstack/tempest master: Make tempest-full-2023-1 job non voting https://review.opendev.org/c/openstack/tempest/+/933770 | 18:35 |
opendevreview | Ghanshyam proposed openstack/tempest master: End support for stable/2023.1 https://review.opendev.org/c/openstack/tempest/+/933771 | 18:35 |
opendevreview | Ghanshyam proposed openstack/tempest master: End support for stable/2023.1 https://review.opendev.org/c/openstack/tempest/+/933771 | 18:40 |
opendevreview | Ghanshyam proposed openstack/tempest master: Use stable branch upper-constraints till 2023.1 https://review.opendev.org/c/openstack/tempest/+/933716 | 18:46 |
opendevreview | Ghanshyam proposed openstack/tempest master: Pin stable/2024.2 constraint in tox to release 42.0.0 https://review.opendev.org/c/openstack/tempest/+/933772 | 18:57 |
opendevreview | Ghanshyam proposed openstack/tempest master: Switch back the tox constraint to master https://review.opendev.org/c/openstack/tempest/+/933773 | 18:59 |
gmann | kopecmartin: I have prepared the series for end of support of stable/2023.1 https://review.opendev.org/q/topic:%222023-1-last%22 | 19:00 |
gmann | this is needed to fix the stable/2023.1 gate as focal job there is blocked by the py3.8 drop | 19:00 |
kopecmartin | thanks gmann .. this https://review.opendev.org/c/openstack/tempest/+/933716/ needs to be merged before or after the pinning process is done (933772 and 933773 changes)? | 19:57 |
gmann | kopecmartin: ah you are right, let me rebase correctly. thanks | 19:58 |
opendevreview | Ghanshyam proposed openstack/tempest master: Pin stable/2024.2 constraint in tox to release 42.0.0 https://review.opendev.org/c/openstack/tempest/+/933772 | 19:58 |
opendevreview | Ghanshyam proposed openstack/tempest master: Switch back the tox constraint to master https://review.opendev.org/c/openstack/tempest/+/933773 | 19:58 |
gmann | kopecmartin: ^^ done | 19:58 |
kopecmartin | perfect, thank you | 19:58 |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily at 20:30 utc (half an hour from now) to apply a configuration change | 20:03 | |
kopecmartin | uff, so let's add a couple of +w before that ^^ so that gates are busy in the meantime | 20:03 |
sean-k-mooney | gmann: the runtime error is happeing in a number fo other jobs too | 20:11 |
sean-k-mooney | for example nova-tox-functional-py312 | 20:11 |
sean-k-mooney | it does not alwasy causes the job to fail | 20:12 |
sean-k-mooney | https://tinyurl.com/bdz95hhb | 20:12 |
sean-k-mooney | this feels like a possibe regression in the recent eventlet release | 20:13 |
sean-k-mooney | im goign to try and down grade eventlet locally and see if the behaivor changes | 20:13 |
sean-k-mooney | hum no diffent perhapse there was a sqlachmey release recently | 20:17 |
sean-k-mooney | 2 weeks ago unlikely but maybe related | 20:18 |
sean-k-mooney | we are using SQLAlchemy==2.0.32 currently so that actully 4 release beind the latest release | 20:19 |
JayF | the bump is waiting in requirements repo | 20:22 |
JayF | just got rebased | 20:22 |
JayF | https://review.opendev.org/c/openstack/requirements/+/924547 to .36 | 20:23 |
sean-k-mooney | JayF: have you seen this error before https://paste.opendev.org/show/bGPgfURx1cZYOsgmtDyw/ | 20:23 |
JayF | yes, but only in cases with insufficient monkey patching | 20:23 |
sean-k-mooney | im not sure if its sqlachmey related, evenlet related or soemthign else | 20:23 |
gmann | sean-k-mooney: yeah, there is no recent change in SQLAlchemy version | 20:23 |
sean-k-mooney | ok well i can look at the monkey patching | 20:24 |
JayF | sean-k-mooney: I'd worry more about oslo.messaging tbh | 20:24 |
JayF | but I don't know if there's any release/patch that would point to it | 20:24 |
sean-k-mooney | watcher mimiced nova and monkey patches in the cmd module __init__.py https://github.com/openstack/watcher/blob/master/watcher/cmd/__init__.py#L20 | 20:26 |
JayF | is https://github.com/openstack/watcher/blob/master/watcher/__init__.py not going to be executed first? | 20:26 |
sean-k-mooney | https://github.com/openstack/watcher/blob/master/watcher/__init__.py is mostly empty | 20:26 |
JayF | that's a legit question, I'm not sure | 20:26 |
JayF | mostly empty is not empty though | 20:26 |
sean-k-mooney | it will btu its not importing anythign that would break things | 20:26 |
sean-k-mooney | so the monky patching looks sane but that does not mean there isnt a bug | 20:27 |
JayF | yeah, I saw similar on IPA when we used to selectively monkey patch | 20:30 |
sean-k-mooney | there are hit for that message agaisnt nova cyborg and mangnu but i think only in fucntional tests | 20:31 |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to apply a configuration change | 20:31 | |
sean-k-mooney | so this makes me uncomfortabel https://zuul.opendev.org/t/openstack/build/d71f5a939e474f87ba535be9f08f4256/logs passed | 20:39 |
sean-k-mooney | btu the traceback is presetn in https://zuul.opendev.org/t/openstack/build/d71f5a939e474f87ba535be9f08f4256/log/controller/logs/screen-magnum-cond.txt#756 | 20:39 |
sean-k-mooney | i have nova is running fine on the same host by the way, i can live migrate create vms ectra so evenlet and sqlachme are not genericlly broken | 20:45 |
sean-k-mooney | oh | 20:52 |
sean-k-mooney | so i went to latest eventlet and sqlachmey adn File "/opt/stack/data/venv/lib/python3.12/site-packages/oslo_log/pipe_mutex.py", line 82, in acquire | 20:55 |
sean-k-mooney | showed up in the trace | 20:55 |
sean-k-mooney | i wonder if this si related to the debug option that turns fo the multi socket reader/write protection | 20:57 |
sean-k-mooney | that for tomorrow but in onecase its failign here https://github.com/openstack/oslo.db/blob/078a67bb2ef406248536fef46b7cba49e06a0143/oslo_db/sqlalchemy/engines.py#L52 | 20:59 |
sean-k-mooney | and in the other when logging so i feel like there is a mix of patched and unpathced thread at work | 20:59 |
frickler | sean-k-mooney: 2 weeks at most is how long opensearch has data. so this message might well have been happening for much longer | 21:23 |
opendevreview | Merged openstack/tempest master: End support for stable/2023.1 https://review.opendev.org/c/openstack/tempest/+/933771 | 22:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!