| *** yamamoto has joined #openstack-qa | 00:26 | |
| *** swp20 has joined #openstack-qa | 00:28 | |
| -openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being quickly restarted to apply a new security patch | 00:57 | |
| *** songwenping_ has joined #openstack-qa | 01:07 | |
| *** swp20 has quit IRC | 01:10 | |
| *** rcernin has quit IRC | 01:44 | |
| *** dviroel has quit IRC | 02:04 | |
| *** rcernin has joined #openstack-qa | 02:09 | |
| *** hamalq has quit IRC | 02:36 | |
| *** paras333 has quit IRC | 02:41 | |
| *** brinzhang has joined #openstack-qa | 03:15 | |
| *** yamamoto has quit IRC | 03:26 | |
| *** yamamoto_ has joined #openstack-qa | 03:26 | |
| *** irclogbot_3 has quit IRC | 03:27 | |
| *** ricolin_ has joined #openstack-qa | 04:13 | |
| *** yamamoto has joined #openstack-qa | 04:26 | |
| *** yamamoto_ has quit IRC | 04:29 | |
| *** zenkuro has joined #openstack-qa | 04:31 | |
| *** ramishra has quit IRC | 04:40 | |
| *** irclogbot_3 has joined #openstack-qa | 04:42 | |
| *** irclogbot_3 has quit IRC | 04:54 | |
| *** irclogbot_0 has joined #openstack-qa | 04:59 | |
| *** ramishra has joined #openstack-qa | 05:04 | |
| *** vishalmanchanda has joined #openstack-qa | 05:07 | |
| *** jparker_ has quit IRC | 05:27 | |
| *** ricolin_ has quit IRC | 05:35 | |
| *** ricolin has joined #openstack-qa | 05:39 | |
| openstackgerrit | Archit Modi proposed openstack/whitebox-tempest-plugin master: Introduce volume encryption tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/771283 | 06:14 |
|---|---|---|
| *** lbragstad_ has joined #openstack-qa | 06:24 | |
| *** lbragstad has quit IRC | 06:24 | |
| *** ralonsoh has joined #openstack-qa | 06:49 | |
| *** songwenping__ has joined #openstack-qa | 06:54 | |
| *** gcheresh has joined #openstack-qa | 06:55 | |
| *** slaweq has joined #openstack-qa | 06:55 | |
| *** lpiwowar has joined #openstack-qa | 06:56 | |
| *** songwenping_ has quit IRC | 06:57 | |
| *** sboyron has joined #openstack-qa | 07:02 | |
| *** ccamposr__ has joined #openstack-qa | 07:08 | |
| *** ccamposr has quit IRC | 07:10 | |
| *** sboyron_ has joined #openstack-qa | 07:19 | |
| *** sboyron has quit IRC | 07:22 | |
| *** rcernin has quit IRC | 07:37 | |
| *** slaweq has quit IRC | 07:40 | |
| *** slaweq has joined #openstack-qa | 07:42 | |
| *** ralonsoh has quit IRC | 07:54 | |
| *** ralonsoh has joined #openstack-qa | 08:01 | |
| *** ralonsoh has quit IRC | 08:02 | |
| *** ralonsoh has joined #openstack-qa | 08:05 | |
| openstackgerrit | Merged openstack/devstack stable/stein: Fix get-pip for python 3.5 https://review.opendev.org/c/openstack/devstack/+/772546 | 08:07 |
| *** hemanth_n has joined #openstack-qa | 08:12 | |
| *** rcernin has joined #openstack-qa | 08:14 | |
| *** rpittau|afk is now known as rpittau | 08:25 | |
| *** rcernin has quit IRC | 08:31 | |
| *** dtantsur|afk is now known as dtantsur | 08:36 | |
| *** gfidente has joined #openstack-qa | 08:44 | |
| *** kopecmartin has quit IRC | 08:48 | |
| *** kopecmartin has joined #openstack-qa | 08:50 | |
| *** lxkong has quit IRC | 08:52 | |
| *** lxkong has joined #openstack-qa | 08:53 | |
| *** lxkong has quit IRC | 08:53 | |
| *** lxkong has joined #openstack-qa | 08:54 | |
| *** paras333 has joined #openstack-qa | 08:55 | |
| *** jpena|off is now known as jpena | 08:56 | |
| *** brinzhang has quit IRC | 08:56 | |
| openstackgerrit | Carlos Gonçalves proposed openstack/devstack master: Add CentOS Stream 8 CI test coverage https://review.opendev.org/c/openstack/devstack/+/773614 | 08:57 |
| *** brinzhang has joined #openstack-qa | 08:59 | |
| *** paras333 has quit IRC | 08:59 | |
| *** brinzhang has quit IRC | 08:59 | |
| *** brinzhang has joined #openstack-qa | 08:59 | |
| *** rcernin has joined #openstack-qa | 09:00 | |
| *** brinzhang has quit IRC | 09:01 | |
| *** brinzhang has joined #openstack-qa | 09:01 | |
| *** tosky has joined #openstack-qa | 09:02 | |
| *** whoami-rajat__ has joined #openstack-qa | 09:04 | |
| whoami-rajat__ | thanks gmann for working on this, | 09:05 |
| *** lucasagomes has joined #openstack-qa | 09:06 | |
| *** rcernin has quit IRC | 09:18 | |
| *** jparker has joined #openstack-qa | 09:18 | |
| *** rcernin has joined #openstack-qa | 09:23 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/tempest master: test volume from snapshot cascade delete for ceph https://review.opendev.org/c/openstack/tempest/+/773621 | 09:49 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: test_live_migration: Rename test_iscsi_volume https://review.opendev.org/c/openstack/tempest/+/710039 | 09:56 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: test_live_migration: Remove block_migration_for_live_migration skip https://review.opendev.org/c/openstack/tempest/+/710040 | 09:56 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: test_live_migration: Remove block_migrate_cinder_iscsi skip https://review.opendev.org/c/openstack/tempest/+/710041 | 09:56 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: test_live_migration: Move _test_live_migration into LiveMigrationTestBase https://review.opendev.org/c/openstack/tempest/+/710042 | 09:56 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: test_live_migration: Remove volume_backed_live_migration skip https://review.opendev.org/c/openstack/tempest/+/710043 | 09:56 |
| masayukig | kopecmartin: gmann I'm sorry for the short notice, but can you chair today's office hour? | 10:03 |
| kopecmartin | masayukig: sure | 10:06 |
| *** rcernin has quit IRC | 10:08 | |
| masayukig | kopecmartin: thanks! | 10:08 |
| *** rcernin has joined #openstack-qa | 10:19 | |
| *** tosky has quit IRC | 10:33 | |
| *** tosky has joined #openstack-qa | 10:34 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW, TESTING! https://review.opendev.org/c/openstack/devstack/+/773630 | 10:34 |
| hemanth_n | Hi, tempest scenario tests fails with devstack stable/rocky, more info here https://bugs.launchpad.net/tempest/+bug/1914037 (this is breaking nova stable/rocky CI jobs) | 11:11 |
| openstack | Launchpad bug 1914037 in tempest "scenario tests tempest.scenario.test_network_v6.TestGettingAddress fails" [Undecided,New] | 11:11 |
| *** rcernin has quit IRC | 11:13 | |
| *** dviroel has joined #openstack-qa | 11:15 | |
| *** Luzi has joined #openstack-qa | 11:20 | |
| *** hemanth_n has quit IRC | 11:23 | |
| *** rcernin has joined #openstack-qa | 11:35 | |
| *** gfidente has quit IRC | 11:35 | |
| *** brinzhang has quit IRC | 11:45 | |
| *** brinzhang has joined #openstack-qa | 11:46 | |
| *** gfidente has joined #openstack-qa | 11:47 | |
| *** hemanth_n has joined #openstack-qa | 11:49 | |
| *** rcernin has quit IRC | 12:04 | |
| *** iurygregory_ has joined #openstack-qa | 12:09 | |
| *** iurygregory has quit IRC | 12:09 | |
| *** yamamoto has quit IRC | 12:15 | |
| *** zenkuro has quit IRC | 12:22 | |
| *** zenkuro has joined #openstack-qa | 12:23 | |
| *** yamamoto has joined #openstack-qa | 12:31 | |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** yamamoto has quit IRC | 12:39 | |
| *** tbachman has quit IRC | 12:44 | |
| *** tbachman has joined #openstack-qa | 12:48 | |
| *** yamamoto has joined #openstack-qa | 12:49 | |
| *** yamamoto has quit IRC | 12:50 | |
| *** iurygregory_ is now known as iurygregory | 13:04 | |
| *** yamamoto has joined #openstack-qa | 13:07 | |
| *** yamamoto has quit IRC | 13:07 | |
| *** yamamoto has joined #openstack-qa | 13:08 | |
| *** songwenping__ has quit IRC | 13:11 | |
| *** yamamoto has quit IRC | 13:12 | |
| *** swp20 has joined #openstack-qa | 13:13 | |
| *** jpena|lunch is now known as jpena | 13:25 | |
| gmann | masayukig: ok | 13:48 |
| *** tbachman has quit IRC | 13:52 | |
| *** tbachman has joined #openstack-qa | 13:55 | |
| *** zenkuro has quit IRC | 13:55 | |
| *** zenkuro has joined #openstack-qa | 13:56 | |
| kopecmartin | gmann: will you chair, or should I? | 14:01 |
| gmann | i can do | 14:01 |
| gmann | #startmeeting qa | 14:01 |
| openstack | Meeting started Tue Feb 2 14:01:31 2021 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| *** openstack changes topic to " (Meeting topic: qa)" | 14:01 | |
| openstack | The meeting name has been set to 'qa' | 14:01 |
| kopecmartin | hi all o/ | 14:01 |
| gmann | who all here today? | 14:01 |
| gmann | kopecmartin: seems two of us | 14:03 |
| kopecmartin | it seems so | 14:03 |
| lpiwowar | Hi | 14:03 |
| gmann | lpiwowar: hi | 14:03 |
| gmann | let's start anyways | 14:04 |
| gmann | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Agenda_for_next_Office_hours | 14:04 |
| gmann | today agenda ^^ | 14:04 |
| *** paras333 has joined #openstack-qa | 14:04 | |
| gmann | #topic Announcement and Action Item (Optional | 14:04 |
| *** openstack changes topic to "Announcement and Action Item (Optional (Meeting topic: qa)" | 14:04 | |
| gmann | nothing on this, no AI from previous meeting | 14:04 |
| gmann | #link Wallaby Priority Items progress | 14:05 |
| gmann | #link https://etherpad.opendev.org/p/qa-wallaby-priority | 14:05 |
| soniya29 | hello all | 14:05 |
| gmann | soniya29: hi | 14:05 |
| paras333 | no update from my end, I will be working on it this week | 14:06 |
| gmann | for RBAC testing strategy, L45: we have most of the patches merged | 14:06 |
| kopecmartin | no updates from my side wrt priority items | 14:06 |
| gmann | but i see some bug for system admin | 14:06 |
| gmann | this one #link https://review.opendev.org/c/openstack/tempest/+/773173/3 | 14:06 |
| gmann | kopecmartin: ^^ | 14:06 |
| gmann | also adding reno and fixing some todo #link https://review.opendev.org/c/openstack/tempest/+/773159/5 | 14:06 |
| gmann | after these I think we can close this item | 14:07 |
| gmann | kopecmartin: soniya29 anything to bring on scenario manager? | 14:07 |
| soniya29 | I am working on the remaining stuffs of sceanrio managaer | 14:08 |
| gmann | #link https://review.opendev.org/q/topic:bp/tempest-scenario-manager-stable+status:open+ | 14:08 |
| gmann | soniya29: great | 14:08 |
| kopecmartin | almost all reviews up, need just core reviews basically | 14:08 |
| soniya29 | two patches are up and i will be adding another two which would complete - non-private apis stuff | 14:09 |
| gmann | yeah, 2-3 are hanging for single +2 for long | 14:09 |
| gmann | I did not propose the single core review yet, I will do right after meeting that will help to merge these fast | 14:09 |
| gmann | #action gmann to propose single core review merge policy for Tempest | 14:10 |
| gmann | anything else on wallaby priority | 14:10 |
| gmann | let's move if nothing | 14:11 |
| gmann | #topic Gate Status Checks | 14:11 |
| *** openstack changes topic to "Gate Status Checks (Meeting topic: qa)" | 14:11 | |
| soniya29 | gmann, I have gone through the patrole links you passed in the earlier meetings. So do the patrole maintainance item requires reviewing of the patches or any other help is required? | 14:11 |
| gmann | get-pip things are merged for py2.7/3.5 in all required stable | 14:11 |
| gmann | soniya29: nice, let's talk that in patrole topic | 14:12 |
| soniya29 | sure | 14:12 |
| gmann | any other gate failure anyone noticing ? | 14:12 |
| kopecmartin | maybe this | 14:12 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1914037 | 14:12 |
| openstack | Launchpad bug 1914037 in tempest "scenario tests tempest.scenario.test_network_v6.TestGettingAddress fails" [Undecided,New] | 14:12 |
| gmann | kopecmartin: is it happening all the time or randomly? | 14:13 |
| kopecmartin | i'd say randomly, i haven't notice such an issue in our gates | 14:14 |
| kopecmartin | it occurred in tempest-slow | 14:14 |
| gmann | ah, and do we have tempest-slow running on rocky gate ? | 14:14 |
| kopecmartin | we had a patch merged which was supposed to address this | 14:14 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/763621 | 14:14 |
| kopecmartin | but seems it's not working for ipv6? | 14:15 |
| kopecmartin | let me check | 14:15 |
| gmann | yeah we do run, i am checking nova gate #link https://review.opendev.org/c/openstack/nova/+/758735 | 14:16 |
| gmann | and it does fail with same error | 14:16 |
| gmann | anyways let's discuss it in bug section | 14:16 |
| gmann | Periodic stable gate is green: #link https://zuul.openstack.org/builds?job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&job_name=tempest-full-train-py3&pipeline=periodic-stable | 14:17 |
| gmann | Periodic master: tempest-all still broken #link https://zuul.openstack.org/builds?job_name=tempest-all&job_name=tempest-full-oslo-master&pipeline=periodic | 14:17 |
| gmann | #topic Sub Teams highlights (Sub Teams means individual projects under QA program) | 14:18 |
| *** openstack changes topic to "Sub Teams highlights (Sub Teams means individual projects under QA program) (Meeting topic: qa)" | 14:18 | |
| gmann | Tempest | 14:18 |
| gmann | many open reviews #link https://review.opendev.org/q/project:openstack/tempest+status:open | 14:18 |
| kopecmartin | swift availability in tempest-full-py3 | 14:19 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/763981 | 14:19 |
| gmann | as we already talked on review policy changethat can improve it | 14:19 |
| kopecmartin | seems it needs a few more modifications | 14:19 |
| *** brinzhang has quit IRC | 14:19 | |
| gmann | ok, I did not get the comment | 14:20 |
| *** brinzhang has joined #openstack-qa | 14:20 | |
| gmann | kopecmartin: i mentioned about swift in L145 | 14:20 |
| gmann | sorry L125 | 14:21 |
| kopecmartin | i meant that if the swift was enabled it would be set accordingly in tempest.conf under service section , but in master and victoria run it's not | 14:21 |
| gmann | ah it should run on master and ussuri onwards. I think branch variant not working or correctly set | 14:21 |
| kopecmartin | yeah, something like that | 14:22 |
| gmann | i need to check zuul inventory. Will check and update accordingly | 14:22 |
| gmann | thanks kopecmartin for checking that | 14:22 |
| gmann | I see some patches for removing thr six | 14:23 |
| gmann | #link https://review.opendev.org/q/topic:%2522remove-six%2522+(status:open+OR+status:merged)+project:openstack/tempest | 14:23 |
| gmann | I think we can do that as Tempest is branchless and no backport issue here | 14:24 |
| gmann | anything else on Tempest? | 14:24 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/763981 | 14:24 |
| kopecmartin | oh, but that's more a bug | 14:24 |
| kopecmartin | so we can discuss that at the end | 14:25 |
| gmann | 763981 ? | 14:25 |
| gmann | or you mean to paste other review | 14:25 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1911044 | 14:25 |
| openstack | Launchpad bug 1911044 in tempest "tempest still uses unittest2 after converting to python3 only" [Medium,Confirmed] | 14:25 |
| kopecmartin | yeah, sorry, i meant this ^^ | 14:25 |
| gmann | ah i see | 14:25 |
| gmann | yeah we can do in bug topic | 14:25 |
| gmann | Patrole | 14:25 |
| gmann | soniya29: go ahead | 14:26 |
| soniya29 | gmann, we were discussing about patrole maintainance in wallaby ptg | 14:26 |
| gmann | #link https://review.opendev.org/q/project:openstack/patrole+status:open | 14:26 |
| gmann | yeah | 14:27 |
| *** Luzi has quit IRC | 14:27 | |
| soniya29 | So in that action item is reviewing patches is required or anything else? | 14:27 |
| gmann | soniya29: its both and keep gate green. there are many API policy changes happen on project side so we need to do some changes in patrole to fix those | 14:28 |
| gmann | soniya29: also as you see many of the open review are merge conflict. if we can rebase/continue those that will be great | 14:29 |
| soniya29 | gmann, okay | 14:29 |
| gmann | and later in wallaby cycle we can judge if patrole is stable and we can release it as stable version or not | 14:29 |
| soniya29 | sure | 14:30 |
| gmann | feel free to reachout for any help or discussion anytime | 14:30 |
| soniya29 | gmann, yeah | 14:30 |
| gmann | soniya29: thanks for heads up and help | 14:30 |
| gmann | Devstack | 14:30 |
| gmann | there are two thing I will bring here and need more attention/review | 14:31 |
| gmann | # centos-8-stream job #link https://review.opendev.org/c/openstack/devstack/+/759122 | 14:31 |
| gmann | and devstack Async task support | 14:31 |
| gmann | #link https://review.opendev.org/c/openstack/devstack/+/771505 | 14:32 |
| gmann | I am testing 771505 locally and we do have job also with async mode in same patch to check the difference. async mdoe is disabled by default | 14:32 |
| gmann | you can try and review that and feel free to add your observation | 14:33 |
| gmann | and make dansmith happy if we can merge this :) | 14:33 |
| gmann | Grenade | 14:34 |
| dansmith | I probably need to clean up the commit message | 14:34 |
| dansmith | but I will do that | 14:34 |
| gmann | +1 | 14:34 |
| gmann | #link https://review.opendev.org/q/project:openstack/grenade+status:open | 14:35 |
| gmann | nothing much on grenade but there are couple of review need feedback | 14:35 |
| gmann | or even more, anyways let's continue reviewing those | 14:35 |
| gmann | Hacking | 14:35 |
| gmann | #link https://review.opendev.org/q/project:openstack/hacking+status:open | 14:35 |
| gmann | paras333: anything you would like to bring here | 14:36 |
| paras333 | gmann: nothing major on this | 14:36 |
| gmann | ok | 14:36 |
| paras333 | tox lower constraint job has been removed | 14:36 |
| paras333 | temporarily, do there a plan of keeping it in future? | 14:37 |
| gmann | yeah, I am testing direct deps on l-c for nova and other project. i can do that for hacking also if that make it green without much fix | 14:37 |
| paras333 | gmann: ack, thanks | 14:37 |
| gmann | #action gmann tro hacking l-c job with direct deps | 14:37 |
| gmann | #topic Gate Blocker Fix / Urgent Change | 14:37 |
| *** openstack changes topic to "Gate Blocker Fix / Urgent Change (Meeting topic: qa)" | 14:37 | |
| gmann | any urgent review other than what we already discussed ? | 14:37 |
| gmann | #topic Open Discussion | 14:38 |
| *** openstack changes topic to "Open Discussion (Meeting topic: qa)" | 14:38 | |
| gmann | nothing here too | 14:38 |
| gmann | #topic Bug Triage (last 30 min) | 14:38 |
| *** openstack changes topic to "Bug Triage (last 30 min) (Meeting topic: qa)" | 14:38 | |
| gmann | let's do bug thing now | 14:39 |
| gmann | first is this #link https://bugs.launchpad.net/tempest/+bug/1914037 | 14:39 |
| openstack | Launchpad bug 1914037 in tempest "scenario tests tempest.scenario.test_network_v6.TestGettingAddress fails" [Undecided,New] | 14:39 |
| gmann | kopecmartin: so this change fix for master and all but did not fix for rocky? https://opendev.org/openstack/tempest/commit/4ad346e6497d4fbad9e379e8ed50730a0b6b55a3 | 14:40 |
| gmann | mean it fix in master Tempest but rocky use tempest 23.0.0 so it is broken in rocky gate? | 14:40 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Async task support https://review.opendev.org/c/openstack/devstack/+/771505 | 14:41 |
| tosky | I was late for the open discussion, sorry for the OT, but there is this devstack refactoring for cinder-backup that is needed to proper test a new driver (cinder/s3): https://review.opendev.org/c/openstack/devstack/+/770171 | 14:41 |
| kopecmartin | gmann: yeah, seems so, the bug mentions that the problem is in stable/rocky | 14:41 |
| kopecmartin | but that's very old release, do we still support that? | 14:42 |
| gmann | tosky: i see, ack thanks for brining that. yeah we need some job with those configure to see the result | 14:42 |
| gmann | kopecmartin: i have not triage that but does this fix it ? -https://opendev.org/openstack/tempest/commit/4ad346e6497d4fbad9e379e8ed50730a0b6b55a3 | 14:43 |
| gmann | seems so from bug comment | 14:43 |
| kopecmartin | gmann: i didn't verify, but it seems so | 14:43 |
| gmann | ok | 14:43 |
| kopecmartin | the review is tied to this bug | 14:43 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1903433 | 14:43 |
| openstack | Launchpad bug 1903433 in neutron "The duplicate route in l3 router namespace results in North-South traffic breaking" [Medium,In progress] - Assigned to yangjianfeng (yangjianfeng) | 14:43 |
| gmann | then we cannot fix it for rocky as it is in EM stage and use old tempest | 14:44 |
| kopecmartin | which was very similar issue like the current one, only with ipv4 not ipv6 | 14:44 |
| kopecmartin | i wrote a comment asking for confirmation if only rocky release is affected | 14:45 |
| gmann | ok but that did not backported to rocky | 14:45 |
| kopecmartin | that's correct | 14:45 |
| gmann | so what we can do is we can skip the test for stable rocky and move unblock the tempest-slow | 14:45 |
| openstackgerrit | Eric Harney proposed openstack/devstack-plugin-ceph master: Test fix for bug #1913449 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/773698 | 14:45 |
| openstack | bug 1913449 in Cinder "rbd/rados lib changes in ceph pacific prevents c-vol to initialize properly" [Undecided,New] https://launchpad.net/bugs/1913449 - Assigned to Eric Harney (eharney) | 14:45 |
| gmann | and if any fix from neutron or so fix or so then we can unskip it | 14:46 |
| kopecmartin | ok | 14:46 |
| gmann | I acn push that with bug skip and feature flag may be | 14:46 |
| gmann | can | 14:46 |
| kopecmartin | ok, sounds greawt | 14:47 |
| kopecmartin | great | 14:47 |
| kopecmartin | let's move on | 14:47 |
| kopecmartin | we have a fix | 14:47 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/642237 | 14:47 |
| gmann | kopecmartin: as tempest does not support any EM stable branch and due to its brachless nature it is not possible for us to backport the things | 14:47 |
| kopecmartin | for a high old priority bug: | 14:48 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1613819 | 14:48 |
| openstack | Launchpad bug 1613819 in tempest "v3 credentials client does not honour the configured domain" [High,In progress] - Assigned to Lukas Piwowarski (lukas-piwowarski) | 14:48 |
| kopecmartin | *old high priority bug :D | 14:48 |
| gmann | kopecmartin: this on right #link https://review.opendev.org/c/openstack/tempest/+/642237 | 14:48 |
| kopecmartin | gmann: yes | 14:48 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Async task support https://review.opendev.org/c/openstack/devstack/+/771505 | 14:49 |
| gmann | cool, this is in my list i was waiting for other system scope things to merge first | 14:49 |
| gmann | will review it by today or tomorrow most probably | 14:49 |
| kopecmartin | ok | 14:49 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1911044 | 14:49 |
| openstack | Launchpad bug 1911044 in tempest "tempest still uses unittest2 after converting to python3 only" [Medium,Confirmed] | 14:49 |
| kopecmartin | seems like we still use unittest2 | 14:49 |
| kopecmartin | it seems that unittest2 has been a part of a workaround as i commented in comment #3 | 14:50 |
| gmann | yeah | 14:50 |
| gmann | testtool still not removed it due to backward compatibility #link https://github.com/testing-cabal/testtools/pull/277 | 14:51 |
| gmann | I think we just need to be careful about stable branch testing which is rain onwards | 14:51 |
| gmann | *train | 14:51 |
| gmann | and kopecmartin you are saying we need to bump the stestr constraint in train ? | 14:52 |
| kopecmartin | i thought so, i need to recheck, i forgot the context | 14:53 |
| *** lpiwowar has quit IRC | 14:53 | |
| gmann | did stestr dropped the workaround ? | 14:53 |
| *** lpiwowar has joined #openstack-qa | 14:53 | |
| kopecmartin | no idea, i'll need to check | 14:55 |
| gmann | it seems yes https://github.com/mtreinish/stestr/blob/master/stestr/subunit_runner/run.py | 14:55 |
| gmann | #link https://github.com/mtreinish/stestr/commit/e2d2fab911a5e696f1092560342aa26b91cbc5ff#diff-45f3f6acc69b6f83389b916cb009a1092d9c08fa0419f7c9e82832467ef5b1f8 | 14:56 |
| gmann | kopecmartin: 3.0.0 removed it | 14:56 |
| kopecmartin | so we cannot drop it now | 14:57 |
| kopecmartin | #link https://opendev.org/openstack/requirements/src/branch/stable/train/upper-constraints.txt#L366 | 14:57 |
| kopecmartin | as train uses 2.5.1 | 14:57 |
| gmann | but train still need to support py2.7 and use stestr 2.5.1 | 14:57 |
| gmann | I need to give more thought on this about all combination of py and stable branch version :) to avoid any break | 14:58 |
| gmann | I will comment on bug later | 14:58 |
| gmann | we are 3 min left | 14:59 |
| kopecmartin | i don't have anything else | 14:59 |
| gmann | any critical bug or other high priority to bring here? | 14:59 |
| gmann | ok, let's close the office hour then | 15:00 |
| gmann | thanks all for joining | 15:00 |
| gmann | #endmeeting | 15:00 |
| *** openstack changes topic to "Discussion of OpenStack Quality Assurance Projects (tempest, devstack, grenade, and friends) | docs https://wiki.openstack.org/wiki/QA | specs https://review.openstack.org/#/q/project:openstack/qa-specs | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-qa/" | 15:00 | |
| openstack | Meeting ended Tue Feb 2 15:00:25 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-02-14.01.html | 15:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-02-14.01.txt | 15:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-02-14.01.log.html | 15:00 |
| openstackgerrit | Eric Harney proposed openstack/devstack-plugin-ceph master: Test fix for bug #1913449 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/773698 | 15:02 |
| openstack | bug 1913449 in Cinder "rbd/rados lib changes in ceph pacific prevents c-vol to initialize properly" [Undecided,New] https://launchpad.net/bugs/1913449 - Assigned to Eric Harney (eharney) | 15:02 |
| *** hemanth_n has quit IRC | 15:02 | |
| kopecmartin | gmann: if we cannot remove unittest2 because of train, then we also can't remove 'import six' .. right? it would also affect train, wouldn't it? | 15:03 |
| gmann | kopecmartin: ah right, good point. i did not think about py2 jobs for train. yeah we need to hold that until train stop using master Tempest | 15:03 |
| gmann | kopecmartin: please add this in those patches may be procedural -2 until stable/train stop using master Tempest | 15:04 |
| amodi | sean-k-mooney: lyarwood hi, if u can look at https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/771283, the gate also failed on the unable to delete security group, idk if that job uses 'creator' role, but i suppose with fixed key, we don't need it | 15:04 |
| lyarwood | amodi: yup will look shortly | 15:05 |
| kopecmartin | gmann: ok | 15:06 |
| amodi | lyarwood: thanks, no rush | 15:06 |
| *** brinzhang has quit IRC | 15:06 | |
| *** brinzhang has joined #openstack-qa | 15:06 | |
| *** psahoo has joined #openstack-qa | 15:08 | |
| sean-k-mooney | is create_test_server coming form whitebox or tempest by the way | 15:08 |
| sean-k-mooney | i think we ahve a verion of that our selve right | 15:08 |
| sean-k-mooney | that calls the tempest one | 15:08 |
| sean-k-mooney | yes https://opendev.org/openstack/whitebox-tempest-plugin/src/commit/61d852056d3539d461e1a7e571f69001f40aa24e/whitebox_tempest_plugin/api/compute/base.py#L43 | 15:08 |
| *** brinzhang has quit IRC | 15:08 | |
| *** brinzhang has joined #openstack-qa | 15:09 | |
| *** brinzhang has quit IRC | 15:10 | |
| *** brinzhang has joined #openstack-qa | 15:11 | |
| *** brinzhang has quit IRC | 15:12 | |
| *** brinzhang has joined #openstack-qa | 15:12 | |
| *** slaweq has quit IRC | 15:14 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW, TESTING! https://review.opendev.org/c/openstack/devstack/+/773630 | 15:14 |
| *** brinzhang has quit IRC | 15:14 | |
| *** slaweq has joined #openstack-qa | 15:14 | |
| *** brinzhang has joined #openstack-qa | 15:15 | |
| amodi | sean-k-mooney: yeah isn't it using that already? the whitebox one | 15:18 |
| amodi | the security group is comgin from validation resources, i wonder what is not allowing it to be deleted :/ | 15:21 |
| sean-k-mooney | why is it trying to delete the security group by the way | 15:36 |
| sean-k-mooney | i guess https://opendev.org/openstack/tempest/src/branch/master/tempest/test.py#L762-L765 might be addign a cleanup routine for it | 15:38 |
| sean-k-mooney | in general however i tought the security group used was configure in the config and shared between tests | 15:38 |
| sean-k-mooney | so it should not be deleting it | 15:38 |
| sean-k-mooney | looking at teh test im not sure its a valid test by the way | 15:41 |
| *** zenkuro has quit IRC | 15:41 | |
| *** gfidente has quit IRC | 15:47 | |
| openstackgerrit | Dan Smith proposed openstack/devstack master: DNM: Test just on vexxhost https://review.opendev.org/c/openstack/devstack/+/773714 | 15:47 |
| *** psahoo has quit IRC | 15:48 | |
| *** gfidente has joined #openstack-qa | 15:49 | |
| lyarwood | amodi: tbh I think the test is failing before we get to your delete_server at the end, it's just not logged | 15:50 |
| openstackgerrit | Lee Yarwood proposed openstack/whitebox-tempest-plugin master: Introduce volume encryption tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/771283 | 15:53 |
| amodi | lyarwood: oh okay checking the updated patch | 15:56 |
| amodi | actually i was debugging it and the test failed after the delete server, on the error i mentioned in the patch comment, this one from Zuul is kind of different as it does find the security group | 15:58 |
| lyarwood | amodi: there's no DELETE /server request in the tempest log AFAICT so it didn't get there | 16:00 |
| amodi | lyarwood: um, okay | 16:07 |
| *** yamamoto has joined #openstack-qa | 16:17 | |
| *** lbragstad_ is now known as lbragstad | 16:21 | |
| *** yamamoto has quit IRC | 16:26 | |
| *** zenkuro has joined #openstack-qa | 16:53 | |
| sean-k-mooney | lyarwood: amodi by the way you cant just do 'echo "Hello, world (encrypted /dev/vdc)" >> /dev/vdc;' right | 16:56 |
| sean-k-mooney | that woudl be echoing into the block device | 16:56 |
| sean-k-mooney | you would need to mount it and put a file systme on it then echo into a file on the file system | 16:56 |
| *** lpiwowar has quit IRC | 16:56 | |
| sean-k-mooney | thats proably whats failng before we get to the delete | 16:57 |
| sean-k-mooney | so lee's version shoudl clean up cleanly but it will still fail the test | 16:58 |
| sean-k-mooney | but not with the security group issue hopefuly | 16:58 |
| lyarwood | sean-k-mooney: yup it's a pattern I've seen in other tempest tests btw so I guess that's where amodi is copying this from | 17:02 |
| lyarwood | sean-k-mooney: but I think they do create a filesystem etc as you say | 17:03 |
| lyarwood | qemu-img info from the host is going to be quicker either way IMHO | 17:03 |
| *** lucasagomes has quit IRC | 17:04 | |
| sean-k-mooney | lyarwood: we need this to work with ceph | 17:06 |
| sean-k-mooney | so we cant assume host mounts | 17:06 |
| sean-k-mooney | so i woule prefer to just check the xml and assert the secret exits | 17:07 |
| sean-k-mooney | and that its updated with the attached volumns | 17:07 |
| sean-k-mooney | if we can do that | 17:07 |
| sean-k-mooney | generically | 17:07 |
| sean-k-mooney | lyarwood: i assume the other exampled dont echo into the device file by the way | 17:11 |
| lyarwood | sean-k-mooney: qemu-img should work fine from the compute | 17:11 |
| lyarwood | sean-k-mooney: for remote rbd volumes that is, without them being attached as host block devices | 17:12 |
| sean-k-mooney | lyarwood: really it can connect to the ceph mon and backend osd? | 17:13 |
| sean-k-mooney | while the vm is running | 17:14 |
| lyarwood | sean-k-mooney: yup | 17:14 |
| sean-k-mooney | i dont really object to checkign for the luks signiture | 17:14 |
| lyarwood | sean-k-mooney: qemu-img info --force-share rbd:$image:id=$user | 17:14 |
| sean-k-mooney | its actully a nice thing to do solong as we can do it generically | 17:14 |
| sean-k-mooney | oh --force-share | 17:14 |
| sean-k-mooney | cool | 17:14 |
| sean-k-mooney | then ya if we can do that then im ok with that | 17:15 |
| lyarwood | cool, brb | 17:15 |
| *** ralonsoh has quit IRC | 17:45 | |
| *** Yarboa has quit IRC | 17:48 | |
| *** gfidente is now known as gfidente|afk | 17:52 | |
| *** jpena is now known as jpena|off | 17:55 | |
| *** rcernin has joined #openstack-qa | 17:59 | |
| *** rcernin has quit IRC | 18:04 | |
| *** gcheresh has quit IRC | 18:17 | |
| openstackgerrit | Merged openstack/devstack master: Drop opensuse platform job https://review.opendev.org/c/openstack/devstack/+/769884 | 18:18 |
| openstackgerrit | Soniya Murlidhar Vyas proposed openstack/tempest master: Need to make certain apis non-private https://review.opendev.org/c/openstack/tempest/+/773783 | 18:29 |
| *** dtantsur is now known as dtantsur|afk | 18:35 | |
| *** rpittau is now known as rpittau|afk | 18:39 | |
| *** artom has quit IRC | 18:56 | |
| *** artom has joined #openstack-qa | 18:56 | |
| *** whoami-rajat__ has quit IRC | 18:59 | |
| *** sboyron_ has quit IRC | 19:03 | |
| lbragstad | gmann qq for you - i'm tinkering around with devstack plugins https://review.opendev.org/c/openstack/glance-tempest-plugin/+/773568 | 19:25 |
| lbragstad | and i can't seem to get the plugin tests to run because it doesn't look like they're installed https://zuul.opendev.org/t/openstack/build/22c087f59957486e91f046152920f935 | 19:26 |
| lbragstad | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/773568/5/.zuul.yaml@8 should be doing that for me, right? | 19:26 |
| lbragstad | gmann oh - nevermind, i think i figured itout | 19:30 |
| *** zenkuro has quit IRC | 19:41 | |
| *** artom has quit IRC | 19:43 | |
| *** artom has joined #openstack-qa | 19:43 | |
| *** rcernin has joined #openstack-qa | 20:00 | |
| *** rcernin has quit IRC | 20:04 | |
| *** stand has joined #openstack-qa | 20:07 | |
| *** Yarboa has joined #openstack-qa | 20:12 | |
| *** yamamoto has joined #openstack-qa | 20:23 | |
| *** rcernin has joined #openstack-qa | 20:25 | |
| *** yamamoto has quit IRC | 20:27 | |
| openstackgerrit | Sergey Bekkerman proposed openstack/tempest master: Set Nova availability zone for non-scenario tests https://review.opendev.org/c/openstack/tempest/+/773476 | 20:56 |
| *** vishalmanchanda has quit IRC | 21:15 | |
| *** tbachman has quit IRC | 21:19 | |
| *** rcernin has quit IRC | 21:22 | |
| *** tbachman has joined #openstack-qa | 21:26 | |
| *** gfidente|afk has quit IRC | 21:33 | |
| *** rcernin has joined #openstack-qa | 21:52 | |
| dansmith | gmann: cleaned up and passing zuul: https://review.opendev.org/c/openstack/devstack/+/771505 :D | 21:57 |
| *** ccamposr has joined #openstack-qa | 22:01 | |
| *** ccamposr__ has quit IRC | 22:03 | |
| gmann | dansmith: ack. | 22:04 |
| gmann | lbragstad: are we going to implement tempest tests for glance too? I was thinking to go with unit tests like nova | 22:05 |
| lbragstad | gmann i'm willing to be convinced either way | 22:05 |
| gmann | lbragstad: also enforce_scope config we need to add in tempest side because anyways we need to add those in tempest in-tree tests too | 22:05 |
| gmann | lbragstad: ok, do we have unit tests there? | 22:06 |
| gmann | if not then i can add some | 22:06 |
| lbragstad | i plunked through glance unit and functional tests a little bit yesterday, but i didn't find a whole lot for policy testing | 22:06 |
| lbragstad | if we go that route, we're going to need another piece of test-only middleware i think | 22:06 |
| lbragstad | abhishekk was testing policy changes manually with devstack, so i started porting his process to use the devstack-tempest-plugin | 22:07 |
| gmann | test-only middleware ? I think existing unit tests middleware should work but with actual API operation mocked | 22:08 |
| *** rcernin has quit IRC | 22:08 | |
| lbragstad | ok - so not a functional API test? | 22:08 |
| *** rcernin has joined #openstack-qa | 22:09 | |
| gmann | yeah not functional only unit tests like done in nova. | 22:09 |
| gmann | let me try one example and then we can see both approach. | 22:09 |
| lbragstad | https://opendev.org/openstack/glance/src/branch/master/glance/tests/functional/__init__.py#L457-L459 | 22:10 |
| lbragstad | i know that's used in the functional test | 22:10 |
| lbragstad | tests* | 22:10 |
| lbragstad | so - i wasn't sure how much i'd break if i started ripping all that apart | 22:10 |
| gmann | this is what is being used in current tests https://opendev.org/openstack/glance/src/branch/master/glance/tests/etc/policy.json | 22:12 |
| gmann | allowed to all | 22:12 |
| *** yamamoto has joined #openstack-qa | 22:12 | |
| lbragstad | yeah - that makes sense | 22:13 |
| lbragstad | most projects are doing something like that with functional api tests | 22:13 |
| gmann | yeah | 22:13 |
| lbragstad | but there are varying degrees of how coupled that is with other test utilities | 22:13 |
| gmann | we should slowly move them to test with actual defaults | 22:14 |
| lbragstad | for example, i tried smoothing some of that out in cinder by adding a new functional base test class and configuring it to use the new defaults, but that broke all the existing functional tests | 22:16 |
| gmann | yeah, that's true. it might need lot f refactorizing. that is why we choose to do it in unit tests so that we can test the policy bits only and with actual defaults | 22:19 |
| lbragstad | so - with the unit test approach, are you only testing that enforce() returns True or False? | 22:20 |
| gmann | moving existing functional actual defaults can be separate effort | 22:20 |
| lbragstad | or with unit tests, are you mocking the underlying subsystems and effectively testing the API layer without a request by invoking the controller? | 22:21 |
| gmann | I tests if API pass through the enforcer() and move to next layer in API or driver | 22:21 |
| gmann | lbragstad: yeah that way | 22:21 |
| lbragstad | ok | 22:21 |
| gmann | invoking the controller | 22:21 |
| gmann | and with that it actually tested the policy things and we found many bugs in API code too where right targets were not passed to oslo policy by API controller. that way we were much satisfy on testing strategy | 22:23 |
| lbragstad | well that's good | 22:23 |
| lbragstad | i've been getting a mixed bag of responses to that level of mocking | 22:24 |
| lbragstad | from other projects | 22:24 |
| gmann | if their policy check are scattered other than API controller and in DB or system level layer then it is valid concern. but if it is only API controller doing the policy checks then these mocking are ok | 22:25 |
| gmann | as per engineering decision and our current infra availability :) | 22:25 |
| gmann | testing infra | 22:25 |
| lbragstad | yeah - so i hit exactly that in cinder | 22:26 |
| gmann | yeah may be cinder is special :), but let's try with that in glance and see. | 22:27 |
| lbragstad | i mocked the backend to return a volume and i wrote a test to see if a user could fish volumes out of another project if they knew the ID - and it passed | 22:27 |
| gmann | as you already have tempest like test for glance, let me push unit tests example too and then we can discuss in glance team | 22:27 |
| lbragstad | ok | 22:27 |
| openstackgerrit | Ghanshyam proposed openstack/hacking master: DNM: testing direct deps in l-c https://review.opendev.org/c/openstack/hacking/+/773814 | 22:52 |
| *** slaweq has quit IRC | 22:55 | |
| *** yamamoto has quit IRC | 23:03 | |
| *** yamamoto_ has joined #openstack-qa | 23:03 | |
| *** Yarboa has quit IRC | 23:10 | |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add config option for enforce_scope https://review.opendev.org/c/openstack/tempest/+/773816 | 23:17 |
| gmann | lbragstad: ^^ adding consolidated config option for enforce_scope. | 23:18 |
| lbragstad | sweet | 23:18 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add system scoped request for compute hypervisor tests https://review.opendev.org/c/openstack/tempest/+/740122 | 23:19 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add system scoped request for compute hypervisor tests https://review.opendev.org/c/openstack/tempest/+/740122 | 23:21 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: DNM: testing hypersivor with system scope https://review.opendev.org/c/openstack/tempest/+/770197 | 23:26 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!