Tuesday, 2020-03-31

*** nweinber has joined #openstack-qa00:11
*** nweinber has quit IRC00:15
*** ccamposr has quit IRC00:15
openstackgerritGhanshyam Mann proposed openstack/patrole master: Update compute os-instance-usage-audit-log tests to adopt new policies  https://review.opendev.org/71611100:17
*** yamamoto has joined #openstack-qa00:30
*** paras333 has joined #openstack-qa00:52
*** paras333 has quit IRC00:57
*** Liang__ has joined #openstack-qa01:12
*** paras333 has joined #openstack-qa01:21
*** paras333 has quit IRC01:25
*** yamamoto has quit IRC01:34
*** yamamoto has joined #openstack-qa01:52
*** ramishra has quit IRC01:59
*** ramishra has joined #openstack-qa02:02
*** dulek has quit IRC02:14
*** dulek has joined #openstack-qa02:20
*** paras333 has joined #openstack-qa02:51
openstackgerritGhanshyam Mann proposed openstack/devstack-tools master: Update hacking for Python3  https://review.opendev.org/71564302:57
*** hongbin has joined #openstack-qa03:00
*** hongbin has quit IRC03:05
gmannfrickler: https://review.opendev.org/#/c/715835/03:20
*** toabctl has joined #openstack-qa03:39
*** toabctl has quit IRC04:03
openstackgerritMerged openstack/stackviz master: Update to hacking 3.0  https://review.opendev.org/71598304:07
*** toabctl has joined #openstack-qa04:07
*** paras333 has quit IRC04:08
*** paras333 has joined #openstack-qa04:08
openstackgerritLiang Fang proposed openstack/tempest master: [WIP] Add test for volume local cache  https://review.opendev.org/71576204:15
*** artom has quit IRC04:31
*** evrardjp has quit IRC04:36
*** evrardjp has joined #openstack-qa04:36
*** udesale has joined #openstack-qa04:42
*** paras333 has quit IRC04:51
*** gcheresh has joined #openstack-qa04:52
*** paras333 has joined #openstack-qa04:52
*** paras333 has quit IRC04:53
*** paras333 has joined #openstack-qa04:54
*** dave-mccowan has quit IRC05:03
*** TerianceVi has joined #openstack-qa05:04
*** TViernion has quit IRC05:05
*** paras333 has quit IRC05:09
*** dave-mccowan has joined #openstack-qa05:12
*** Liang__ is now known as LiangFang05:18
*** gcheresh has quit IRC05:53
*** gcheresh has joined #openstack-qa06:22
*** vishalmanchanda has joined #openstack-qa06:39
*** gcheresh has quit IRC06:42
openstackgerritAndreas Jaeger proposed openstack/bashate master: Update hacking for Python3  https://review.opendev.org/71553506:49
openstackgerritAndreas Jaeger proposed openstack/tempest master: Update hacking for Python3  https://review.opendev.org/71568206:50
openstackgerritAndreas Jaeger proposed openstack/tempest-stress master: Update hacking for Python3  https://review.opendev.org/71561306:50
*** slaweq has joined #openstack-qa07:00
*** ccamposr has joined #openstack-qa07:08
*** gcheresh has joined #openstack-qa07:13
*** rpittau|afk is now known as rpittau07:17
*** ccamposr__ has joined #openstack-qa07:22
*** ccamposr has quit IRC07:25
*** iurygregory has quit IRC07:28
*** tosky has joined #openstack-qa07:29
*** iurygregory has joined #openstack-qa07:29
*** gcheresh has quit IRC07:38
*** jpena|off is now known as jpena07:53
*** ralonsoh has joined #openstack-qa08:02
*** lucasagomes has joined #openstack-qa08:13
brinzhang_AJaeger: for https://review.opendev.org/#/c/715625/4 this patch, can you enable the import file order check? In other words, do you know what I want to say?08:16
brinzhang_Ajaeger: We want to check the import order at top of the file, and sorted them, what do you think?08:20
openstackgerritAndrey Volkov proposed openstack/tempest master: Check no specific group present in group list  https://review.opendev.org/71458308:27
*** yamamoto has quit IRC08:28
*** yamamoto has joined #openstack-qa08:33
AJaegerbrinzhang_: it should be enabled. Let me send a test patch08:41
brinzhang_AJeager: From your patch, in the file, it seems not order by right order. such as https://review.opendev.org/#/c/715625/4/cyborgclient/common/utils.py, maybe it should sort by:08:43
brinzhang_import logging08:43
brinzhang_08:43
brinzhang_from oslo_serialization import jsonutils08:43
brinzhang_08:43
brinzhang_from cyborgclient import exceptions as exc08:43
AJaegerbrinzhang_: check https://review.opendev.org/71620908:43
AJaeger(just tested locally)08:43
*** Luzi has joined #openstack-qa08:44
brinzhang_I will pull your patch, and have a test in my local08:44
AJaegerbrinzhang_: I see what you mean...08:45
AJaegerpatch coming for that file...08:46
brinzhang_Ajeager: wait for that, thanks08:46
AJaegerbrinzhang_: updated - and thanks for reaching out08:46
AJaegerseems hacking did not check properly  for that one.08:46
AJaegerbrinzhang_: is https://review.opendev.org/#/c/715951/ fine? That does not change any imports08:48
AJaegerPlease double check08:48
AJaegerbrinzhang_: H306 is enabled in cyborg08:48
brinzhang_AJaeger: yes, we need  "H306  imports not in alphabetical order", that my mean08:51
AJaegerbrinzhang_: and that is enabled!08:52
AJaegerbrinzhang_: my change is not touching that at all08:53
AJaegerbrinzhang_: so, I'm arguing that the change is fine as is - and 716209 shows that it's working.08:54
AJaegerbrinzhang_: Or did I overlook anything?08:54
*** rcernin has quit IRC08:54
brinzhang_AJaeger: in https://review.opendev.org/#/c/715951/1/tox.ini@121 I dont find the H306 check, what do you point?08:55
brinzhang_AJaeger: Listed at https://review.opendev.org/#/c/715951/1/tox.ini@121 Is this not checked?08:58
*** jpena is now known as jpena|away09:00
*** gcheresh has joined #openstack-qa09:00
*** dtantsur|afk is now known as dtantsur09:00
AJaegerbrinzhang_: it's enabled by default as 716209 shows09:02
AJaegerbrinzhang_: line 113 shows disabled extensions, and 115 shows enabled extensions that are disabled by default.09:03
AJaegerbrinzhang_: line 121 shows the local extensions you use09:03
brinzhang_Ajaeger: Forgive me for being unfamiliar with hacviing and look forward to your further explanation09:03
AJaegerbrinzhang_: glad that we can discuss here - easier than in the review.09:04
brinzhang_AJaeger: because of H306 not in ignore(L113), so we enabled it by default, right?09:06
AJaegerbrinzhang_: correct!09:06
brinzhang_AJaeger: ok, got it, will +A for that change.09:07
AJaeger716209 shows the H306 failure, see https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_554/716209/2/check/openstack-tox-pep8/554b0f5/job-output.txt09:08
AJaeger(jobs are still running, just pep8 finished according to http://zuul.opendev.org/t/openstack/status/change/716209,209:08
*** gfidente has joined #openstack-qa09:09
AJaegerbrinzhang_: and please review https://review.opendev.org/715951 as well - thanks!09:09
brinzhang_AJaeger: I know, thanks for you explain09:09
AJaegerbrinzhang_: you're welcome! Have a great day!09:10
brinzhang_AJaeger: the same to you.09:10
*** LiangFang has quit IRC09:16
AJaegerthanks09:17
*** yamamoto has quit IRC09:25
*** yamamoto has joined #openstack-qa09:31
*** yamamoto has quit IRC09:33
*** gcheresh has quit IRC09:39
*** rcernin has joined #openstack-qa09:59
*** jpena|away is now known as jpena10:00
*** ccamposr__ has quit IRC10:04
*** ccamposr__ has joined #openstack-qa10:04
openstackgerritMerged openstack/devstack master: Support extras in Glance Store install  https://review.opendev.org/71583510:05
*** rcernin has quit IRC10:14
*** rnoriega_ has quit IRC10:36
*** jpena has quit IRC10:37
*** jpena has joined #openstack-qa10:43
*** rnoriega_ has joined #openstack-qa10:51
*** ccamposr__ has quit IRC11:04
*** ccamposr__ has joined #openstack-qa11:04
*** rpittau is now known as rpittau|bbl11:05
*** gcheresh has joined #openstack-qa11:23
*** jpena is now known as jpena|lunch11:40
openstackgerritAndreas Jaeger proposed openstack/os-testr master: Update to hacking 3.0  https://review.opendev.org/71626011:44
*** yamamoto has joined #openstack-qa11:49
*** rnoriega_ has quit IRC11:50
*** kopecmartin has quit IRC11:51
*** kopecmartin has joined #openstack-qa11:52
*** kopecmartin has quit IRC11:58
*** jpena|lunch has quit IRC11:59
*** ralonsoh has quit IRC11:59
*** ralonsoh has joined #openstack-qa12:00
*** raildo has joined #openstack-qa12:00
*** kopecmartin has joined #openstack-qa12:01
*** rh-jelabarre has joined #openstack-qa12:03
*** kopecmartin has quit IRC12:06
*** kopecmartin has joined #openstack-qa12:07
*** jpena has joined #openstack-qa12:07
openstackgerritMartin Kopec proposed openstack/tempest master: Add tempest-cleanup ansible role  https://review.opendev.org/69861012:07
*** rnoriega_ has joined #openstack-qa12:08
*** slaweq has quit IRC12:14
*** slaweq_ has joined #openstack-qa12:14
*** rpittau|bbl is now known as rpittau12:15
*** artom has joined #openstack-qa12:17
*** ccamposr__ has quit IRC12:18
*** artom has quit IRC12:18
*** ccamposr__ has joined #openstack-qa12:18
*** artom has joined #openstack-qa12:18
*** rfolco has joined #openstack-qa12:27
*** rh-jelabarre has quit IRC12:31
*** rh-jelabarre has joined #openstack-qa12:31
*** udesale_ has joined #openstack-qa12:38
*** udesale has quit IRC12:41
*** rh-jelabarre has quit IRC12:44
*** rh-jelabarre has joined #openstack-qa12:54
*** nweinber has joined #openstack-qa13:02
toskyquestion before the office hour, just to be sure: when a branchless repository (R) is running also some job against the old branches, and those branches can't deploy using py3, should the branch-specific jobs be removed from R?13:10
*** slaweq_ is now known as slaweq13:11
openstackgerritAndreas Jaeger proposed openstack/patrole master: Update hacking for Python3  https://review.opendev.org/71564413:17
*** rnoriega_ has quit IRC13:22
*** jpena has quit IRC13:25
*** brinzhang has joined #openstack-qa13:28
*** paras333 has joined #openstack-qa13:29
*** jpena has joined #openstack-qa13:31
*** brinzhang_ has quit IRC13:31
*** rnoriega_ has joined #openstack-qa13:32
*** paras333 has quit IRC13:32
*** paras333 has joined #openstack-qa13:33
*** yamamoto has quit IRC13:38
paras333gmann:  kopecmartin : soniya29 : Did we change the meeting schedule today?13:45
soniya29paras333, I don't think scheduled is changed13:46
paras333soniya29:  ack13:46
kopecmartinparas333: seems like we skipped the meeting today, nothing has changed13:46
paras333kopecmartin:  ok, anyways I also don't have much to share as well:)13:46
paras333there was a day light saving happen in europe time so thought to ask:)13:47
masayukigparas333: soniya29 kopecmartin hi, I was thinking there's our office hour today. But we can skip if nobody has things to share :)13:49
paras333masayukig:  I am ok with that13:50
soniya29masayukig: yeah13:50
masayukigparas333: soniya29 me too13:51
*** Luzi has quit IRC13:54
*** yamamoto has joined #openstack-qa13:55
kopecmartinmasayukig: if you have some time, could you check bug triaging etherpad? https://etherpad.openstack.org/p/qa-bug-triage-ussuri13:55
kopecmartinmasayukig: i put there some old bugs i'd need a second opinion, they might be already fixed or not valid anymore13:55
kopecmartinand also we quite a few high priority bugs with reviews ready13:56
toskyI just have the question I asked ~40 minutes ago13:56
masayukigkopecmartin: ack, Thanks. I'll check it tomorrow.13:56
kopecmartinmasayukig: thanks13:57
kopecmartintosky: unfortunately i don't know the answer, but gmann might13:58
masayukigtosky: I'm not sure I get your question correctly. Do you want to run tempest master against a branchless repo(R)?14:03
toskymasayukig: tempest plugin repositories are supposed to test also against the stable branches14:06
toskythose jobs are executed with pythong 3 nowadays, because they are running against master for the tempest plugin (but devstack and services from the stable branch),14:07
toskybut sometimes the older branches don't work with python3, so I was wondering if I should just remove the jobs against stable branches from a certain tempest plugin I'm fixing14:09
masayukigtosky: ah, I see. I agree with you. I think the jobs should be removed from the repository if it doesn't work. But I don't have 100% confident. So, let's wait for gmann's comment14:11
toskythanks14:13
openstackgerritJames Parker proposed openstack/tempest master: Add compute API locked_reason response for 2.73  https://review.opendev.org/71631114:22
*** dave-mccowan has quit IRC14:23
*** dave-mccowan has joined #openstack-qa14:34
openstackgerritRodolfo Alonso Hernandez proposed openstack/tempest master: Remove any reference to "tenant_id" in network  https://review.opendev.org/70793814:38
openstackgerritWitold Bedyk proposed openstack/devstack master: Add python3-dev package for Debian based distros  https://review.opendev.org/71632414:51
*** dave-mccowan has quit IRC15:04
*** dave-mccowan has joined #openstack-qa15:06
gmanntosky: which branch ? if EM you can easily remove otherwise you should keep running. if not running on py3 then you can run py2 job with tempest and plugins on py3 venv.  tempest master tox is basepython as python3 so it will install py3 env on py2 jobs too.15:09
gmanntosky: if tested node (like xenial for rocky) does not have py3.6 that is what tempest master support as min py version then remove.15:10
toskygmann: yeah, it's about pike, queens and rocky; they are still there in sahara-tests, but I guess it's time to remove them15:12
*** gcheresh has quit IRC15:12
toskythanks15:12
gmannyeah, those can be removed as no py3.6 there.15:13
gmannrunning tests on their gate is enough15:13
toskyand in the worst case I can pin an older version of the plugin for those branches (in the service jobs which use them) if needed15:14
gmanntosky: on stable gate right not master ?15:16
toskygmann: yes, stable gates15:17
gmann+115:18
*** vishalmanchanda has quit IRC15:18
*** gcheresh has joined #openstack-qa15:26
openstackgerritJames Parker proposed openstack/tempest master: Add migration uuid API schema validation for 2.59  https://review.opendev.org/71634415:27
*** vishalmanchanda has joined #openstack-qa15:41
*** nweinber has quit IRC15:51
*** nweinber has joined #openstack-qa15:54
*** udesale_ has quit IRC16:01
openstackgerritRodolfo Alonso Hernandez proposed openstack/tempest master: Prevent raising exception if IP address is set in "_hotplug_server"  https://review.opendev.org/71635516:03
*** lucasagomes has quit IRC16:04
*** rpittau is now known as rpittau|afk16:07
*** dtantsur is now known as dtantsur|afk16:08
*** ccamposr has joined #openstack-qa16:29
*** ccamposr__ has quit IRC16:32
*** evrardjp has quit IRC16:36
*** evrardjp has joined #openstack-qa16:36
openstackgerritAndreas Jaeger proposed openstack/tempest master: Update hacking for Python3  https://review.opendev.org/71568216:39
*** dustinc has joined #openstack-qa17:00
*** yamamoto has quit IRC17:01
*** jpena is now known as jpena|off17:02
*** yamamoto has joined #openstack-qa17:03
*** yamamoto has quit IRC17:09
openstackgerritMerged openstack/bashate master: Update hacking for Python3  https://review.opendev.org/71553517:10
openstackgerritMerged openstack/os-testr master: Update to hacking 3.0  https://review.opendev.org/71626017:13
openstackgerritJames Parker proposed openstack/tempest master: Add compute API locked_reason response for 2.73  https://review.opendev.org/71631117:33
openstackgerritJames Parker proposed openstack/tempest master: Add compute API locked_reason response for 2.73  https://review.opendev.org/71631117:59
openstackgerritJames Parker proposed openstack/tempest master: Add migration uuid API schema validation for 2.59  https://review.opendev.org/71634418:01
*** gfidente is now known as gfidente|afk18:13
*** brinzhang_ has joined #openstack-qa18:17
*** brinzhang has quit IRC18:20
AJaegerany idea why tempest-stress now fails? https://review.opendev.org/715613 it errors with "mock not found"18:28
yoctozeptoAJaeger: on it18:33
yoctozepto2020-03-31 18:26:08.924428 | ubuntu-bionic |   File "/home/zuul/src/opendev.org/openstack/tempest-stress/.tox/py37/lib/python3.7/site-packages/tempest/tests/base.py", line 15, in <module>18:34
yoctozeptowell, tox should install proper requirements...18:34
*** ralonsoh has quit IRC18:40
yoctozeptoAJaeger: I wonder whether tempest should not just have it in requirements.txt, then it would get installed there18:40
yoctozeptoby quick code glimpse most projects simply use mock anyway in test-requirements.txt18:41
yoctozeptoAJaeger: but it's interesting that it broke today...18:42
yoctozeptoAJaeger: https://review.opendev.org/71635018:43
AJaegerthanks, yoctozepto !18:44
AJaegerso, need this in tempest-stress as well? Let me try...18:45
yoctozeptoAJaeger: well, tempest-stress does not really use it directly18:46
yoctozeptoAJaeger: I guess it would be cleanest to make tempest list its deps in requirements.txt - tempest is called to run its tests, not to pretty print error messages :-)18:47
yoctozeptomasayukig, gmann: ^ what are your thoughts?18:47
AJaegeryoctozepto: on it, I agree18:49
openstackgerritAndreas Jaeger proposed openstack/tempest master: Require mock  https://review.opendev.org/71638618:50
AJaegeryoctozepto, masayukig, gmann ^18:50
gmannyoctozepto: AJaeger yeah, we can add tempest as req in tempest-stress18:53
yoctozeptogmann: it is req, mock is not18:53
gmannyoctozepto: but mock is not used in tempest-stress right ?18:55
AJaegergmann: see the failure in 71561318:56
AJaegerit fails since tempest uses mock18:56
gmannbut tempest is installed so does mock should be18:57
AJaegergmann: tempest has mock in test-requirements only19:00
AJaegergmann: the change worked earlier and now fails, so something was merged/released that had mock in requires and not anymore19:00
yoctozepto^ agreed19:01
AJaegerpatrole is also suddenly failing, see https://review.opendev.org/#/c/715644/ - worked 36 hours ago and my changes since then should not have an effect19:05
gmannAJaeger: tempest only use mock in tests, issue is tempest-stress use tests/base.py which is internal to tempest and should not be used. fixing that19:09
AJaegergmann: Ah! Thanks19:10
yoctozeptogmann: ack! that makess sense19:10
yoctozeptodoes it also apply to patrole?19:10
gmannAJaeger: patrole is actually going under policy changes for nova new policy, i keep fixing that as soon as nova policy merge - https://review.opendev.org/#/q/topic:bp/policy-defaults-refresh+(status:open+OR+status:merged)19:10
yoctozeptook19:10
AJaegerthanks, gmann - I'll wait for both - happily a few days ;)19:11
AJaegerwill abandon my tempest change19:11
gmanni fixed one and it pass but nova new chage merge before patorle fix merge so i combine them :)19:11
AJaegerhope you catch up eventually ;)19:12
openstackgerritGhanshyam Mann proposed openstack/tempest-stress master: Use oslotest base class instead of tempest tests  https://review.opendev.org/71639819:13
gmannAJaeger: ^^19:13
AJaegerthanks19:14
gmannAJaeger: I will check patorle one later, need to finish nova stuff before feature freeze19:14
AJaegergmann: no stress, thanks19:14
openstackgerritGhanshyam Mann proposed openstack/tempest-stress master: Use oslotest base class instead of tempest tests  https://review.opendev.org/71639819:21
*** dustinc has quit IRC19:23
*** iurygregory has quit IRC19:27
*** vishalmanchanda has quit IRC19:28
*** gcheresh has quit IRC19:32
openstackgerritMerged openstack/tempest-stress master: Use oslotest base class instead of tempest tests  https://review.opendev.org/71639820:08
openstackgerritGhanshyam Mann proposed openstack/patrole master: Update compute instance-usage and agents tests to adopt new policies  https://review.opendev.org/71611120:13
*** paras333 has quit IRC20:26
*** paras333 has joined #openstack-qa20:26
openstackgerritMerged openstack/tempest-stress master: Update hacking for Python3  https://review.opendev.org/71561320:27
*** gcheresh has joined #openstack-qa20:59
*** rcernin has joined #openstack-qa21:02
*** gcheresh has quit IRC21:12
*** nweinber has quit IRC21:16
*** slaweq has quit IRC21:19
*** iurygregory has joined #openstack-qa21:21
*** artom has quit IRC21:39
*** yamamoto has joined #openstack-qa21:45
*** yamamoto has quit IRC21:50
*** raildo has quit IRC22:00
*** ccamposr has quit IRC22:06
*** tosky has quit IRC22:54
*** artom has joined #openstack-qa23:37
*** rh-jelabarre has quit IRC23:43

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!