| stewie925 | "Failed to import test module: ord.tests.unit.api.controllers.v1.test_api" | 00:00 |
|---|---|---|
| stewie925 | but the module 'ord.tests.unit.api.controllers.v1.test_api' is part of the commit https://review.opendev.org/#/c/677258/5/ord/tests/unit/api/controllers/v1/test_api.py | 00:02 |
| *** markvoelker has joined #openstack-qa | 00:10 | |
| *** rh-jelabarre has quit IRC | 00:14 | |
| *** markvoelker has quit IRC | 00:15 | |
| clarkb | stewie925: urllib.error doesnt exist | 00:16 |
| clarkb | it is finding the file you list just fien but failing to import the code because urllib.error does not exoet | 00:17 |
| stewie925 | hi clarkb sorry for the late response | 00:30 |
| stewie925 | here is my paste.openstack.org (http://paste.openstack.org/show/765324/) to explain the issues - it seems tox py36 is looking at the old version of the code, instead of the committed code. | 00:31 |
| clarkb | stewie925: I'm confused that is a different change than the one above | 00:35 |
| stewie925 | sorry for the confusion clarkb | 00:36 |
| stewie925 | but I hope my pastebin clears things up | 00:36 |
| clarkb | can we start from the beginning. Which change is the one with the problem? | 00:36 |
| *** hongbin has joined #openstack-qa | 00:37 | |
| stewie925 | please ignore my earlier messages, I laid out my concerns in the http://paste.openstack.org/show/765324/ | 00:37 |
| clarkb | ok | 00:37 |
| stewie925 | I just posted the first two test cases | 00:38 |
| stewie925 | for the first test case- the py36 seems to be looking at file "/home/zuul/src/opendev.org/x/ranger-agent/ord/tests/unit/api/controllers/v1/test_api.py" - it does not match the committed code from https://review.opendev.org/#/c/677258/5/ord/tests/unit/api/controllers/v1/test_api.py | 00:39 |
| stewie925 | line 20 from "/home/zuul/src.../test_api.py" shows mox "from mox import stubout" while the actual committed code shows mox3 "from mox3 import stubout" | 00:41 |
| openstackgerrit | zhufl proposed openstack/tempest master: Add response schema validation for volume quota_set https://review.opendev.org/603339 | 00:43 |
| stewie925 | the second test case has the same issue - the tox py36 process reads from "/home/zuul/src/opendev.org/x/ranger-agent/ord/common/utils.py" file, which is different from the committed code "https://review.opendev.org/#/c/677258/5/ord/common/utils.py | 00:44 |
| clarkb | ya the log implies it may be checking out https://review.opendev.org/#/c/670909/ | 00:45 |
| stewie925 | oh yikes | 00:45 |
| stewie925 | clarkb - really appreciate you looking at this | 00:46 |
| stewie925 | let me look for that 670909 in that job-output.txt | 00:46 |
| clarkb | https://zuul.opendev.org/t/openstack/build/b405f247cf3a441b924bce4c35b2f635/log/job-output.txt#274 is where it implies that | 00:46 |
| clarkb | the py27 job did not do that | 00:48 |
| clarkb | py27 checked out the correct commit https://zuul.opendev.org/t/openstack/build/da6aa38314d9494d9528ce122593c831/log/job-output.txt#275 | 00:48 |
| stewie925 | thanks Clark, too bad I am working behind vpn I have no access to the py27 file referenced | 00:50 |
| stewie925 | but I can see the first file line 274 - I do not see 670909 referenced though, strange... | 00:50 |
| stewie925 | 274 2019-08-26 16:36:29.124659 | ubuntu-bionic | opendev.org/x/ranger-agent checked out to: <blank> | 00:51 |
| clarkb | stewie925: ya look at the next line | 00:52 |
| stewie925 | oh - 2019-08-26 16:36:29.124777 | ubuntu-bionic | e796eb53956f1e83ddc53a8dce9cf3957b3b59ef Minor fix - Catch DBConnectionError when update target data | 00:52 |
| stewie925 | thank you - now I understand.... I looked up commit e796eb53956f1e83ddc53a8dce9cf3957b3b59ef and it does point to 670909 | 00:53 |
| stewie925 | I will try to do recheck, maybe this was just a quirk | 00:53 |
| stewie925 | recheck right now | 00:54 |
| stewie925 | thank you very much clarkb | 00:56 |
| clarkb | I'm reading the ara report and it seems that the git repo setup may have pushed that in as master rather than the speculative head | 00:56 |
| clarkb | let me see if I can quickly find anything about that in the zuul logs | 00:57 |
| clarkb | stewie925: I think part of the problem here is it checked out master but should've checked out python3 | 01:01 |
| stewie925 | right, it should have :) | 01:01 |
| clarkb | and sure enough the zuul inventory file shows a checkout value of master | 01:02 |
| clarkb | the python27 job has a proper checkout value | 01:03 |
| clarkb | usually this is becuse you've got a hard set override somewhere /me looks | 01:03 |
| stewie925 | thanks for the step by step explanation | 01:03 |
| stewie925 | oh man, clarkb - my recheck https://review.opendev.org/#/c/677258/ now shows both py27 and py36 passed, but pep8 failed | 01:04 |
| stewie925 | :)) | 01:05 |
| stewie925 | looking in now | 01:05 |
| clarkb | you've only defined py36 and tox in your change too so why is py27 runnin | 01:07 |
| clarkb | ? | 01:07 |
| clarkb | from openstack/project-config/zuul.d/projects.yaml entry | 01:08 |
| stewie925 | hi clarkb, yeah we havent committed another zuul.yaml file yet | 01:09 |
| stewie925 | correction - there is a project-config change we made that has not yet committed - that should fix it | 01:10 |
| clarkb | the recheck used the correct branch | 01:10 |
| clarkb | so whatever caused it doesn't seem to keep causing it or isn't causing it 100% of the time | 01:10 |
| stewie925 | it looks like this second time around, the pep8 is looking at the 670909 commit | 01:10 |
| *** slaweq has joined #openstack-qa | 01:11 | |
| openstackgerrit | wanglbj proposed openstack/tempest master: Modify the list group date https://review.opendev.org/674741 | 01:12 |
| stewie925 | so this is an issue only if its processing a non-master branch | 01:12 |
| clarkb | stewie925: https://zuul.opendev.org/t/openstack/build/ff691a33f03945c49c98be8dac5593a1/log/job-output.txt#276that is the correct commit, it isn't master | 01:12 |
| clarkb | that is the pep8 job | 01:12 |
| *** rcernin has quit IRC | 01:13 | |
| clarkb | stewie925: pep8 failed because you are using python2 to flake8 but have python3 only print syntax | 01:14 |
| clarkb | either keep the print function from future or switch to python3 for the linter tox entry | 01:15 |
| stewie925 | oh we are dropping python2.7 | 01:15 |
| *** slaweq has quit IRC | 01:16 | |
| stewie925 | the first commit's pep8 was good (no errors) but the py tests failed - the recheck is the other way around | 01:16 |
| clarkb | probably because the first one ran against master | 01:16 |
| clarkb | yup confirmed | 01:17 |
| stewie925 | am checking my tox.ini - i do remember I removed all references to py27 and replaced with py36 | 01:17 |
| stewie925 | https://review.opendev.org/#/c/677258/5/tox.ini | 01:18 |
| clarkb | the default is to run against whatever python tox runs under | 01:18 |
| stewie925 | how do I fix that on my end? | 01:18 |
| clarkb | and that must be python2. If you want no python2 then you should explicitly set the python version you want to python3 | 01:18 |
| clarkb | stewie925: set https://tox.readthedocs.io/en/latest/config.html#conf-basepython to python3 iirc | 01:20 |
| stewie925 | ok I see it - I missed the setup.cfg - its still pointing to python2 and python2.7 | 01:21 |
| stewie925 | ahh, thank you clarkb! I will include that to tox.ini as well | 01:21 |
| clarkb | as for why master was chosen before zuul selected the default branch for the repo (master) for some reason | 01:23 |
| clarkb | on the jobs that worked it correctly logs checking out python3 | 01:23 |
| clarkb | ok the problem is that some zuul nodes don't see the python3 branch | 01:28 |
| clarkb | (I don't know why yet) | 01:28 |
| stewie925 | its alright , must be 8 pm on your end | 01:28 |
| stewie925 | 8:28 pm your end | 01:28 |
| clarkb | only 6:29 | 01:29 |
| clarkb | I'm going to write down my notes of what I've found in #zuul so that we can debu further tomorrow | 01:29 |
| stewie925 | ahhahha same here | 01:29 |
| stewie925 | thanks so much Clarkb PST here too | 01:29 |
| stewie925 | finally it passed | 01:42 |
| stewie925 | oops - pep8 still pointing to 670909 | 01:43 |
| stewie925 | thanks again Clarkb - it does look like pep8 is pointing to python3 branch this time | 01:44 |
| stewie925 | ubuntu-bionic | 09c25c49951c2f1c8c6ea5fe60ee24063bb164a8 Update ranger-agent to python3.6/mox3 | 01:44 |
| openstackgerrit | zhufl proposed openstack/tempest master: Use create_image_from_server to create image for server https://review.opendev.org/678711 | 02:04 |
| *** slaweq has joined #openstack-qa | 02:11 | |
| *** rcernin has joined #openstack-qa | 02:13 | |
| *** slaweq has quit IRC | 02:16 | |
| *** stewie925 has quit IRC | 02:16 | |
| *** larainema has joined #openstack-qa | 02:23 | |
| openstackgerrit | wanglbj proposed openstack/tempest master: Modify the list group date https://review.opendev.org/674741 | 03:02 |
| *** markvoelker has joined #openstack-qa | 03:10 | |
| *** markvoelker has quit IRC | 03:15 | |
| openstackgerrit | zhufl proposed openstack/tempest master: Add response schema validation for volume quota_set https://review.opendev.org/603339 | 03:18 |
| *** ramishra has joined #openstack-qa | 03:36 | |
| *** gkadam has joined #openstack-qa | 03:51 | |
| *** gkadam has quit IRC | 03:51 | |
| *** rfolco has quit IRC | 04:03 | |
| *** udesale has joined #openstack-qa | 04:06 | |
| *** markvoelker has joined #openstack-qa | 04:10 | |
| *** slaweq has joined #openstack-qa | 04:11 | |
| *** hongbin has quit IRC | 04:13 | |
| *** markvoelker has quit IRC | 04:15 | |
| *** slaweq has quit IRC | 04:16 | |
| *** raukadah is now known as chkumar|rover | 04:26 | |
| *** dave-mccowan has quit IRC | 04:36 | |
| *** sridharg has joined #openstack-qa | 05:10 | |
| *** sridharg has quit IRC | 05:10 | |
| *** markvoelker has joined #openstack-qa | 05:10 | |
| *** sridharg has joined #openstack-qa | 05:11 | |
| *** slaweq has joined #openstack-qa | 05:11 | |
| *** markvoelker has quit IRC | 05:15 | |
| *** slaweq has quit IRC | 05:16 | |
| *** gcheresh has joined #openstack-qa | 05:18 | |
| *** gcheresh has quit IRC | 05:27 | |
| *** Luzi has joined #openstack-qa | 05:43 | |
| *** e0ne has joined #openstack-qa | 06:06 | |
| *** e0ne has quit IRC | 06:11 | |
| *** slaweq has joined #openstack-qa | 06:11 | |
| *** slaweq has quit IRC | 06:15 | |
| *** surpatil has joined #openstack-qa | 06:22 | |
| *** trident has quit IRC | 07:00 | |
| *** markvoelker has joined #openstack-qa | 07:06 | |
| *** trident has joined #openstack-qa | 07:10 | |
| *** slaweq has joined #openstack-qa | 07:11 | |
| *** markvoelker has quit IRC | 07:15 | |
| *** slaweq has quit IRC | 07:15 | |
| *** markvoelker has joined #openstack-qa | 07:35 | |
| *** dtantsur|afk is now known as dtantsur | 07:37 | |
| *** rcernin has quit IRC | 07:40 | |
| *** jpena|off is now known as jpena | 07:40 | |
| *** markvoelker has quit IRC | 07:40 | |
| *** surpatil has quit IRC | 07:44 | |
| openstackgerrit | zhufl proposed openstack/tempest master: Add response schema validation for volume quota_set https://review.opendev.org/603339 | 07:50 |
| *** slaweq has joined #openstack-qa | 07:52 | |
| *** lucasagomes has joined #openstack-qa | 07:53 | |
| *** gfidente has joined #openstack-qa | 08:00 | |
| *** markvoelker has joined #openstack-qa | 08:40 | |
| *** markvoelker has quit IRC | 08:45 | |
| *** slaweq has quit IRC | 08:53 | |
| openstackgerrit | Sai Kiran proposed openstack/tempest master: Validation of openstack keystone version status https://review.opendev.org/673466 | 09:01 |
| *** gcheresh has joined #openstack-qa | 09:03 | |
| *** surpatil has joined #openstack-qa | 09:04 | |
| *** slaweq has joined #openstack-qa | 09:10 | |
| *** slaweq has quit IRC | 09:14 | |
| *** slaweq has joined #openstack-qa | 09:20 | |
| *** slaweq has quit IRC | 09:25 | |
| *** dtantsur is now known as dtantsur|bbl | 09:31 | |
| *** brinzhang_ has quit IRC | 09:32 | |
| *** brinzhang_ has joined #openstack-qa | 09:32 | |
| *** sshnaidm is now known as sshnaidm|afk | 09:41 | |
| *** brinzhang_ has quit IRC | 09:49 | |
| *** brinzhang_ has joined #openstack-qa | 09:50 | |
| *** lucasagomes has quit IRC | 09:53 | |
| *** lucasagomes has joined #openstack-qa | 09:58 | |
| *** cgoncalves has quit IRC | 10:04 | |
| *** cgoncalves has joined #openstack-qa | 10:04 | |
| *** e0ne has joined #openstack-qa | 10:11 | |
| *** e0ne has quit IRC | 10:26 | |
| *** e0ne has joined #openstack-qa | 10:28 | |
| *** e0ne has quit IRC | 10:32 | |
| openstackgerrit | Will Szumski proposed openstack/tempest master: Add option to set compute quotas for dynamic users https://review.opendev.org/656534 | 10:53 |
| *** slaweq has joined #openstack-qa | 10:55 | |
| *** udesale has quit IRC | 11:02 | |
| *** brinzhang_ has quit IRC | 11:09 | |
| *** brinzhang_ has joined #openstack-qa | 11:09 | |
| *** tesseract has joined #openstack-qa | 11:11 | |
| *** jpena is now known as jpena|lunch | 11:30 | |
| *** slaweq has quit IRC | 11:31 | |
| *** slaweq has joined #openstack-qa | 11:33 | |
| *** slaweq has quit IRC | 11:41 | |
| *** slaweq has joined #openstack-qa | 11:53 | |
| *** slaweq has quit IRC | 11:57 | |
| *** sshnaidm|afk is now known as sshnaidm | 11:59 | |
| *** rh-jelabarre has joined #openstack-qa | 12:05 | |
| *** dtantsur|bbl is now known as dtantsur | 12:08 | |
| *** markvoelker has joined #openstack-qa | 12:10 | |
| *** gcheresh has quit IRC | 12:10 | |
| *** gcheresh has joined #openstack-qa | 12:12 | |
| *** slaweq has joined #openstack-qa | 12:12 | |
| *** slaweq has quit IRC | 12:17 | |
| *** rfolco has joined #openstack-qa | 12:22 | |
| *** gcheresh has quit IRC | 12:26 | |
| *** jpena|lunch is now known as jpena | 12:32 | |
| *** larainema has quit IRC | 12:35 | |
| *** surpatil has quit IRC | 12:39 | |
| *** slaweq has joined #openstack-qa | 12:40 | |
| *** nweinber has joined #openstack-qa | 12:43 | |
| *** slaweq has quit IRC | 13:04 | |
| *** brinzhang_ has quit IRC | 13:12 | |
| *** brinzhang has joined #openstack-qa | 13:14 | |
| *** mriedem has joined #openstack-qa | 13:22 | |
| *** udesale has joined #openstack-qa | 13:32 | |
| *** e0ne has joined #openstack-qa | 13:54 | |
| *** Luzi has quit IRC | 14:02 | |
| *** sshnaidm_ has joined #openstack-qa | 14:29 | |
| *** sshnaidm has quit IRC | 14:30 | |
| *** udesale has quit IRC | 14:32 | |
| *** stewie925 has joined #openstack-qa | 14:34 | |
| *** sshnaidm__ has joined #openstack-qa | 14:34 | |
| *** spsurya has joined #openstack-qa | 14:35 | |
| *** sshnaidm_ has quit IRC | 14:36 | |
| *** sshnaidm__ is now known as sshnaidm | 14:51 | |
| *** sridharg has quit IRC | 15:02 | |
| *** mattw4 has joined #openstack-qa | 15:17 | |
| *** mattw4 has quit IRC | 15:22 | |
| *** chkumar|rover is now known as raukadah | 15:31 | |
| *** e0ne has quit IRC | 15:52 | |
| *** brinzhang has quit IRC | 15:52 | |
| *** brinzhang has joined #openstack-qa | 15:52 | |
| *** sshnaidm is now known as sshnaidm|afk | 15:54 | |
| *** jpena is now known as jpena|off | 16:04 | |
| *** lucasagomes has quit IRC | 16:07 | |
| *** slaweq has joined #openstack-qa | 16:33 | |
| *** dtantsur is now known as dtantsur|afk | 16:42 | |
| *** amotoki is now known as amotoki_ | 16:42 | |
| stewie925 | hello - I'm looking at the setup.cfg for a few Openstack projects (nova/neutron) when defining the 'Programming Language ' - they define separate entries for Python 3 and Python 3.6. | 16:56 |
| stewie925 | why is there a need to define python 3 and python 3.6? | 16:57 |
| *** e0ne has joined #openstack-qa | 17:02 | |
| *** gfidente has quit IRC | 17:02 | |
| clarkb | https://pypi.org/pypi?%3Aaction=list_classifiers is a full list. I think it is just a way to have a heirarchy | 17:04 |
| clarkb | eventually you'll have python 3 and python 3.6 and 3.7 and 3.8 and so on depending on what is tested | 17:05 |
| stewie925 | ahh thank you clarkb! | 17:05 |
| clarkb | stewie925: fyi you caught a zuul bug yesterday fixed in https://review.opendev.org/#/c/678895/3 | 17:05 |
| stewie925 | hehehe clarkb thank you for following up on the zuul bug - you're the man! | 17:06 |
| clarkb | stewie925: https://pypi.org/classifiers/ has more info | 17:07 |
| *** slaweq has quit IRC | 17:27 | |
| *** slaweq has joined #openstack-qa | 17:28 | |
| *** tesseract has quit IRC | 17:30 | |
| *** slaweq has quit IRC | 17:33 | |
| *** rh-jelabarre has quit IRC | 17:36 | |
| *** rh-jelabarre has joined #openstack-qa | 17:38 | |
| *** e0ne has quit IRC | 17:49 | |
| openstackgerrit | Rick Bartra proposed openstack/patrole master: Fix 'use_dynamic_credentials = false' with Neutron extension tests https://review.opendev.org/676979 | 17:57 |
| openstackgerrit | Rick Bartra proposed openstack/patrole master: Fix 'use_dynamic_credentials = false' with Neutron extension tests https://review.opendev.org/676979 | 18:01 |
| *** ramishra has quit IRC | 18:43 | |
| openstackgerrit | Nicolas Helgeson proposed openstack/tempest master: [WIP] Add specifiable clean up function https://review.opendev.org/597733 | 18:45 |
| *** e0ne has joined #openstack-qa | 18:51 | |
| *** markvoelker has quit IRC | 19:07 | |
| *** markvoelker has joined #openstack-qa | 19:08 | |
| *** e0ne has quit IRC | 19:19 | |
| *** gcheresh has joined #openstack-qa | 19:22 | |
| *** artom has quit IRC | 19:36 | |
| *** artom has joined #openstack-qa | 19:43 | |
| *** nweinber has quit IRC | 19:56 | |
| *** artom has quit IRC | 20:23 | |
| *** artom has joined #openstack-qa | 20:25 | |
| *** gcheresh has quit IRC | 20:30 | |
| openstackgerrit | Merged openstack/tempest master: Add response schema validation for volume hosts https://review.opendev.org/603295 | 20:34 |
| openstackgerrit | Nicolas Helgeson proposed openstack/tempest master: [WIP] Add specifiable clean up function https://review.opendev.org/597733 | 20:59 |
| *** spsurya has quit IRC | 21:07 | |
| *** markvoelker has quit IRC | 21:19 | |
| openstackgerrit | Nicolas Helgeson proposed openstack/qa-specs master: Add a user-isolated cleanup option to tempest cleanup https://review.opendev.org/595277 | 21:30 |
| *** mriedem has quit IRC | 21:48 | |
| *** trident has quit IRC | 22:05 | |
| *** trident has joined #openstack-qa | 22:13 | |
| *** rcernin has joined #openstack-qa | 22:15 | |
| *** markvoelker has joined #openstack-qa | 22:15 | |
| *** markvoelker has quit IRC | 22:20 | |
| *** markvoelker has joined #openstack-qa | 23:15 | |
| *** markvoelker has quit IRC | 23:20 | |
| openstackgerrit | Merged openstack/tempest master: Add response schema validation for volume capabilities https://review.opendev.org/605574 | 23:31 |
| *** markvoelker has joined #openstack-qa | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!