| opendevreview | Dr. Jens Harbott proposed openstack/grenade master: Disable using GLOBAL_VENV for now https://review.opendev.org/c/openstack/grenade/+/880266 | 06:00 |
|---|---|---|
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 06:01 |
| frickler | dansmith: somehow dbcounter pip install is failing on bullseye with the global_venv patch, maybe you can take a look https://zuul.opendev.org/t/openstack/build/5f2f9230eebb48fbabb2de903da7c324 I've disabled it for that job in the latest PS for now | 06:55 |
| frickler | kopecmartin: next step for patrole retirement could be approved now https://review.opendev.org/c/openstack/patrole/+/880012 /me isn't core on that | 08:37 |
| opendevreview | Merged openstack/patrole master: Retire patrole https://review.opendev.org/c/openstack/patrole/+/880012 | 08:42 |
| kopecmartin | thanks frickler, +2 +w | 08:42 |
| kopecmartin | oh, that was quick :D | 08:42 |
| kopecmartin | i also updated this https://wiki.openstack.org/wiki/QA#Projects | 08:43 |
| lajoskatona | kopecmartin: Hi, shall I disturb you with a grenade bug? This one: https://bugs.launchpad.net/neutron/+bug/2015065 | 09:13 |
| lajoskatona | kopecmartin: the issue mostly coming with security-group-rule create but rarely with other calls (test_add_remove_fixed_ip, or create_server) and we started to check it as Neutron bug but actually these tempest tests are from compute group and calling the legacy os-security-group-rules API of compute for example, and even these calls are not visible in the n-api log for example | 09:24 |
| lajoskatona | kopecmartin: so my first question is shall we change these tests instead to call the Neutron API, or deprecated these tests and use only the tests that call the Neutron API? | 09:26 |
| * kopecmartin checking and loading the context | 09:28 | |
| lajoskatona | kopecmartin: thanks | 09:43 |
| frickler | kopecmartin: noop jobs run pretty fast usually, yes ;) | 09:52 |
| frickler | lajoskatona: are there matching tempest.api.network.security_groups checks? if yes, I'd say the compute checks should be skipped except for legacy clouds or maybe even dropped completely in tempest master | 09:54 |
| lajoskatona | frickler: yes there are 2 modules under network: test_security_groups & test_security_groups_negative, as I see there's some overlap between the 2 (I mean what is under api/compute and what is under api/network) | 10:58 |
| opendevreview | Merged openstack/grenade stable/yoga: Don't enable the dstat service in CI jobs https://review.opendev.org/c/openstack/grenade/+/873956 | 11:15 |
| lajoskatona | frickler, kopecmartin: I quickly checked the relate tempest tests and under api/compute/security_groups there's more and from wider use cases, like create a server attach sg and try to delete it and check the raised exception. | 11:36 |
| lajoskatona | frickler, kopecmartin: so what should be discussed is to change these tests to use Neutron API instead of old/legacy Nova API (the same is true for other legacy APIs like floaing IP....) | 11:37 |
| lajoskatona | frickler, kopecmartin: but that is a longer thing and I still don't see why there's urllib3.exceptions.ReadTimeoutError during grenade executions recenlty, and by opensearch no similar issue for other jobs (see the links in the LP as opensearch links are not friendly with IRC) | 11:39 |
| opendevreview | Merged openstack/grenade stable/xena: Don't enable the dstat service in CI jobs https://review.opendev.org/c/openstack/grenade/+/873957 | 13:20 |
| opendevreview | Jorge San Emeterio proposed openstack/tempest master: DNM: Give more time to the detach operation https://review.opendev.org/c/openstack/tempest/+/880306 | 13:33 |
| dansmith | frickler: ugh, yeah will look | 13:34 |
| opendevreview | Merged openstack/grenade stable/wallaby: Don't enable the dstat service in CI jobs https://review.opendev.org/c/openstack/grenade/+/873958 | 13:34 |
| dansmith | frickler: hmm, it seems to work on jammy | 13:46 |
| dansmith | frickler: that error seems really weird to me and I wonder if it's something that has been changed since python 3.9 | 13:46 |
| dansmith | er, I guess not because it also seems okay on focal | 13:50 |
| dansmith | ...but also works on lunar, which has the "managed" flag | 13:52 |
| frickler | dansmith: yes, it is weird, it also works without the global venv on bullseye. it also works with the venv on bookworm. so maybe don't worry about it too much, then | 13:58 |
| dansmith | it's like, the least-offensive python module possible, so it's hard to understand how we could be breaking rules :) | 13:58 |
| dansmith | the attributeerror is actually in setuptools code, so perhaps it's just a problem in the setuptools in bullseye and we could/should update setuptools in the venv? | 13:59 |
| dansmith | This makes it look like a combination of pip, setuptools versions and --system-site-packages is the culprit: https://github.com/pypa/setuptools/issues/3240#issuecomment-1086828146 | 14:05 |
| dansmith | which maybe supports the position of one of those needing an update | 14:05 |
| dansmith | the original bug mentions bullseye specifically but goes on to replicate it in other places with specific versions | 14:05 |
| kopecmartin | gmann: it seems that we use in tempest an old compute's network api https://bugs.launchpad.net/neutron/+bug/2015065 as lajoskatona pointed out earlier today | 15:35 |
| kopecmartin | i wonder whether we should get rid of this import https://opendev.org/openstack/tempest/src/branch/master/tempest/api/compute/security_groups/test_security_group_rules.py#L16 | 15:36 |
| gmann | kopecmartin: thanks for updating wiki, I sent it on ML also. | 17:04 |
| gmann | kopecmartin: lajoskatona: give me sometime, I will check grenade things today | 17:05 |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Add option to install everything in global venvs https://review.opendev.org/c/openstack/devstack/+/558930 | 18:21 |
| frickler | dansmith: I tested locally and it seems upgrading pip in the venv does help, let's see how this goes now | 18:22 |
| dansmith | frickler: ah cool | 18:22 |
| gmann | kopecmartin: lajoskatona : we can use the neutron client which should be our direction but compute API even that is deprecated should work unless anything broke them | 21:09 |
| gmann | kopecmartin: frickler: can you check these hacking open reviews and once they are merged then we can release new version https://review.opendev.org/q/project:openstack/hacking+status:open | 21:11 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Run scenario test in parallel in tempest-full-py3 job https://review.opendev.org/c/openstack/tempest/+/880393 | 21:55 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Run scenario test in parallel in tempest-full-py3 job https://review.opendev.org/c/openstack/tempest/+/880393 | 22:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!