| opendevreview | melanie witt proposed openstack/devstack master: DNM test lvm filter https://review.opendev.org/c/openstack/devstack/+/892631 | 00:48 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/devstack master: Fix configuration of LVM global_filter https://review.opendev.org/c/openstack/devstack/+/892631 | 03:28 |
| melwitt | gmann, dansmith: not sure whether this will help anything but I found that the global_filter setting didn't seem to be making it into the lvm.conf ^ | 03:30 |
| opendevreview | Merged openstack/devstack master: Fix glance-remote with global venv https://review.opendev.org/c/openstack/devstack/+/892571 | 03:41 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use service role also in test_assisted_volume_snapshots https://review.opendev.org/c/openstack/tempest/+/892638 | 05:27 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use service role only to acess Assisted volume snapshots API https://review.opendev.org/c/openstack/tempest/+/892639 | 05:31 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use service role also in test_server_external_events https://review.opendev.org/c/openstack/tempest/+/892640 | 05:54 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use service role only to access Server External Events API https://review.opendev.org/c/openstack/tempest/+/892642 | 06:23 |
| opendevreview | Merged openstack/tempest master: Do not retry immediately after server fault https://review.opendev.org/c/openstack/tempest/+/890911 | 08:30 |
| opendevreview | Lukas Piwowarski proposed openstack/tempest master: Fix cleanup for volume backup tests https://review.opendev.org/c/openstack/tempest/+/890798 | 08:57 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Use assertGreater in application_credentials test https://review.opendev.org/c/openstack/tempest/+/892663 | 10:41 |
| opendevreview | Jakub Skunda proposed openstack/tempest master: Remove test duplication between tempest and neutron-tempest-plugin QuotasTest https://review.opendev.org/c/openstack/tempest/+/892677 | 14:14 |
| opendevreview | Tianqi Xiao proposed openstack/tempest master: Add domain id related config options in auth https://review.opendev.org/c/openstack/tempest/+/889664 | 14:36 |
| *** TheJulia is now known as confused | 15:34 | |
| *** confused is now known as TheJulia | 15:34 | |
| opendevreview | Rodolfo Alonso proposed openstack/tempest master: [WIP][DNM] Test fix for LP#2027605 https://review.opendev.org/c/openstack/tempest/+/888770 | 16:25 |
| melwitt | gmann, dansmith: I've been looking into lvm volume command slowness/hanging and one thing I found is that the lvm global_filter config (that's supposed to help with that) didn't appear to be getting applied in devstack. I proposed a patch to fix that, not sure if it will make a difference but it's something https://review.opendev.org/c/openstack/devstack/+/892631 | 18:02 |
| gmann | melwitt: yeah, checking that. | 18:02 |
| dansmith | melwitt: wasn't applied because the regex wasn't matching? | 18:03 |
| dansmith | that's cool, I knew we had that in there, but I guess I assumed it was working :) | 18:03 |
| dansmith | I think I even tweaked it years ago | 18:03 |
| melwitt | dansmith: I believe so. and also there seemed to be a copy pasta with the value | 18:03 |
| melwitt | yeah I saw that :) | 18:03 |
| gmann | yeah, i also did not know if it was not working. even adding in log is nice | 18:04 |
| melwitt | I assumed too but started poking around on a local devstack and found it not in lvm.conf despite the devstack log saying it added it | 18:04 |
| dansmith | ah, I guess the * -> .* is the fix? | 18:06 |
| dansmith | that's rough | 18:06 |
| dansmith | I thought this was only used for finding PVs, and the majority of what we're doing at test/runtime is creating LVs in VGs, so I'm not sure this will help | 18:07 |
| melwitt | there's also the $global_filter$filter_string but I dunno if the non existent $global_filter variable hurt anything. so it might have just been the * -> .* | 18:08 |
| dansmith | unless us doing lists of LVs is causing it to scan for any PVs that have shown up lately or something | 18:08 |
| dansmith | yeah, the empty var should be okay | 18:08 |
| melwitt | yeah, not sure about the conditions under which it scans | 18:10 |
| melwitt | but the scanning is pretty much the only thing I could find with regard to slow or hanging lvm commands | 18:10 |
| dansmith | so I just +W and then realized.. I'm not sure we ran any lvm-based jobs against that? | 18:14 |
| dansmith | melwitt: ? | 18:14 |
| melwitt | dansmith: I thought since cinder uses lvm volumes by default, any job with volume tests in it would be running lvm commands? | 18:16 |
| dansmith | oh right I guess so, I was thinking about the LVM backend job.. duh | 18:16 |
| dansmith | we're currently still trying to land a thing to unbreak glance's gate because of something we landed in devstack that didn't hit that config, so I'm feeling jumpy :) | 18:17 |
| gmann | yeah I saw it in tempest full job https://zuul.opendev.org/t/openstack/build/fc321f331fc24088aef89e2a992a4776/log/job-output.txt#21412 | 18:17 |
| gmann | kopecmartin: can you check these two, these are needed to make service APIs default to service role in nova. after nova change I am making it to access via service role only (their child patches with depends-on nova changes) https://review.opendev.org/c/openstack/tempest/+/892638 https://review.opendev.org/c/openstack/tempest/+/892640 | 18:18 |
| melwitt | dansmith: which is the glance thing? | 18:18 |
| gmann | i thikn that is merged venv rihgt? | 18:18 |
| gmann | oh it failed in check pipeline. | 18:19 |
| dansmith | melwitt: yeah the venv thing | 18:25 |
| dansmith | and it has failed many times on other things since the actual fix was ready :/ | 18:26 |
| dansmith | this last one was flakiness in glance's own functional tests though I think, so can't blame it on anyone :) | 18:26 |
| opendevreview | Merged openstack/devstack master: Fix configuration of LVM global_filter https://review.opendev.org/c/openstack/devstack/+/892631 | 19:57 |
| opendevreview | Merged openstack/tempest master: Use service role also in test_assisted_volume_snapshots https://review.opendev.org/c/openstack/tempest/+/892638 | 20:22 |
| opendevreview | Merged openstack/tempest master: Use service role also in test_server_external_events https://review.opendev.org/c/openstack/tempest/+/892640 | 20:24 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use service role only to access Server External Events API https://review.opendev.org/c/openstack/tempest/+/892642 | 21:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!