| opendevreview | Nobuhiro MIKI proposed openstack/devstack master: Disable memory_tracker and file_tracker in unstask.sh properly https://review.opendev.org/c/openstack/devstack/+/865237 | 01:39 |
|---|---|---|
| gmann | kopecmartin: meanwhile you push the fix and it get merged. can we merge this revert to unblock the gate https://review.opendev.org/c/openstack/devstack/+/874625 | 02:49 |
| gmann | we need to make tempest release also as release time is approaching | 02:49 |
| gmann | kopecmartin: ykarel: stable/wallaby and older tempest-full-py3 fix is working and ready to review but that is blocked by cirrors new version https://review.opendev.org/c/openstack/tempest/+/874704 | 03:32 |
| *** yadnesh|away is now known as yadnesh | 04:42 | |
| opendevreview | yatin proposed openstack/tempest master: Fix tempest-full-py3 for stable/ussuri to wallaby https://review.opendev.org/c/openstack/tempest/+/874704 | 06:57 |
| opendevreview | yatin proposed openstack/tempest master: Fix tempest-full-py3 for stable/ussuri to wallaby https://review.opendev.org/c/openstack/tempest/+/874704 | 06:59 |
| ykarel | Thanks gmann | 07:25 |
| *** jpena|off is now known as jpena | 08:26 | |
| *** jpena is now known as jpena|off | 09:35 | |
| *** jpena|off is now known as jpena | 09:44 | |
| opendevreview | Merged openstack/devstack master: Revert "Bump cirros version to 0.6.1" https://review.opendev.org/c/openstack/devstack/+/874625 | 09:52 |
| opendevreview | Merged openstack/tempest master: Fix tempest-full-py3 for stable/ussuri to wallaby https://review.opendev.org/c/openstack/tempest/+/874704 | 12:32 |
| opendevreview | Ade Lee proposed openstack/tempest master: Testing fips jobs https://review.opendev.org/c/openstack/tempest/+/873697 | 13:59 |
| *** yadnesh is now known as yadnesh|away | 13:59 | |
| opendevreview | Ade Lee proposed openstack/devstack master: Modify devstack-base to allow for fips https://review.opendev.org/c/openstack/devstack/+/871606 | 13:59 |
| *** haleyb_out is now known as haleyb | 14:01 | |
| opendevreview | Jorge San Emeterio proposed openstack/tempest master: Add a tempest test to verify lack of cgroupsv2 support for cpu controller. https://review.opendev.org/c/openstack/tempest/+/873706 | 14:11 |
| dansmith | gmann: uuuugh. Now that we don't wait for the attachment to complete, we race with server delete to be able to delete the volume | 14:51 |
| dansmith | I'm not really sure what to do to address that really | 14:52 |
| dansmith | because the "resources are unwound in reverse order" kinda ties our hands a bit in what we can do and when | 14:53 |
| dansmith | we could go back to always waiting, but make the wait non-fatal (and maybe shorter) | 14:53 |
| dansmith | no, no we can't... we'll still race with server delete in that case | 14:54 |
| dansmith | argh. | 14:54 |
| opendevreview | Dan Smith proposed openstack/tempest master: Log console on failure to delete attachment https://review.opendev.org/c/openstack/tempest/+/874782 | 14:57 |
| opendevreview | Dan Smith proposed openstack/tempest master: Avoid long wait for volume detach in some tests https://review.opendev.org/c/openstack/tempest/+/874700 | 15:23 |
| dansmith | gmann: here's one option ^ | 15:25 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Fix creation of requested creds within the same project https://review.opendev.org/c/openstack/tempest/+/871018 | 15:46 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use project_reader in server test https://review.opendev.org/c/openstack/tempest/+/871210 | 15:46 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add releasenote to tag the Tempest for 2023.1 release https://review.opendev.org/c/openstack/tempest/+/874357 | 15:46 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use stable constraint in tox to release new tag for 2023.1 https://review.opendev.org/c/openstack/tempest/+/874359 | 15:46 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Switch to the master constraint in tox.ini https://review.opendev.org/c/openstack/tempest/+/874360 | 15:47 |
| gmann | dansmith: humm, and can we delete the server before volume? | 15:49 |
| gmann | currently it is volume delete then server delete | 15:50 |
| dansmith | gmann: I don't think so | 15:50 |
| dansmith | gmann: we delete the server as part of the test cleanup, and the volume as part of class/resource cleanup right? | 15:50 |
| dansmith | ohhh, I guess I'm wrong | 15:51 |
| dansmith | I was reading the resource_setup() which creates a volume, | 15:51 |
| dansmith | but I guess we also create a volume in that test as well, after the server | 15:51 |
| dansmith | cripes | 15:51 |
| dansmith | gmann: okay I give up | 15:52 |
| gmann | dansmith: ah right, we use two volume in that test. | 15:53 |
| dansmith | I don't think we actually _use_ two, but we _have_ two | 15:53 |
| dansmith | but you're right, we're attaching the volume we created *after* the server | 15:54 |
| dansmith | which means we rely on the detach in order to be able to delete it, before we delete the server | 15:54 |
| gmann | ok, I saw another test. not sure why we create class level volume. so in this test if we create the volume before server then deletion order will be server delete first and then volume | 15:56 |
| dansmith | yeah | 15:56 |
| dansmith | so I can slip another patch to change that ordering in underneath the patch to not hang on detach | 15:57 |
| gmann | +1 | 15:57 |
| dansmith | ah, also, | 15:57 |
| dansmith | test_snapshot_create_offline_delete_online | 15:57 |
| dansmith | uses the class-based volume | 15:58 |
| dansmith | and creates the server after | 15:58 |
| dansmith | so I was probably not being specific enough about which test I was looking at across multiple flavors | 15:58 |
| gmann | I think we can create test level volume here when we know even single test does a lot on guest resources there is less point to make resouces sharable among tests | 15:59 |
| gmann | let's continue this after TC meeting | 15:59 |
| dansmith | yup | 16:00 |
| opendevreview | Dan Smith proposed openstack/tempest master: Avoid long wait for volume detach in some tests https://review.opendev.org/c/openstack/tempest/+/874700 | 17:26 |
| opendevreview | Dan Smith proposed openstack/tempest master: Change order of server/volume in snapshot test https://review.opendev.org/c/openstack/tempest/+/874794 | 17:26 |
| *** jpena is now known as jpena|off | 17:35 | |
| dansmith | man the gate is fuuuul | 18:07 |
| *** JasonF is now known as JayF | 19:51 | |
| dansmith | gmann: kopecmartin: is this okay to land? it will help debug volume failures: https://review.opendev.org/c/openstack/tempest/+/874782/ | 20:45 |
| dansmith | alternately we could recheck a few times and hope we get one of those failures in waiting to detach | 20:45 |
| gmann | dansmith: checking | 20:46 |
| gmann | dansmith: +2, will see if kopecmartin is around to +W otherwise will approve it after my lunch (~2 hrs from now) | 20:49 |
| dansmith | cool, thanks | 20:49 |
| gmann | kopecmartin: once you are online https://review.opendev.org/c/openstack/tempest/+/874349 | 21:21 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add RBAC new defaults and scope job for stable/zed https://review.opendev.org/c/openstack/tempest/+/874811 | 21:28 |
| dansmith | gmann: this is about to pass on the first try (since I fixed it), which is a good sign: https://review.opendev.org/c/openstack/tempest/+/874700/3 | 21:43 |
| dansmith | should we recheck that a few times to see if we can shake out a volume failure that this doesn't fix? | 21:43 |
| gmann | dansmith: nice. yeah we can recheck few times just to make sur | 21:57 |
| gmann | sure | 21:57 |
| dansmith | cool | 22:14 |
| opendevreview | Merged openstack/tempest master: Use assertLessEqual for downtime validation https://review.opendev.org/c/openstack/tempest/+/874232 | 22:45 |
| opendevreview | Ghanshyam proposed openstack/patrole master: Fix tox4 error https://review.opendev.org/c/openstack/patrole/+/874817 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!