| *** tosky has quit IRC | 00:12 | |
| *** sshnaidm is now known as sshnaidm|afk | 00:22 | |
| *** simondodsley has quit IRC | 00:45 | |
| *** simondodsley has joined #openstack-qa | 00:47 | |
| *** dave-mccowan has quit IRC | 00:57 | |
| *** elod_pto has quit IRC | 00:57 | |
| *** freerunner has quit IRC | 00:57 | |
| *** tbarron has quit IRC | 00:57 | |
| *** early has quit IRC | 00:57 | |
| *** dansmith has quit IRC | 00:57 | |
| *** irclogbot_3 has quit IRC | 00:59 | |
| *** openstackgerrit has quit IRC | 01:00 | |
| *** irclogbot_1 has joined #openstack-qa | 01:00 | |
| *** dave-mccowan has joined #openstack-qa | 01:17 | |
| *** elod_pto has joined #openstack-qa | 01:17 | |
| *** freerunner has joined #openstack-qa | 01:17 | |
| *** tbarron has joined #openstack-qa | 01:17 | |
| *** early has joined #openstack-qa | 01:17 | |
| *** dansmith has joined #openstack-qa | 01:17 | |
| *** rcernin has quit IRC | 01:56 | |
| *** rcernin has joined #openstack-qa | 01:59 | |
| *** rcernin has quit IRC | 02:12 | |
| *** rcernin has joined #openstack-qa | 02:20 | |
| *** tkajinam has quit IRC | 02:35 | |
| *** tkajinam has joined #openstack-qa | 02:35 | |
| *** dave-mccowan has quit IRC | 02:39 | |
| *** brinzhang has joined #openstack-qa | 02:41 | |
| *** dave-mccowan has joined #openstack-qa | 02:42 | |
| *** ramishra has joined #openstack-qa | 03:45 | |
| *** brinzhang_ has joined #openstack-qa | 04:12 | |
| *** brinzhang has quit IRC | 04:15 | |
| *** ianw is now known as ianw_pto | 04:22 | |
| *** rcernin has quit IRC | 04:39 | |
| *** rcernin has joined #openstack-qa | 04:39 | |
| *** rcernin has quit IRC | 05:24 | |
| *** bhagyashris is now known as bhagyashri|ruck | 05:27 | |
| *** bhagyashri|ruck is now known as bhagyashris|ruck | 05:28 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-qa | 05:33 | |
| *** rcernin has joined #openstack-qa | 05:47 | |
| *** vishalmanchanda has joined #openstack-qa | 05:56 | |
| *** soniya29 is now known as soniya29|rover | 06:10 | |
| *** gcheresh has joined #openstack-qa | 06:22 | |
| *** hamalq has joined #openstack-qa | 06:50 | |
| *** hamalq has quit IRC | 06:55 | |
| *** lifeless has quit IRC | 07:19 | |
| *** lifeless has joined #openstack-qa | 07:20 | |
| *** ccamposr has joined #openstack-qa | 07:26 | |
| *** rcernin has quit IRC | 07:26 | |
| *** ramishra has quit IRC | 07:34 | |
| *** viks____ has joined #openstack-qa | 07:43 | |
| *** slaweq has joined #openstack-qa | 07:45 | |
| *** ramishra has joined #openstack-qa | 07:57 | |
| *** ralonsoh has joined #openstack-qa | 08:02 | |
| *** ramishra has quit IRC | 08:02 | |
| *** rpittau|afk is now known as rpittau | 08:07 | |
| *** rcernin has joined #openstack-qa | 08:35 | |
| *** rcernin has quit IRC | 08:47 | |
| *** jpena|off is now known as jpena | 08:55 | |
| *** ramishra has joined #openstack-qa | 09:05 | |
| *** yamamoto has quit IRC | 09:08 | |
| *** yamamoto has joined #openstack-qa | 09:10 | |
| *** gfidente has joined #openstack-qa | 09:12 | |
| *** yamamoto has quit IRC | 09:12 | |
| *** rcernin has joined #openstack-qa | 09:12 | |
| *** rcernin has quit IRC | 09:17 | |
| *** yamamoto has joined #openstack-qa | 09:28 | |
| *** hamalq has joined #openstack-qa | 09:28 | |
| *** hamalq_ has joined #openstack-qa | 09:29 | |
| *** hamalq has quit IRC | 09:33 | |
| *** hamalq_ has quit IRC | 09:34 | |
| lyarwood | https://review.opendev.org/c/openstack/grenade/+/767569 - if anyone is still around, this could use another grenade core review. | 09:45 |
|---|---|---|
| lyarwood | https://review.opendev.org/c/openstack/tempest/+/767515 - could also use a review from some tempest cores | 09:54 |
| lyarwood | ^ both gate issues seen in Nova | 09:54 |
| *** rcernin has joined #openstack-qa | 10:00 | |
| *** tosky has joined #openstack-qa | 10:03 | |
| *** rcernin has quit IRC | 10:06 | |
| *** brinzhang_ has quit IRC | 10:16 | |
| *** brinzhang_ has joined #openstack-qa | 10:17 | |
| *** rcernin has joined #openstack-qa | 10:17 | |
| frickler | lyarwood: two comments on the grenade patch | 10:18 |
| *** rcernin has quit IRC | 10:21 | |
| *** rcernin has joined #openstack-qa | 10:36 | |
| *** yamamoto has quit IRC | 10:41 | |
| *** rcernin has quit IRC | 10:41 | |
| *** rcernin has joined #openstack-qa | 10:54 | |
| *** rcernin has quit IRC | 10:59 | |
| *** yamamoto has joined #openstack-qa | 11:09 | |
| *** rcernin has joined #openstack-qa | 11:12 | |
| *** yamamoto has quit IRC | 11:12 | |
| lyarwood | frickler: ack sorry ssh dropped, thanks for the review! | 11:14 |
| *** sshnaidm|afk is now known as sshnaidm | 11:17 | |
| *** rcernin has quit IRC | 11:17 | |
| *** yamamoto has joined #openstack-qa | 11:24 | |
| *** hamalq has joined #openstack-qa | 11:30 | |
| *** dviroel has joined #openstack-qa | 11:32 | |
| *** hamalq has quit IRC | 11:35 | |
| *** lseki has joined #openstack-qa | 11:44 | |
| *** larainema has quit IRC | 11:46 | |
| *** dtantsur|afk is now known as dtantsur | 11:47 | |
| *** yamamoto has quit IRC | 11:47 | |
| *** rfolco has joined #openstack-qa | 11:57 | |
| *** hamalq has joined #openstack-qa | 11:58 | |
| *** hamalq has quit IRC | 12:03 | |
| *** raildo has joined #openstack-qa | 12:09 | |
| *** rcernin has joined #openstack-qa | 12:14 | |
| *** rcernin has quit IRC | 12:23 | |
| *** yamamoto has joined #openstack-qa | 12:31 | |
| *** jpena is now known as jpena|lunch | 12:33 | |
| *** yamamoto has quit IRC | 12:39 | |
| *** raissa has joined #openstack-qa | 12:40 | |
| *** rcernin has joined #openstack-qa | 12:50 | |
| *** artom has joined #openstack-qa | 12:53 | |
| *** akahat|ruck is now known as akahat | 12:57 | |
| *** rcernin has quit IRC | 13:05 | |
| *** iurygregory has joined #openstack-qa | 13:16 | |
| *** paras333 has joined #openstack-qa | 13:27 | |
| *** paras333 has quit IRC | 13:27 | |
| *** jpena|lunch is now known as jpena | 13:33 | |
| *** paras333_ has joined #openstack-qa | 13:49 | |
| *** brinzhang_ has quit IRC | 13:54 | |
| *** brinzhang_ has joined #openstack-qa | 13:55 | |
| *** hamalq has joined #openstack-qa | 13:59 | |
| *** hamalq has quit IRC | 14:04 | |
| gmann | lyarwood: replied on https://review.opendev.org/c/openstack/tempest/+/767515 | 14:31 |
| gmann | lyarwood: if any test or common method doing detach then it should wait for detach to complete. that is how all async operation needs to handle other we can get race condition in other scenario also | 14:32 |
| gmann | lyarwood: do you know exact test causing this issue? | 14:32 |
| gmann | lyarwood: + A on grenade one | 14:33 |
| gmann | masayukig: thanks. I will try to check 25.0.1 tag it should work basically but may be new bug. | 14:34 |
| gmann | lyarwood: also commented on bug about two different code path which should not cause race. - https://bugs.launchpad.net/tempest/+bug/1908399 | 14:53 |
| openstack | Launchpad bug 1908399 in tempest "nova_volume_attach adds volume detach cleanup twice" [Undecided,Confirmed] - Assigned to Lee Yarwood (lyarwood) | 14:53 |
| gmann | kopecmartin: changing constraint name is not easy in Tempest as upstream CI/CD heavily rely on the old name - https://review.opendev.org/c/openstack/tempest/+/766909 | 14:57 |
| gmann | it keep coming, this is old review https://review.opendev.org/c/openstack/tempest/+/760714 | 14:57 |
| lyarwood | gmann: back, reading | 15:01 |
| lyarwood | gmann: tempest.scenario.test_encrypted_cinder_volumes.TestEncryptedCinderVolumes.test_encrypted_cinder_volumes_cryptsetup and tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario both hit this in my example test run | 15:02 |
| lyarwood | gmann: both already use nova_volume_detach that correctly waits until the volume is available | 15:04 |
| lyarwood | gmann: nova_volume_attach just duplicated the call to nova_volume_detach in an additional cleanup | 15:04 |
| lyarwood | gmann: the race here is that the cleanup in nova_volume_attach only handles a 404 when detaching the volume when we can actually get a 400 back *if* the second call to detach is sent while the volume is being detached by the first. | 15:08 |
| lyarwood | hmm actually I see your point, how do we make the second call if the first should be waiting for the volume to be available | 15:09 |
| lyarwood | ah wait, it's a race between c-api and n-api | 15:12 |
| lyarwood | now I remember | 15:12 |
| lyarwood | so the volume is already available | 15:12 |
| lyarwood | but we haven't deleted the attachment in nova | 15:12 |
| gmann | lyarwood: I see. I think we can add nova attachment wait also. this one - wait_for_volume_attachment_remove_from_server | 15:19 |
| gmann | and this is needed only on common method as all scenario tests use the common method. | 15:20 |
| gmann | https://github.com/openstack/tempest/blob/master/tempest/common/waiters.py#L320 | 15:21 |
| lyarwood | gmann: ack let me respin | 15:21 |
| gmann | thanks, I will be back after my breakfast. | 15:21 |
| lyarwood | gmann: cleanups are fifo right? | 15:21 |
| gmann | lyarwood: in reverse order. lifo | 15:22 |
| lyarwood | I always get it the wrong way around, thanks | 15:22 |
| *** raissa has quit IRC | 15:43 | |
| *** vishalmanchanda has quit IRC | 15:53 | |
| *** hamalq has joined #openstack-qa | 16:00 | |
| *** hamalq has quit IRC | 16:04 | |
| *** openstackgerrit has joined #openstack-qa | 16:05 | |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: scenario: Use wait_for_volume_attachment_remove_from_server in nova_volume_detach https://review.opendev.org/c/openstack/tempest/+/767515 | 16:05 |
| *** ramishra has quit IRC | 16:16 | |
| *** yamamoto has joined #openstack-qa | 16:37 | |
| *** yamamoto has quit IRC | 16:41 | |
| *** ramishra has joined #openstack-qa | 16:52 | |
| *** ramishra has quit IRC | 16:52 | |
| *** ramishra has joined #openstack-qa | 16:52 | |
| *** ramishra has quit IRC | 16:52 | |
| openstackgerrit | Merged openstack/grenade master: cinder: Wait for encrypted volume to be deleted before removing type https://review.opendev.org/c/openstack/grenade/+/767569 | 16:58 |
| *** hamalq has joined #openstack-qa | 17:00 | |
| *** ralonsoh has quit IRC | 17:05 | |
| *** ralonsoh has joined #openstack-qa | 17:05 | |
| *** rpittau is now known as rpittau|afk | 17:13 | |
| *** hamalq has quit IRC | 17:14 | |
| *** hamalq has joined #openstack-qa | 17:14 | |
| *** hamalq has quit IRC | 17:17 | |
| *** hamalq has joined #openstack-qa | 17:17 | |
| *** ralonsoh_ has joined #openstack-qa | 17:24 | |
| *** ralonsoh has quit IRC | 17:25 | |
| *** lseki has quit IRC | 17:25 | |
| *** afazekas has quit IRC | 17:44 | |
| *** akahat has quit IRC | 17:45 | |
| *** jpena has quit IRC | 17:45 | |
| *** jpena has joined #openstack-qa | 17:48 | |
| *** akahat has joined #openstack-qa | 17:55 | |
| *** jpena is now known as jpena|off | 18:01 | |
| *** ralonsoh_ has quit IRC | 18:01 | |
| *** gfidente is now known as gfidente|afk | 18:02 | |
| *** dtantsur is now known as dtantsur|afk | 18:15 | |
| *** gcheresh has quit IRC | 18:15 | |
| *** sean-k-mooney has joined #openstack-qa | 18:27 | |
| *** sean-k-mooney2 has quit IRC | 18:28 | |
| gmann | masayukig: seems like we have not capped the API extensions for victoria stable - https://github.com/openstack/devstack/blob/stable/victoria/lib/tempest#L658 | 18:31 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Remove stable/stein testing jobs https://review.opendev.org/c/openstack/tempest/+/768118 | 18:43 |
| openstackgerrit | Ghanshyam proposed openstack/patrole master: Remove stable/stein testing jobs https://review.opendev.org/c/openstack/patrole/+/768119 | 18:44 |
| *** ccamposr__ has joined #openstack-qa | 18:55 | |
| gmann | kopecmartin: ^^ | 18:55 |
| *** ccamposr has quit IRC | 18:58 | |
| *** rcernin has joined #openstack-qa | 19:01 | |
| *** rcernin has quit IRC | 19:06 | |
| openstackgerrit | Lee Yarwood proposed openstack/devstack stable/train: Stop installing test-requirements with projects https://review.opendev.org/c/openstack/devstack/+/766622 | 19:10 |
| openstackgerrit | Radosław Piliszek proposed openstack/devstack stable/train: Stop installing test-requirements with projects https://review.opendev.org/c/openstack/devstack/+/766622 | 19:26 |
| lyarwood | /o\ whops that's for the quick fix there | 19:31 |
| lyarwood | thanks* | 19:31 |
| *** raissa has joined #openstack-qa | 19:47 | |
| gmann | masayukig: left comment in this, need to update network extensions - https://review.opendev.org/c/openstack/devstack/+/755415 | 19:47 |
| *** yoctozepto has quit IRC | 19:55 | |
| *** yoctozepto has joined #openstack-qa | 19:55 | |
| *** raissa has quit IRC | 20:54 | |
| kopecmartin | gmann: fyi https://review.opendev.org/q/topic:%22tempest_manager%22+(status:open%20OR%20status:merged) i put there reviews to delete deprecated tempest.manager.py | 20:58 |
| *** rcernin has joined #openstack-qa | 21:19 | |
| *** paras333_ has quit IRC | 21:30 | |
| *** gcheresh has joined #openstack-qa | 21:30 | |
| *** tkajinam_ has joined #openstack-qa | 21:31 | |
| *** masayukig has quit IRC | 21:32 | |
| *** TheJulia has quit IRC | 21:33 | |
| *** hamalq has quit IRC | 21:34 | |
| *** masayukig has joined #openstack-qa | 21:34 | |
| *** TheJulia has joined #openstack-qa | 21:34 | |
| *** tkajinam has quit IRC | 21:34 | |
| *** hamalq has joined #openstack-qa | 21:34 | |
| *** rcernin has quit IRC | 21:38 | |
| *** nweinber has joined #openstack-qa | 21:40 | |
| *** nweinber has quit IRC | 21:40 | |
| gmann | kopecmartin: thanks, will check. | 21:53 |
| gmann | kopecmartin: this will be quick https://review.opendev.org/c/openstack/tempest/+/767515 | 21:54 |
| *** yamamoto has joined #openstack-qa | 22:00 | |
| *** gcheresh has quit IRC | 22:02 | |
| *** yamamoto has quit IRC | 22:30 | |
| *** yamamoto has joined #openstack-qa | 22:30 | |
| *** rcernin has joined #openstack-qa | 22:52 | |
| *** rcernin has quit IRC | 22:52 | |
| *** rcernin has joined #openstack-qa | 22:52 | |
| openstackgerrit | Merged openstack/tempest master: Also calculate external subnets when check cidr whether in use https://review.opendev.org/c/openstack/tempest/+/763621 | 22:55 |
| openstackgerrit | Merged openstack/devstack stable/train: Stop installing test-requirements with projects https://review.opendev.org/c/openstack/devstack/+/766622 | 22:56 |
| *** rpittau|afk has quit IRC | 23:21 | |
| *** johnsom has quit IRC | 23:21 | |
| *** flaviof has quit IRC | 23:22 | |
| *** TheJulia has quit IRC | 23:22 | |
| *** jamespage has quit IRC | 23:22 | |
| *** dpaterson has quit IRC | 23:22 | |
| *** vdrok has quit IRC | 23:22 | |
| *** masayukig has quit IRC | 23:23 | |
| *** tosky has quit IRC | 23:41 | |
| *** redrobot has quit IRC | 23:43 | |
| openstackgerrit | Masayuki Igawa proposed openstack/devstack master: Cap stable/victoria network, swift, volume api_extensions for tempest https://review.opendev.org/c/openstack/devstack/+/768133 | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!