| opendevreview | Ghanshyam proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 00:26 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/tempest master: Add test for assisted volume snapshot https://review.opendev.org/c/openstack/tempest/+/864839 | 00:30 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add test for assisted volume snapshot https://review.opendev.org/c/openstack/tempest/+/864839 | 00:47 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: Test RBAC Nova service role https://review.opendev.org/c/openstack/tempest/+/864595 | 00:52 |
| opendevreview | melanie witt proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 01:08 |
| opendevreview | likui proposed openstack/tempest master: tox: Remove basepython https://review.opendev.org/c/openstack/tempest/+/890376 | 02:45 |
| opendevreview | melanie witt proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 03:51 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: Test RBAC Nova service role https://review.opendev.org/c/openstack/tempest/+/864595 | 04:10 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: Test RBAC Nova service role https://review.opendev.org/c/openstack/tempest/+/864595 | 04:30 |
| *** dmellado8 is now known as dmellado | 05:00 | |
| opendevreview | likui proposed openstack/tempest master: tox: Remove basepython https://review.opendev.org/c/openstack/tempest/+/890376 | 05:07 |
| opendevreview | melanie witt proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 09:24 |
| opendevreview | Merged openstack/devstack master: Add rocky job to gate https://review.opendev.org/c/openstack/devstack/+/885469 | 10:57 |
| opendevreview | Merged openstack/tempest master: Overlap volume and server creation in backup https://review.opendev.org/c/openstack/tempest/+/889200 | 12:49 |
| opendevreview | Merged openstack/tempest master: Avoid duplicate server creation in rebuild test https://review.opendev.org/c/openstack/tempest/+/889202 | 12:49 |
| kopecmartin | \o/ ^^^ \o/ | 12:53 |
| dansmith | zomg! | 13:31 |
| ykarel | gmann, frickler can you check https://review.opendev.org/c/openstack/devstack/+/890280 | 14:39 |
| dansmith | ykarel: I got it | 14:51 |
| ykarel | Thanks dansmith++ | 14:51 |
| opendevreview | Abhishek Kekane proposed openstack/tempest master: Remove image upload call which leads to race https://review.opendev.org/c/openstack/tempest/+/890429 | 15:02 |
| frickler | kopecmartin: needed to rebase the bookworm stack after the fedora removal, please have another look https://review.opendev.org/c/openstack/devstack/+/887547?usp=dashboard | 15:35 |
| frickler | dansmith: did you see my replies to your comments? in essence feel free to update as you see fit | 15:36 |
| dansmith | frickler: I did.. felt like a brush-off of concerns, so I just figured I'd just let someone else jump on it | 15:47 |
| opendevreview | Dan Smith proposed openstack/devstack master: Add SERVICE_REPORT_INTERVAL knob https://review.opendev.org/c/openstack/devstack/+/890439 | 16:19 |
| opendevreview | Dan Smith proposed openstack/devstack master: Add SERVICE_REPORT_INTERVAL knob https://review.opendev.org/c/openstack/devstack/+/890439 | 16:29 |
| melwitt | gmann: do you know if this is the right way to specify FORCE=yes for jobs? (under vars: devstack_localrc: FORCE: yes) when I did this it did not show it as being 'yes' in the job-output.txt https://review.opendev.org/c/openstack/tempest/+/890360/7/zuul.d/project.yaml#7 | 16:38 |
| gmann | melwitt: not sure, I have not tried that anytime. but it seems not picking it form localrc and it should run with ./stack | 16:42 |
| melwitt | gmann: gotcha, thanks | 16:42 |
| gmann | melwitt: I think doing change in devstack and add depends-on is needed | 16:43 |
| gmann | devstack stable/train | 16:43 |
| melwitt | gmann: ack, thanks! | 16:43 |
| dansmith | gmann: are you okay with this? https://review.opendev.org/c/openstack/devstack/+/890439 | 17:08 |
| dansmith | if so, I'll set that on the tempest jobs | 17:08 |
| dansmith | ykarel: I'm sure that failure is unrelated to your patch, but that's the second time I've seen that symptom on that job in the last hour | 17:21 |
| dansmith | so given it's a neutron-specific job, I wonder if you could do some triage? | 17:21 |
| dansmith | ykarel: here's the other one: https://zuul.opendev.org/t/openstack/build/36d6da389b554464af0fde183e970f38 | 17:21 |
| dansmith | any chance we're interrupting networking such that we stop being able to hit keystone briefly or something? | 17:22 |
| gmann | dansmith: checking | 17:23 |
| gmann | dansmith: commented. I am also ok with setting it directly. | 17:41 |
| dansmith | gmann: ack, happy to do that :) | 17:41 |
| gmann | dansmith: and if we set this in devstack base job then we do not need to set it in tempest job right? or you meant something different ? | 17:42 |
| dansmith | gmann: idk, I wasn't sure if tempest inherits from the devstack job, I haven't gone to look | 17:43 |
| dansmith | gmann: should we not keep the variable somewhere common so it's easy to change? perhaps not in stackrc, but somewhere else? or should I hard-code it in each of nova and cinder? | 17:45 |
| gmann | dansmith: all tempest jobs inherit from devstack-base (devstack-tempest->devstack-minimum->devstack-base) | 17:45 |
| dansmith | ack | 17:45 |
| gmann | dansmith: if you set it in devstack-base (as you are doing) it set the same for other devstackl based functional jobs too | 17:46 |
| dansmith | ack, but it sounds like the preference is no variable | 17:46 |
| gmann | dansmith: you mean to have same value for nova, cinder and other services in future? | 17:46 |
| dansmith | I'm just asking about a common location in devstack for it to be set so I'm not hard-coding magics in each place | 17:46 |
| dansmith | yeah | 17:46 |
| frickler | dansmith: gmann: https://review.opendev.org/c/openstack/tempest/+/889202 is broken, see https://112ec12279aaa8378fc1-83b0835fd19557deaa4055c10b617b58.ssl.cf5.rackcdn.com/890280/1/gate/neutron-linuxbridge-tempest/6ace2b5/testr_results.html | 17:47 |
| frickler | you can't create a volume_backed server when there's no cinder deployed | 17:47 |
| gmann | dansmith: humm, ok we can do that. I am ok with that too. having single place to change it for both service is good point | 17:47 |
| dansmith | frickler: oohh, that's the catalog failure | 17:48 |
| frickler | yes, the catalog is correct in fact | 17:48 |
| dansmith | eff, that's been like two weeks in the making | 17:48 |
| dansmith | trying to save creating a server for no reason, but ... :( | 17:48 |
| dansmith | I guess we have no jobs in tempest that don't run with cinder? | 17:49 |
| gmann | yeah we have all cinder enabled jobs. but neutron does run them i think all of them | 17:49 |
| dansmith | I didn't change any skips, how was this not already failing? | 17:50 |
| frickler | there was a failure, but it was non-voting | 17:50 |
| frickler | the tests themselves have a proper decorator | 17:50 |
| frickler | but the class setup doesnt | 17:50 |
| dansmith | we setup the class even if there's nothing to run? | 17:51 |
| gmann | dansmith: we do not check for cinder service to skip it as it is in compute test class | 17:51 |
| frickler | well running the skip decoratur is still kind of running the test I guess | 17:51 |
| gmann | we should add that in this test class skip | 17:51 |
| dansmith | ah I guess that doesn't affect discovery yeah | 17:52 |
| gmann | frickler: no, it skip at the first step itself before any creds setup | 17:52 |
| gmann | resource_setup will not be called if skip does not pass through | 17:52 |
| dansmith | gmann: so I can do this in skip_checks()? | 17:53 |
| gmann | also, neutron does not want to run cinder based test but we have mixed up the compute/cinder test in api.compute dir and it is hard to make it exclude list | 17:53 |
| frickler | anyway, this would be a good argument to make neutron-ovs-tempest-dvr voting | 17:53 |
| gmann | dansmith: yes, if you check cinder service availability then t should just skip test doing anything if no cinder | 17:53 |
| gmann | frickler: ++ | 17:53 |
| gmann | dansmith: we do that for api.compute.volume tests example https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_attach_volume.py#L36 | 17:54 |
| opendevreview | Dan Smith proposed openstack/tempest master: Skip rebuild_volume_backed_server if no cinder https://review.opendev.org/c/openstack/tempest/+/890448 | 17:55 |
| dansmith | how's that ^ / | 17:55 |
| gmann | yeah, that will work | 17:57 |
| gmann | seems like neutron-ovs-tempest-dvr job only running in tempest and I am seeing it failing many time so not sure we should make it votinghttps://zuul.openstack.org/builds?job_name=neutron-ovs-tempest-dvr&skip=0 | 17:58 |
| gmann | ykarel: slaweq do you know if you are running neutron-ovs-tempest-dvr or any job inherited from it in neutron gate ? i cannot find it. or what you say to make it voting in tempest gate ? | 17:59 |
| dansmith | just voting on check and not in gate would be enough I think | 18:00 |
| dansmith | the tempest job set is pretty big and hard to ever pass :/ | 18:00 |
| gmann | dansmith: ++ yeah at least in check if we can do it will be great | 18:01 |
| gmann | it is hard to observe all non voting jobs and all such configuration on each change. | 18:01 |
| dansmith | frickler: thanks for spotting my error there, I noticed the recheck of a similar failure earlier (by ykarel) but didn't put the two together | 18:10 |
| frickler | dansmith: np, actually it was yatin's "recheck unrelated failure" that got me looking at this, because I thought that that was a bit unspecific ;) | 18:26 |
| dansmith | yeah, I've been going after people for that lately :) | 18:27 |
| dansmith | "recheck unrelated" is the new "recheck" | 18:27 |
| frickler | +1 | 18:28 |
| opendevreview | Maxim Sava proposed openstack/tempest master: Add image task client and image tests task APIs. https://review.opendev.org/c/openstack/tempest/+/888755 | 18:35 |
| dansmith | guest kernel crash on one of the jobs of that cinder-catalog fix :( | 19:10 |
| opendevreview | Dan Smith proposed openstack/devstack master: Add SERVICE_REPORT_INTERVAL knob https://review.opendev.org/c/openstack/devstack/+/890439 | 19:21 |
| opendevreview | melanie witt proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 20:26 |
| opendevreview | melanie witt proposed openstack/tempest master: DNM test rbd retype https://review.opendev.org/c/openstack/tempest/+/890360 | 21:37 |
| gmann | kopecmartin: this is ready https://review.opendev.org/c/openstack/tempest/+/864839 | 22:54 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Skip test early to improve memory footprint and time https://review.opendev.org/c/openstack/tempest/+/890469 | 23:43 |
| gmann | kopecmartin: dansmith: melwitt: ^^ this will save keystone, neutron resource creation when test class are skipped. we do create them even when all tests in test class are skipped and do not need those resources | 23:49 |
| melwitt | gmann: nice! | 23:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!