| *** tosky has quit IRC | 00:30 | |
| *** zenkuro has quit IRC | 00:36 | |
| *** ccamposr has quit IRC | 00:56 | |
| *** dviroel has quit IRC | 01:06 | |
| *** sean-k-mooney has quit IRC | 01:25 | |
| *** sean-k-mooney has joined #openstack-qa | 01:27 | |
| *** hamalq has quit IRC | 01:47 | |
| *** mtreinish has quit IRC | 02:16 | |
| *** mtreinish has joined #openstack-qa | 02:16 | |
| *** rcernin has quit IRC | 02:31 | |
| *** rcernin has joined #openstack-qa | 03:23 | |
| *** rcernin has quit IRC | 03:28 | |
| *** hemanth_n has joined #openstack-qa | 03:28 | |
| *** rcernin has joined #openstack-qa | 03:28 | |
| *** jparker has quit IRC | 03:33 | |
| *** raukadah is now known as chandankumar | 04:37 | |
| *** jparker has joined #openstack-qa | 05:05 | |
| *** vishalmanchanda has joined #openstack-qa | 05:28 | |
| *** hemanth_n has quit IRC | 05:44 | |
| *** hemanth_n has joined #openstack-qa | 05:50 | |
| *** gcheresh has joined #openstack-qa | 06:18 | |
| *** ianw has quit IRC | 06:18 | |
| *** ianw has joined #openstack-qa | 06:19 | |
| *** sean-k-mooney has quit IRC | 06:20 | |
| *** sean-k-mooney has joined #openstack-qa | 06:20 | |
| *** slaweq has joined #openstack-qa | 06:35 | |
| *** mugsie has quit IRC | 06:37 | |
| *** jparker has quit IRC | 06:59 | |
| *** lpiwowar has joined #openstack-qa | 07:02 | |
| *** ccamposr has joined #openstack-qa | 07:17 | |
| *** lpiwowar has quit IRC | 07:23 | |
| *** lpiwowar has joined #openstack-qa | 07:24 | |
| *** eolivare has joined #openstack-qa | 07:49 | |
| *** sboyron_ has joined #openstack-qa | 08:10 | |
| masayukig | gmann: kopecmartin I'm sorry for the short notice but can you chair the office hour today? Or I will do. | 08:34 |
|---|---|---|
| *** rpittau|afk is now known as rpittau | 08:37 | |
| *** rcernin has quit IRC | 08:40 | |
| *** tosky has joined #openstack-qa | 08:44 | |
| *** dtantsur|afk is now known as dtantsur | 08:53 | |
| *** jpena|off is now known as jpena | 08:59 | |
| kopecmartin | masayukig: me or gmann can chair the meeting if needed | 09:08 |
| *** lucasagomes has joined #openstack-qa | 09:08 | |
| *** yamamoto has quit IRC | 09:13 | |
| masayukig | kopecmartin: thanks! | 09:18 |
| *** rcernin has joined #openstack-qa | 09:18 | |
| openstackgerrit | Lajos Katona proposed openstack/tempest master: WIP: Add live migration with trunk test https://review.opendev.org/c/openstack/tempest/+/774689 | 09:19 |
| openstackgerrit | Lukas Piwowarski proposed openstack/eslint-config-openstack master: Fix building of the pdf documentation https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 09:20 |
| *** whoami-rajat__ has joined #openstack-qa | 09:25 | |
| *** rcernin has quit IRC | 09:35 | |
| openstackgerrit | Lukas Piwowarski proposed openstack/eslint-config-openstack master: Fix building of the pdf documentation https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 09:39 |
| *** yamamoto has joined #openstack-qa | 09:48 | |
| *** zenkuro has joined #openstack-qa | 09:57 | |
| *** yamamoto has quit IRC | 10:01 | |
| *** yamamoto has joined #openstack-qa | 10:18 | |
| *** zenkuro has quit IRC | 10:46 | |
| *** zenkuro has joined #openstack-qa | 10:46 | |
| openstackgerrit | Carlos Gonçalves proposed openstack/devstack master: Install master delorean-deps https://review.opendev.org/c/openstack/devstack/+/775802 | 10:48 |
| *** yamamoto has quit IRC | 10:49 | |
| *** yamamoto has joined #openstack-qa | 10:53 | |
| *** yamamoto has quit IRC | 10:55 | |
| openstackgerrit | Merged openstack/devstack stable/stein: Pin Tempest to 26.0.0 for stable stein testing https://review.opendev.org/c/openstack/devstack/+/774103 | 10:56 |
| *** yamamoto has joined #openstack-qa | 10:58 | |
| *** dviroel has joined #openstack-qa | 11:01 | |
| *** yamamoto has quit IRC | 11:02 | |
| *** gfidente has joined #openstack-qa | 11:07 | |
| *** mugsie has joined #openstack-qa | 11:13 | |
| *** yamamoto has joined #openstack-qa | 11:21 | |
| *** vishalmanchanda has quit IRC | 11:22 | |
| *** vishalmanchanda has joined #openstack-qa | 11:29 | |
| *** yamamoto has quit IRC | 11:32 | |
| openstackgerrit | Merged openstack/tempest master: Optimize the check pipeline runtime https://review.opendev.org/c/openstack/tempest/+/775043 | 11:37 |
| *** yamamoto has joined #openstack-qa | 11:39 | |
| *** yamamoto has quit IRC | 11:44 | |
| *** yamamoto has joined #openstack-qa | 11:46 | |
| *** lpiwowar has quit IRC | 12:16 | |
| *** lpiwowar has joined #openstack-qa | 12:17 | |
| *** slaweq has quit IRC | 12:23 | |
| *** jpena is now known as jpena|lunch | 12:27 | |
| *** ramishra has quit IRC | 12:41 | |
| *** ramishra has joined #openstack-qa | 12:41 | |
| *** ramishra has quit IRC | 12:42 | |
| *** ramishra has joined #openstack-qa | 12:44 | |
| *** Luzi has joined #openstack-qa | 12:55 | |
| *** zenkuro has quit IRC | 12:59 | |
| *** yamamoto has quit IRC | 13:07 | |
| *** yamamoto has joined #openstack-qa | 13:13 | |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: compute: Add [compute-feature-enabled]ide_bus flag https://review.opendev.org/c/openstack/tempest/+/775630 | 13:13 |
| *** yamamoto has quit IRC | 13:18 | |
| *** yamamoto has joined #openstack-qa | 13:18 | |
| *** yamamoto has quit IRC | 13:18 | |
| *** jpena|lunch is now known as jpena | 13:28 | |
| openstackgerrit | ansuman bebarta proposed openstack/tempest master: Add option to support volume_type in volume creation https://review.opendev.org/c/openstack/tempest/+/775830 | 13:30 |
| *** rcernin has joined #openstack-qa | 13:33 | |
| *** rcernin has quit IRC | 13:38 | |
| openstackgerrit | Lajos Katona proposed openstack/tempest master: WIP: Add live migration with trunk test https://review.opendev.org/c/openstack/tempest/+/774689 | 13:55 |
| *** yamamoto has joined #openstack-qa | 13:56 | |
| gmann | masayukig: np!, i can do | 13:57 |
| *** tbarron|out has quit IRC | 13:59 | |
| gmann | yoctozepto: can you check this please https://review.opendev.org/c/openstack/devstack/+/774524 | 14:00 |
| gmann | office hour time | 14:00 |
| gmann | #startmeeting qa | 14:00 |
| openstack | Meeting started Tue Feb 16 14:00:17 2021 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: qa)" | 14:00 | |
| openstack | The meeting name has been set to 'qa' | 14:00 |
| gmann | who all here today? | 14:00 |
| kopecmartin | hi o/ | 14:01 |
| gmann | let's wait for couple of min. | 14:02 |
| kopecmartin | sure | 14:03 |
| gmann | kopecmartin: seems two of us only. | 14:05 |
| kopecmartin | yeah :) | 14:05 |
| gmann | I would like to discuss on cutting new release for Tempest, so let's directly jump on those and skip the general agenda | 14:05 |
| kopecmartin | ok | 14:06 |
| gmann | and gate status we can check | 14:06 |
| gmann | #topic Gate Status Checks | 14:06 |
| *** openstack changes topic to "Gate Status Checks (Meeting topic: qa)" | 14:06 | |
| *** nweinber has joined #openstack-qa | 14:07 | |
| gmann | stable/stein started failing with Tempest and plugins master, for neutron it did break | 14:07 |
| gmann | capping Tempest 26.0.0 for stable/stein is merged #link https://review.opendev.org/c/openstack/devstack/+/774103 | 14:07 |
| gmann | and plugins also can cap the plugins compatible version in stein job | 14:07 |
| gmann | py2 fixes are not merged yet, I need to fix the review comments | 14:08 |
| *** yamamoto has quit IRC | 14:08 | |
| gmann | #link https://review.opendev.org/q/I5b2217d85e6871ca3f7a3f6f859fdce9a50d3946 | 14:08 |
| gmann | any other gate failure? | 14:09 |
| kopecmartin | I'm not aware of any other | 14:09 |
| gmann | cool | 14:09 |
| gmann | #topic Periodic jobs Status Checks | 14:09 |
| *** openstack changes topic to "Periodic jobs Status Checks (Meeting topic: qa)" | 14:09 | |
| gmann | #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:10 |
| gmann | #link https://zuul.openstack.org/builds?job_name=tempest-all&job_name=tempest-full-oslo-master&pipeline=periodic | 14:10 |
| gmann | its all green | 14:10 |
| gmann | tempest-all also green now | 14:10 |
| kopecmartin | \o/ | 14:10 |
| gmann | #topic Sub Teams highlights (Sub Teams means individual projects under QA program) | 14:10 |
| *** openstack changes topic to "Sub Teams highlights (Sub Teams means individual projects under QA program) (Meeting topic: qa)" | 14:10 | |
| gmann | Tempest | 14:10 |
| gmann | we need to cut new release for tempest run new interface to be used by plugins | 14:10 |
| gmann | I will open the bug for devstack/tempest for plugins not tested with Tempest master, so that we can track that issue separately | 14:11 |
| openstackgerrit | Lajos Katona proposed openstack/tempest master: WIP: Add live migration with trunk test https://review.opendev.org/c/openstack/tempest/+/774689 | 14:11 |
| kopecmartin | perfect | 14:11 |
| gmann | #action gmann to open bug for devstack/tempest for plugins not tested with Tempest master. | 14:11 |
| gmann | and for new release, did your fix merge? | 14:12 |
| kopecmartin | yes | 14:12 |
| gmann | perfect | 14:12 |
| gmann | #link https://review.opendev.org/c/openstack/tempest/+/774835 | 14:12 |
| gmann | few patches I would like to wait for new release | 14:12 |
| gmann | 1. #link https://review.opendev.org/c/openstack/tempest/+/773159 | 14:13 |
| gmann | this is in gate | 14:13 |
| gmann | 2. #link https://review.opendev.org/c/openstack/tempest/+/773177/3 | 14:13 |
| gmann | this I will work today, need only unit test | 14:13 |
| gmann | 3. https://review.opendev.org/c/openstack/tempest/+/774913 | 14:14 |
| gmann | this need devstack patches to merged first | 14:14 |
| gmann | so 3rd I can do after release too if it taking time so let's see | 14:14 |
| gmann | kopecmartin: anything from your side, you want to include in new release? | 14:14 |
| kopecmartin | we merged a ton of patches regarding the scenario manager effort | 14:15 |
| kopecmartin | do we want to mention that in the releasenotes for this release? | 14:15 |
| gmann | not yet until we declare that it is stable | 14:15 |
| gmann | we can cut one more new release for that | 14:15 |
| kopecmartin | ok, makes sense | 14:15 |
| kopecmartin | i don't have probably any patch which should be merged before the relase | 14:16 |
| gmann | cool. | 14:16 |
| kopecmartin | i'll check after the meeting, if any i'll ping you | 14:16 |
| gmann | kopecmartin: that is all from my side, anything you would like to discuss before we close office hour? | 14:16 |
| kopecmartin | it's also worth to mention that we will go forward with patches removing six | 14:16 |
| kopecmartin | #link https://review.opendev.org/q/project:openstack/tempest+status:open+topic:remove-six | 14:16 |
| gmann | ah right | 14:17 |
| kopecmartin | I made a few comments there | 14:17 |
| gmann | six removal is all good now and we can merge them anytime, do not need to wait for release time or so | 14:17 |
| kopecmartin | good | 14:17 |
| gmann | if we do not hear anything from authors in this week then either of us can fix those and merge. | 14:18 |
| gmann | let me add actions | 14:18 |
| gmann | #action gmann to cut the Tempest new release after required patches merged | 14:18 |
| gmann | #action kopecmartin /gmann to follow up on six removal patches | 14:19 |
| gmann | anything else? | 14:19 |
| kopecmartin | great | 14:19 |
| kopecmartin | bugs? | 14:19 |
| kopecmartin | there is one with high priority which has a fix | 14:19 |
| gmann | any critical bug you would like to talk? | 14:19 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/642237 | 14:19 |
| gmann | sure | 14:19 |
| gmann | #topic Bug Triage (last 30 min) | 14:20 |
| *** openstack changes topic to "Bug Triage (last 30 min) (Meeting topic: qa)" | 14:20 | |
| gmann | #link https://review.opendev.org/c/openstack/tempest/+/769238/1 | 14:20 |
| gmann | #link https://review.opendev.org/c/openstack/tempest/+/642237 | 14:20 |
| gmann | this one | 14:20 |
| gmann | I need to review that, again forget | 14:20 |
| kopecmartin | sure, np | 14:20 |
| gmann | I think this also need to get merged before release | 14:21 |
| *** slaweq has joined #openstack-qa | 14:21 | |
| kopecmartin | it would be better | 14:21 |
| gmann | 4. release candidate #link https://review.opendev.org/c/openstack/tempest/+/642237 | 14:21 |
| gmann | I will review after my breakfast | 14:21 |
| kopecmartin | this is not a bug, but it's last patch related to pdf documentation QA effort | 14:22 |
| kopecmartin | #link https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227/ | 14:22 |
| kopecmartin | just wanted to bring this up | 14:22 |
| gmann | done still no link in generated doc | 14:23 |
| gmann | #link https://098ce36f4570c4ae2179-f63b2c309fc0040fbb4a377b77794f40.ssl.cf2.rackcdn.com/752227/11/check/nodejs-run-docs/babd34e/docs/ | 14:23 |
| kopecmartin | ah, will fix that today | 14:23 |
| gmann | ack | 14:24 |
| gmann | thanks | 14:24 |
| kopecmartin | last one | 14:24 |
| kopecmartin | #link https://bugs.launchpad.net/tempest/+bug/1668407 | 14:24 |
| openstack | Launchpad bug 1668407 in tempest "os-assisted-volume-snapshots client is not available" [Low,In progress] - Assigned to Martin Kopec (mkopec) | 14:24 |
| kopecmartin | i found a related patch | 14:24 |
| kopecmartin | #link https://review.opendev.org/c/openstack/tempest/+/466902 | 14:24 |
| kopecmartin | however it turned out that the client is meant to be used by cinder internally | 14:25 |
| kopecmartin | which made me wondering if the bug is still valid | 14:25 |
| kopecmartin | as it's quite old | 14:25 |
| kopecmartin | s/client/API | 14:25 |
| gmann | kopecmartin: yeah but there is no better way we differentiate such APIs, update volume attachment is another example and i think we do have test for that | 14:26 |
| gmann | we can add this client. | 14:26 |
| gmann | we have two patches up for that | 14:27 |
| gmann | let me see which one is close and then we can work on those | 14:27 |
| *** Luzi has quit IRC | 14:27 | |
| gmann | either of onwe | 14:27 |
| gmann | any other bug to discuss? | 14:28 |
| kopecmartin | no,that's all | 14:29 |
| *** jparker has joined #openstack-qa | 14:29 | |
| gmann | ok, let's close office hour then | 14:29 |
| gmann | thanks kopecmartin for joining | 14:29 |
| gmann | #endmeeting | 14:29 |
| *** 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/" | 14:29 | |
| openstack | Meeting ended Tue Feb 16 14:29:37 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:29 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-16-14.00.html | 14:29 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-16-14.00.txt | 14:29 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-02-16-14.00.log.html | 14:29 |
| soniya29 | hello | 14:37 |
| soniya29 | i am late i guess | 14:37 |
| kopecmartin | :) | 14:38 |
| kopecmartin | soniya29: don't worry | 14:38 |
| kopecmartin | we discussed basically the new release | 14:38 |
| soniya29 | kopecmartin, ohh | 14:39 |
| gmann | yeah, soniya29 anything you need to discuss | 14:40 |
| soniya29 | gmann, yes | 14:41 |
| soniya29 | i was going through this bug https://storyboard.openstack.org/#!/story/2006070 | 14:41 |
| gmann | yeah | 14:42 |
| openstackgerrit | Lukas Piwowarski proposed openstack/eslint-config-openstack master: Fix building of the pdf documentation https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 14:42 |
| soniya29 | i didn't get which tasks are pending in it, how can i make that out | 14:42 |
| soniya29 | ? | 14:42 |
| gmann | let me check | 14:43 |
| gmann | soniya29: all doc repo merged except this one https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 14:44 |
| soniya29 | okay | 14:44 |
| soniya29 | gmann, apart from that, the bug:- https://storyboard.openstack.org/#!/story/2004771 is inactive from long time | 14:45 |
| *** hemanth_n has quit IRC | 14:45 | |
| *** slaweq has quit IRC | 14:46 | |
| soniya29 | gmann, i think we can close it | 14:46 |
| gmann | soniya29: this patch is still not merged https://review.opendev.org/c/openstack/patrole/+/630647 | 14:47 |
| gmann | I will review that later and leave comment | 14:47 |
| gmann | not sure if that is test issue or server side | 14:47 |
| soniya29 | gmann, the patch is last updated at feb 2019 | 14:48 |
| gmann | yeah its very old, not sure if tests still failing, anyways will check review | 14:48 |
| soniya29 | gmann, sure, if patch is required to get merged, i can help there | 14:49 |
| gmann | cool | 14:50 |
| *** sboyron_ is now known as sboyron | 14:51 | |
| soniya29 | gmann, I am tracking storyboard for starters. Anything else where I can help in patrole stable release? | 14:52 |
| dansmith | yoctozepto: frickler: this minor devstack patch is holding up a tempest test and completion of a glance feature, if you could have a look: https://review.opendev.org/c/openstack/devstack/+/771252 | 14:53 |
| *** rh-jlabarre has joined #openstack-qa | 14:54 | |
| *** rh-jelabarre has quit IRC | 14:54 | |
| gmann | soniya29: yeah, I need to go through the TODO for that, I think few network tests remaining | 14:54 |
| soniya29 | gmann, let me know about them. | 14:55 |
| gmann | sure | 14:59 |
| *** rh-jelabarre has joined #openstack-qa | 15:00 | |
| *** rh-jlabarre has quit IRC | 15:00 | |
| openstackgerrit | Lukas Piwowarski proposed openstack/eslint-config-openstack master: Fix building of the pdf documentation https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 15:06 |
| openstackgerrit | Pierre Riteau proposed openstack/devstack master: Fix DevStack setup on CentOS 8.3 https://review.opendev.org/c/openstack/devstack/+/775850 | 15:07 |
| soniya29 | gmann, there? | 15:10 |
| *** zenkuro has joined #openstack-qa | 15:14 | |
| openstackgerrit | Lukas Piwowarski proposed openstack/eslint-config-openstack master: Fix building of the pdf documentation https://review.opendev.org/c/openstack/eslint-config-openstack/+/752227 | 15:18 |
| *** zenkuro has quit IRC | 15:26 | |
| *** rcernin has joined #openstack-qa | 15:34 | |
| *** rcernin has quit IRC | 15:39 | |
| gmann | soniya29: hi | 15:42 |
| openstackgerrit | Lajos Katona proposed openstack/tempest master: Add live migration with trunk test https://review.opendev.org/c/openstack/tempest/+/774689 | 15:51 |
| gmann | soniya29: replied on this, I do not think we need this https://review.opendev.org/c/openstack/patrole/+/630647/2 | 15:52 |
| frickler | dansmith: I can't find that flag in tempest, do you have a reference to where it is being defined/used? | 15:55 |
| dansmith | frickler: https://review.opendev.org/c/openstack/tempest/+/771071 | 15:56 |
| dansmith | frickler: that depends on the devstack patch | 15:57 |
| dansmith | frickler: I guess I could have arranged that in the opposite way which would have been more obvious, but I'd need more DNM patches to test I think | 15:58 |
| soniya29 | gmann, okay | 16:03 |
| frickler | dansmith: yeah, or make a Needed-By link. or at least keep the topic consistent ;). but with the tempest patch already approved, too, I think we're good to go | 16:05 |
| dansmith | frickler: yeah, sorry :) | 16:05 |
| *** yamamoto has joined #openstack-qa | 16:06 | |
| openstackgerrit | Dan Smith proposed openstack/tempest master: Add image task validation https://review.opendev.org/c/openstack/tempest/+/775679 | 16:10 |
| *** rpittau is now known as rpittau|afk | 16:10 | |
| *** gcheresh has quit IRC | 16:12 | |
| openstackgerrit | Ghanshyam proposed openstack/devstack master: Add a variable to configure the Tempest venv upper constraints https://review.opendev.org/c/openstack/devstack/+/774711 | 16:13 |
| openstackgerrit | Ghanshyam proposed openstack/devstack master: Add a variable to configure the Tempest venv upper constraints https://review.opendev.org/c/openstack/devstack/+/774711 | 16:14 |
| gmann | frickler: ^^ can you check this now | 16:15 |
| *** yamamoto has quit IRC | 16:17 | |
| gmann | dansmith: replied on this, let's make the new endpoints name/filter configurable https://review.opendev.org/c/openstack/tempest/+/770520 | 16:49 |
| dansmith | gmann: okay not sure I understand | 16:50 |
| gmann | dansmith: https://review.opendev.org/c/openstack/tempest/+/770520/4/tempest/clients.py#92 | 16:52 |
| gmann | as of now it is hard coded to what devstack create and that would not work if any other env have different endpoint name | 16:53 |
| gmann | or region or endpoint_type | 16:53 |
| dansmith | gmann: no, I get that one | 16:53 |
| dansmith | makes sense | 16:53 |
| gmann | k | 16:53 |
| dansmith | it's the other "sanity check" comment I'm not sure about | 16:53 |
| gmann | ah | 16:53 |
| gmann | dansmith: if you making import call on new endpoint then auth will take care the exception https://github.com/openstack/tempest/blob/9b6f441fdc2a970410ea631dc1318896349e010f/tempest/lib/auth.py#L557-L576 | 16:54 |
| gmann | dansmith: or you want early skip? | 16:54 |
| gmann | so that we do not create image etc at all | 16:54 |
| dansmith | gmann: right... I could just do all the setup and then find out at the last minute that it's going to fail, but for an expected reaso, | 16:55 |
| dansmith | but everything else seems to skip early if we're not configured for something | 16:55 |
| dansmith | more efficient that way anyway | 16:55 |
| dansmith | gmann: that "_require_import_method" was my attempt to answer feedback from kopecmartin about a couple duplicated lines of initial checking, | 16:56 |
| gmann | dansmith: ok, +1. in that case we can query /v2/info/import or any other fast APi | 16:56 |
| dansmith | which I'm not sure was really worth it for this reason since it pulls the context away, but.. | 16:56 |
| dansmith | gmann: ah sure, that makes sense | 16:57 |
| *** lucasagomes has quit IRC | 17:09 | |
| dansmith | gmann: I guess I should just default the config to None, and check that instead of trying the remote to determine if it's working | 17:16 |
| dansmith | gmann: especially if devstack is going to configure it anyway | 17:16 |
| dansmith | it's just more moving parts and linkage that has to get merged, but I guess that's the right way | 17:16 |
| *** slaweq has joined #openstack-qa | 17:24 | |
| *** slaweq has quit IRC | 17:24 | |
| *** slaweq has joined #openstack-qa | 17:26 | |
| *** slaweq has quit IRC | 17:31 | |
| gmann | dansmith: +1. as it is not default service for image, having it None make sense | 17:32 |
| *** rcernin has joined #openstack-qa | 17:34 | |
| *** gfidente is now known as gfidente|afk | 17:37 | |
| *** rcernin has quit IRC | 17:39 | |
| *** eolivare has quit IRC | 17:44 | |
| openstackgerrit | Merged openstack/bashate master: remove unicode from code https://review.opendev.org/c/openstack/bashate/+/768970 | 17:46 |
| gmann | kopecmartin: this is ready and working now https://review.opendev.org/c/openstack/tempest/+/763981 | 17:54 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Simulate a remote/standalone g-api worker https://review.opendev.org/c/openstack/devstack/+/770487 | 17:59 |
| *** priteau has joined #openstack-qa | 17:59 | |
| priteau | Hi. I discovered today that DevStack is broken on CentOS 8.3, which has been out since early December. Here's a fix: https://review.opendev.org/c/openstack/devstack/+/775850 | 18:00 |
| openstackgerrit | Dan Smith proposed openstack/tempest master: Test glance distributed import https://review.opendev.org/c/openstack/tempest/+/770520 | 18:00 |
| dansmith | gmann: ^ | 18:00 |
| dansmith | gmann: what are the chances of getting the OCaaS patch merged too? that combined with async makes life soooo much better :) | 18:00 |
| gmann | dansmith: ok, i have not looked into that yet. I frickler and ianw reviewed that earlier, let's get weightage from them first | 18:05 |
| dansmith | gmann: I've done a good amount of testing with it on top of async and have no complaints | 18:06 |
| gmann | priteau: ack, we are moving towards centos stream but this release we should fix currnet centos version too. let's wait for gate result | 18:06 |
| gmann | i see | 18:06 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add log for all assigned roles for dynamic cred users https://review.opendev.org/c/openstack/tempest/+/775887 | 18:17 |
| *** zenkuro has joined #openstack-qa | 18:20 | |
| *** jpena is now known as jpena|off | 18:24 | |
| kopecmartin | gmann: one detail regarding https://review.opendev.org/c/openstack/tempest/+/763981 I don't understand, if swift is disabled for pike to train, how is possible that tempest.conf in tempest-full-train-py3 has swift set to True under [service_available] section | 18:24 |
| kopecmartin | gmann: https://ff8a97ef92072468be9b-8be9667f5530aa8ec0c6e8c86da1b76d.ssl.cf5.rackcdn.com/763981/7/check/tempest-full-train-py3/f73072c/controller/logs/tempest_conf.txt | 18:24 |
| *** ralonsoh has quit IRC | 18:25 | |
| gmann | kopecmartin: yeah so branches variant in zuul job is for where that job run not what branch source code it run | 18:26 |
| gmann | kopecmartin: means stable/train job on master gate will pick the master job definition and same job on stable/train gate will pick the rain job definiton | 18:26 |
| gmann | so all stable job on master will enable the swift | 18:27 |
| gmann | and tempest-full-py3 running on stable/train/stein etc gate will disable it as you can see in this stable/stein gate patch https://zuul.opendev.org/t/openstack/build/551eac72fc5c412ba1cfdbb07cb82d48/log/controller/logs/local_conf.txt | 18:28 |
| openstackgerrit | Ghanshyam proposed openstack/devstack stable/train: Fix devstack-platform-xenial by using stable upper-constraints in Tempest venv https://review.opendev.org/c/openstack/devstack/+/774717 | 18:28 |
| kopecmartin | gmann: i see, thanks | 18:29 |
| gmann | kopecmartin: its confusing :) but if you see job inventory then it will be clear how zuul pick jobs based on branches varient | 18:30 |
| *** hamalq has joined #openstack-qa | 18:35 | |
| *** lpiwowar has quit IRC | 18:43 | |
| *** slaweq has joined #openstack-qa | 19:07 | |
| *** dtantsur is now known as dtantsur|afk | 19:20 | |
| openstackgerrit | Dan Smith proposed openstack/tempest master: Test glance distributed import https://review.opendev.org/c/openstack/tempest/+/770520 | 19:32 |
| dansmith | kopecmartin: sorry for the confusion ^ | 19:32 |
| *** whoami-rajat__ has quit IRC | 19:34 | |
| *** rcernin has joined #openstack-qa | 19:35 | |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add other project's admin in dynamic credential https://review.opendev.org/c/openstack/tempest/+/773177 | 19:35 |
| openstackgerrit | Merged openstack/grenade master: remove unicode from code https://review.opendev.org/c/openstack/grenade/+/769693 | 19:40 |
| *** rcernin has quit IRC | 19:40 | |
| kopecmartin | dansmith: no problem ;) thanks | 19:40 |
| *** zenkuro has quit IRC | 20:12 | |
| openstackgerrit | Merged openstack/whitebox-tempest-plugin master: Remove f string formatting https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/775321 | 20:17 |
| *** gfidente|afk has quit IRC | 20:19 | |
| *** jparker_ has joined #openstack-qa | 21:10 | |
| *** jparker has quit IRC | 21:12 | |
| openstackgerrit | Merged openstack/devstack master: Always verify os_glance reserved namespace https://review.opendev.org/c/openstack/devstack/+/771252 | 21:16 |
| *** rcernin has joined #openstack-qa | 21:20 | |
| *** rcernin has quit IRC | 21:29 | |
| *** rcernin has joined #openstack-qa | 21:34 | |
| openstackgerrit | Merged openstack/tempest master: Test glance reserved properties https://review.opendev.org/c/openstack/tempest/+/771071 | 21:37 |
| openstackgerrit | Merged openstack/tempest master: Make create_user domain-aware for V3CredsClient https://review.opendev.org/c/openstack/tempest/+/642237 | 21:38 |
| *** rcernin has quit IRC | 21:39 | |
| *** nweinber has quit IRC | 21:47 | |
| *** sboyron has quit IRC | 21:47 | |
| lxkong | Hi there, trove devstack installation starts to fail after this patch merged https://review.opendev.org/c/openstack/devstack/+/771505, see https://236d08ab34093be5198b-231091edc31f40b3d022f1a19bae7260.ssl.cf2.rackcdn.com/775928/1/check/trove-functional-mysql/279f027/job-output.txt | 21:57 |
| lxkong | Can anyone please help to tell me what we should fix this in Trove? Or is that a incompatible change for devstack? | 21:58 |
| lxkong | I can see the order of recreating nova databases is changed | 22:00 |
| lxkong | dansmith, hi, i see you are the author of that patch :-) | 22:01 |
| dansmith | lxkong: looking | 22:01 |
| lxkong | dansmith, ty | 22:01 |
| dansmith | lxkong: this is disabled by default, so you must be inheriting from some job that has it on? | 22:02 |
| dansmith | oh nevermind, | 22:02 |
| dansmith | it's off but still broken I see | 22:02 |
| lxkong | not sure if that's related, in Trove, we have `CELLSV2_SETUP=singleconductor` | 22:03 |
| dansmith | lxkong: what is different about this job from normal? it's failing in nova db setup, so something has to be different | 22:03 |
| dansmith | lxkong: ah okay, is there a reason or just because it's always been that way? | 22:03 |
| lxkong | This is a log that was successful days ago: https://437f0305aaeb110f17b8-ca10241a0f1644ffcc712dd8c5919b75.ssl.cf5.rackcdn.com/774762/1/check/trove-tempest/a063436/job-output.txt | 22:03 |
| lxkong | we didn't change anything related to trove devstack plugin or settigns | 22:04 |
| lxkong | neither the job definination | 22:04 |
| dansmith | lxkong: right but singleconductor is not the default right? | 22:05 |
| lxkong | no, default is superconductor | 22:05 |
| lxkong | trove changed that ages ago | 22:05 |
| dansmith | right, but for some reason? | 22:06 |
| lxkong | not sure the reason, but from the comment https://github.com/openstack/trove/blob/master/devstack/settings#L81 | 22:06 |
| dansmith | tbh, I'm not sure if any other jobs run with that, so ... | 22:06 |
| dansmith | ah | 22:06 |
| lxkong | not sure if that is still the case, but i'm happy to change anything that is unnecessary | 22:07 |
| dansmith | give me a few to try to figure out why this is failing | 22:09 |
| lxkong | sure | 22:09 |
| *** yamamoto has joined #openstack-qa | 22:11 | |
| *** zenkuro has joined #openstack-qa | 22:11 | |
| openstackgerrit | Dan Smith proposed openstack/devstack master: WIP: Recreate nova cell0 after others https://review.opendev.org/c/openstack/devstack/+/775940 | 22:15 |
| dansmith | lxkong: thinking maybe this ^ | 22:15 |
| dansmith | will try to repo here | 22:15 |
| * lxkong is adding a dependency in his trove patch and recheck | 22:16 | |
| dansmith | lxkong: in that patch, I reordered some nova stuff for speed and cleanliness but I think because of the weird DB arrangement singleconductor has, it's just breaking because cell1 isn't created before we run cell0 | 22:18 |
| lxkong | trove CI job triggered, will let you know if that works or not | 22:20 |
| *** vishalmanchanda has quit IRC | 22:22 | |
| dansmith | lxkong: okay, trying to reproduce your original problem locally and then will test the fix, but I have to run off for a bit now | 22:23 |
| lxkong | dansmith, ok, thanks so much for your help | 22:24 |
| *** rcernin has joined #openstack-qa | 22:24 | |
| dansmith | hmm, just made it past that point, no failure so I dunno | 22:25 |
| dansmith | maybe because my system isn't fully clean | 22:25 |
| dansmith | oh, nm, got it | 22:25 |
| gmann | dansmith: do we need to do 'db sync' after cell0 creation? or it can be done after other cell's DB is created | 22:25 |
| dansmith | gmann: I think it's because with this singleconductor mode, we actually point to cell1 from the cell0 *config* | 22:26 |
| dansmith | hmm, my error is a little different actually | 22:27 |
| gmann | ohk, did not see it is with singleconductor. | 22:27 |
| dansmith | ugh | 22:30 |
| dansmith | what I have now is so much cleaner than it was, so I hate to undo that | 22:30 |
| *** slaweq has quit IRC | 22:30 | |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add alt project persona in dynamic credential https://review.opendev.org/c/openstack/tempest/+/773177 | 22:30 |
| dansmith | because this singleconductor mode is a bit loopy | 22:30 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Add log for all assigned roles for dynamic cred users https://review.opendev.org/c/openstack/tempest/+/775887 | 22:33 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: WIP: Recreate nova cell0 after others https://review.opendev.org/c/openstack/devstack/+/775940 | 22:35 |
| dansmith | lxkong: this ^ is messy but more like what was there for singleconductor mode | 22:36 |
| dansmith | lxkong: if that works, we should merge that to unblock you and I'll try to do a larger refactor of the nova db setup code so this doesn't suck so much | 22:36 |
| dansmith | and again without the typo: | 22:37 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: WIP: Recreate nova cell0 after others https://review.opendev.org/c/openstack/devstack/+/775940 | 22:37 |
| *** dtruong has quit IRC | 22:37 | |
| *** dtruong has joined #openstack-qa | 22:37 | |
| openstackgerrit | Ghanshyam proposed openstack/devstack stable/train: Fix devstack-platform-xenial by using stable upper-constraints in Tempest venv https://review.opendev.org/c/openstack/devstack/+/774717 | 22:46 |
| *** arxcruz|rover has quit IRC | 22:46 | |
| *** arxcruz has joined #openstack-qa | 22:47 | |
| openstackgerrit | Merged openstack/whitebox-tempest-plugin master: Added emulator thread pinning tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/687643 | 23:09 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: WIP: Recreate nova cell0 after others https://review.opendev.org/c/openstack/devstack/+/775940 | 23:14 |
| dansmith | lxkong: this ^ passed locally, so hopefully won't make it any worse, but tbh I didn't see your failure locally | 23:24 |
| *** tosky has quit IRC | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!