| gmann | kopecmartin: can you check this https://review.opendev.org/c/openstack/tempest/+/870974 | 00:24 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/tempest master: Fix creation of requested creds within the same project https://review.opendev.org/c/openstack/tempest/+/871018 | 05:26 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: for test https://review.opendev.org/c/openstack/tempest/+/871019 | 05:40 |
| *** soniya is now known as soniya29|rover | 05:41 | |
| *** jpena|off is now known as jpena | 08:05 | |
| opendevreview | Lajos Katona proposed openstack/tempest master: Move test_dhcp_port_status_active from neutron-tempest-plugin https://review.opendev.org/c/openstack/tempest/+/824440 | 09:09 |
| *** soniya29|rover is now known as soniya29|rover|brb | 09:44 | |
| opendevreview | Martin Kopec proposed openstack/tempest-stress master: Job update for Antelope cycle https://review.opendev.org/c/openstack/tempest-stress/+/871062 | 09:52 |
| opendevreview | Martin Kopec proposed openstack/tempest-stress master: Job update for Antelope cycle https://review.opendev.org/c/openstack/tempest-stress/+/871062 | 09:53 |
| opendevreview | Martin Kopec proposed openstack/karma-subunit-reporter master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/karma-subunit-reporter/+/871063 | 09:56 |
| opendevreview | Martin Kopec proposed openstack/eslint-config-openstack master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/eslint-config-openstack/+/871064 | 09:58 |
| opendevreview | Martin Kopec proposed openstack/devstack-tools master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/devstack-tools/+/871066 | 10:01 |
| opendevreview | Martin Kopec proposed openstack/devstack-plugin-nfs master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/871072 | 10:10 |
| opendevreview | Merged openstack/devstack stable/zed: [OVN] Ensure socket files are absent in init_ovn https://review.opendev.org/c/openstack/devstack/+/870505 | 10:11 |
| opendevreview | Martin Kopec proposed openstack/bashate master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/bashate/+/871074 | 10:14 |
| opendevreview | Martin Kopec proposed openstack/os-performance-tools master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/os-performance-tools/+/871075 | 10:18 |
| opendevreview | Martin Kopec proposed openstack/coverage2sql master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/coverage2sql/+/871076 | 10:20 |
| *** soniya29|rover|brb is now known as soniya29|rover | 11:25 | |
| opendevreview | Martin Kopec proposed openstack/tempest-stress master: Job update for Antelope cycle https://review.opendev.org/c/openstack/tempest-stress/+/871062 | 15:03 |
| opendevreview | Martin Kopec proposed openstack/os-performance-tools master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/os-performance-tools/+/871075 | 15:07 |
| opendevreview | Dan Smith proposed openstack/tempest master: WIP: Chunked GET request support https://review.opendev.org/c/openstack/tempest/+/871000 | 15:18 |
| dansmith | gmann: gibi: ^ that now works locally for me | 15:18 |
| dansmith | I need to survey which and what places need tests added | 15:21 |
| opendevreview | Martin Kopec proposed openstack/tempest-stress master: Job update for Antelope cycle https://review.opendev.org/c/openstack/tempest-stress/+/871062 | 16:02 |
| opendevreview | Merged openstack/bashate master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/bashate/+/871074 | 16:12 |
| opendevreview | Merged openstack/devstack-tools master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/devstack-tools/+/871066 | 16:15 |
| opendevreview | Merged openstack/coverage2sql master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/coverage2sql/+/871076 | 16:23 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: for test https://review.opendev.org/c/openstack/tempest/+/871019 | 16:26 |
| opendevreview | Merged openstack/eslint-config-openstack master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/eslint-config-openstack/+/871064 | 16:26 |
| gibi | dansmith: added to my queue | 16:57 |
| dansmith | gibi: still needs tests, I'm hoping gmann can say "yes this approach is reasonable" before I spend too much time on it | 16:57 |
| dansmith | or maybe kopecmartin | 16:57 |
| gmann | dansmith: sure, I will check it in an hr. | 16:59 |
| dansmith | thx | 16:59 |
| gibi | ack, I will check it tomorrow :) | 17:05 |
| *** jpena is now known as jpena|off | 17:30 | |
| dansmith | it's passing what I expect, but looks like a bunch of other failures in other places | 17:30 |
| opendevreview | Dan Smith proposed openstack/tempest master: Chunked GET request support https://review.opendev.org/c/openstack/tempest/+/871000 | 18:47 |
| abdi | We are consistently seeing failures of volume attachment to instances in our CI. The consistent signature is the volume is successfully created, transitioned to Available, transitioned to Reserved as it is getting ready to be attached, transitioned to Attaching, but never transitioned to Attached. Has anyone else seen this behavior?? | 18:54 |
| abdi | We are testing with Master (oslo.messaging 14.1.0, Nova 26.1.0.dev141, os-brick 6.3.1, Python 3.8.10). | 18:54 |
| abdi | Correction: I should have said... Never transitioned to "In Use" instead of Attached above. | 18:57 |
| gmann | dansmith: direction looks good to me and that is right way for such large response data. one question about closing the http connection https://review.opendev.org/c/openstack/tempest/+/871000/3/tempest/api/compute/admin/test_volume.py#68 | 19:16 |
| gmann | as you tested and it worked fine I think I am missing something but want to clarify my doubt | 19:17 |
| dansmith | gmann: gah, I forgot to finish that. it doesn't break anything it just leaks a (single in this case) pool worker, but yeah | 19:18 |
| dansmith | I was just trying to get it working and never did the close, thanks for the reminder | 19:18 |
| gmann | dansmith: ok. in that case we need to download the chunked data before closing so that we can pass it to upload API | 19:19 |
| dansmith | gmann: we need to close it after we do the feed, which means either in the test, or somehow in tearDown | 19:20 |
| gmann | not sure if in teardown will work as we are making another request in same test case | 19:20 |
| dansmith | gmann: it will work, but do you mean it's not ideal? it would just leave one pool worker unable to make more requests until it's closed | 19:21 |
| gmann | dansmith: yeah. but if that does not cause any issue it is fine in teardown. because closing it test need test to download the compleet data which is again oom | 19:21 |
| gmann | issue | 19:22 |
| dansmith | gmann: the test can do it after it's done, we just have to remember | 19:22 |
| dansmith | having the test close it is the easiest, is that okay? | 19:22 |
| gmann | ok. and we can document in service client if you are requesting with chunked=True then remember to close it in test itself | 19:23 |
| dansmith | ack, I will | 19:23 |
| gmann | thanks | 19:23 |
| gmann | abdi: we have seen volume status transition error many time especially in centos stream jobs but have not looked into the exact logs for your error. good to check in nova and cinder logs where it stuck | 19:25 |
| opendevreview | Dan Smith proposed openstack/tempest master: Chunked GET request support https://review.opendev.org/c/openstack/tempest/+/871000 | 19:28 |
| dansmith | gmann: ^ | 19:28 |
| dansmith | oops, I missed your other comment | 19:29 |
| gmann | dansmith: that is ok, you can update if you re-spin it otherwise we can do in followup | 19:32 |
| dansmith | gmann: I don't think it's a huge rush, but look at my reply (when you can) - it's not so easy | 19:32 |
| gmann | dansmith: yeah, we can skip the resp data/header to log but as it need little refactoring in log_request, we can do separately. not blocker for this change | 19:35 |
| gmann | I am ok the way you prefer | 19:36 |
| dansmith | okay | 19:36 |
| opendevreview | Merged openstack/tempest-stress master: Job update for Antelope cycle https://review.opendev.org/c/openstack/tempest-stress/+/871062 | 20:02 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Fix creation of requested creds within the same project https://review.opendev.org/c/openstack/tempest/+/871018 | 21:04 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 21:09 |
| abdi | gmann: We have check cinder and nova logs. Cinder appears to do all ok. In Nova, other than Rabbit AMQP connection resets, there's nothing else that points to root cause. This is the single issue failing all of our CI runs and it does not point to our cinder driver so we're perplexed. | 21:17 |
| *** arxcruz|ruck is now known as arxcruz | 21:50 | |
| opendevreview | Dan Smith proposed openstack/tempest master: Chunked GET request support https://review.opendev.org/c/openstack/tempest/+/871000 | 22:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!