Friday, 2020-03-27

*** iokiwi has joined #openstack-qa00:02
*** rfolco has joined #openstack-qa00:06
gmannmtreinish: 'python-requires' support in there for sure pbr  - https://opendev.org/openstack/pbr/src/branch/master/pbr/util.py#L11600:18
gmannmtreinish: clarkb which is from PEP440 version but older like PEP345 used to support 'requires-python also 'https://www.python.org/dev/peps/pep-0345/#requires-python00:19
gmanni am checking if 'requires-python' is still supported or it is being removed.00:21
gmannpbr itself use 'requires-python' so i am still hoping that is supported - https://opendev.org/openstack/pbr/src/branch/master/setup.cfg#L1400:23
*** artom has quit IRC00:41
*** mvkr has quit IRC00:50
*** eandersson8 has joined #openstack-qa01:00
*** eandersson has quit IRC01:01
*** eandersson8 is now known as eandersson01:01
*** mvkr has joined #openstack-qa01:02
*** Liang__ has joined #openstack-qa01:03
openstackgerritGhanshyam Mann proposed openstack/patrole master: Changing the review policy to approve with single +2  https://review.opendev.org/71533501:20
*** paras333 has quit IRC01:33
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing requires-python vs python-requires  https://review.opendev.org/71533901:40
openstackgerritLiang Fang proposed openstack/devstack-plugin-open-cas master: Add tempest tests to zuul  https://review.opendev.org/71377201:44
*** paras333 has joined #openstack-qa01:51
openstackgerritLiang Fang proposed openstack/devstack-plugin-open-cas master: Add tempest tests to zuul  https://review.opendev.org/71377202:14
*** rh-jelabarre has quit IRC02:21
openstackgerritGhanshyam Mann proposed openstack/tempest master: Fixes race condition in test_add_remove_fixed_ip  https://review.opendev.org/69216802:42
*** artom has joined #openstack-qa02:43
*** artom has quit IRC02:43
*** artom has joined #openstack-qa02:44
*** rfolco has quit IRC02:55
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing  https://review.opendev.org/71533902:56
*** gkadam has joined #openstack-qa03:03
*** gkadam_ has joined #openstack-qa03:16
*** gkadam has quit IRC03:16
*** gkadam_ has quit IRC03:17
*** Liang__ is now known as LiangFang03:46
LiangFanghi QA exports, when running tempests test, it seems the volume created for VM would be some local qcow2 file03:54
LiangFanglike:03:54
LiangFang-drive file=/opt/stack/data/nova/instances/d5b93c55-8321-4b12-b772-a475d07dd41d/disk,format=qcow2,if=none,id=drive-virtio-disk0,cache=none03:54
LiangFangIs my understanding correct? thanks03:55
LiangFangI want to add tests for volume local cache feature, it requires the volume should be a block device in host machine, e.g. the remote volume mounts to host machine as /dev/sdd first.03:57
LiangFangIn tempest test, is it possible to mount volume to host as this?03:58
LiangFang/opt/stack/data/nova/instances/d5b93c55-8321-4b12-b772-a475d07dd41d/disk  is not a block device in host machine, so volume local cache feature could not be used03:59
*** paras333 has quit IRC04:44
*** udesale has joined #openstack-qa05:06
*** evrardjp has quit IRC05:36
*** evrardjp has joined #openstack-qa05:36
*** paras333 has joined #openstack-qa05:52
*** gcheresh has joined #openstack-qa05:55
*** paras333 has quit IRC06:22
*** gcheresh has quit IRC06:50
*** paras333 has joined #openstack-qa06:52
*** paras333 has quit IRC06:56
*** gcheresh has joined #openstack-qa07:06
*** ralonsoh has joined #openstack-qa07:08
*** vishalmanchanda has joined #openstack-qa07:16
*** gcheresh has quit IRC07:28
*** paras333 has joined #openstack-qa07:46
*** slaweq has joined #openstack-qa08:00
*** elod has quit IRC08:13
*** jpena|off is now known as jpena08:14
*** paras333 has quit IRC08:16
*** rpittau|afk is now known as rpittau08:23
*** paras333 has joined #openstack-qa08:52
*** AJaeger has joined #openstack-qa08:58
*** AJaeger has left #openstack-qa09:00
*** lucasagomes has joined #openstack-qa09:07
*** paras333 has quit IRC09:22
*** dtantsur|afk is now known as dtantsur09:28
*** tosky has joined #openstack-qa09:29
*** udesale_ has joined #openstack-qa09:35
*** udesale has quit IRC09:38
*** ccamposr__ has joined #openstack-qa09:47
*** ccamposr has quit IRC09:49
*** paras333 has joined #openstack-qa09:52
*** LiangFang has quit IRC09:53
*** jpena has quit IRC09:59
*** gcheresh has joined #openstack-qa10:00
*** jpena has joined #openstack-qa10:06
*** paras333 has quit IRC10:22
*** kopecmartin has quit IRC10:22
*** kopecmartin has joined #openstack-qa10:27
*** paras333 has joined #openstack-qa10:52
*** paras333 has quit IRC10:57
*** ccamposr__ has quit IRC10:59
*** rnoriega_ has quit IRC11:00
*** rnoriega_ has joined #openstack-qa11:03
*** arxcruz is now known as arxcruz|off11:19
*** rpittau is now known as rpittau|bbl11:32
*** yamamoto has joined #openstack-qa11:35
*** ccamposr has joined #openstack-qa11:43
*** ccamposr__ has joined #openstack-qa11:46
*** ccamposr has quit IRC11:47
*** ccamposr__ has quit IRC11:49
*** ccamposr__ has joined #openstack-qa11:49
*** jpena is now known as jpena|lunch12:05
openstackgerritMerged openstack/devstack master: Run pip check at end of devstack  https://review.opendev.org/71494112:10
*** dtantsur is now known as dtantsur|brb12:15
*** rh-jelabarre has joined #openstack-qa12:17
*** elod_ has joined #openstack-qa12:20
*** raildo has joined #openstack-qa12:25
*** paras333 has joined #openstack-qa12:40
*** rfolco has joined #openstack-qa12:48
*** yamamoto has quit IRC12:51
*** elod_ is now known as elod12:53
*** openstackstatus has quit IRC13:01
*** openstack has joined #openstack-qa13:02
*** ChanServ sets mode: +o openstack13:02
*** yamamoto has joined #openstack-qa13:06
*** rpittau|bbl is now known as rpittau13:10
*** paras333 has quit IRC13:10
*** nweinber has joined #openstack-qa13:19
openstackgerritMerged openstack/patrole master: Changing the review policy to approve with single +2  https://review.opendev.org/71533513:22
*** rcernin has quit IRC13:23
*** dtantsur|brb is now known as dtantsur13:40
*** yamamoto has quit IRC13:43
*** yamamoto has joined #openstack-qa13:44
*** yamamoto has quit IRC13:44
*** yamamoto has joined #openstack-qa13:45
*** paras333 has joined #openstack-qa13:48
*** yamamoto has quit IRC13:50
*** yamamoto has joined #openstack-qa13:51
*** gcheresh has quit IRC14:08
*** gcheresh has joined #openstack-qa14:17
*** vishalmanchanda has quit IRC14:32
*** yamamoto has quit IRC14:41
*** yamamoto has joined #openstack-qa14:41
*** yamamoto has quit IRC14:46
*** udesale_ has quit IRC15:20
*** yamamoto has joined #openstack-qa15:20
openstackgerritMerged openstack/patrole master: Update compute tests to adopt new policies  https://review.opendev.org/71136715:34
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing  https://review.opendev.org/71533915:53
openstackgerritMonty Taylor proposed openstack/devstack master: Revert "Run pip check at end of devstack"  https://review.opendev.org/71546716:10
*** melwitt is now known as jgwentworth16:12
openstackgerritMonty Taylor proposed openstack/devstack master: Revert "Run pip check at end of devstack"  https://review.opendev.org/71546716:12
*** slaweq has quit IRC16:18
*** gcheresh has quit IRC16:23
openstackgerritMonty Taylor proposed openstack/devstack master: Stop installing test-requirements with projects  https://review.opendev.org/71546916:23
*** mordred has joined #openstack-qa16:30
mordredgmann: so .... the pip_check patch I added the other day has uncovered a giant can of worms16:30
gmannmordred: ohk, you want fast approve that revert ?16:32
mordredgmann: yeah - we should re-revert it and add it back - because it turns out we ARE going to have a problem with new pip16:33
mordredbut right now it's a spread out problem with no easy solution - so we're going to have to think about it16:33
gmannmordred: done16:34
mordredgmann: also - I pushed up the "stop installing test-reuqirements" patch - because that's where the issues are actually coming from and I can't think of a valid reason why we'd install every project's test-requirements - but I think that one can go through normal discussion :)16:34
mordred(seems like if a project needs test-requirements for their own functional tests that would be part of their functional test setup - but otherwise the tests are run by tempest which has its own setup)16:35
mordredgmann: thanks!!!16:35
gmannmordred: as long as project list requirements in right place. but yes on integrated testing we should not need those.16:35
clarkbmordred: problems due to the dep resolver?16:36
gmannmordred: waiting for gate results and then i am +2 on that16:36
mordredclarkb: yeah16:38
mordredclarkb: becuase we don't apply constraints to linters16:38
clarkbah16:38
mordredclarkb: and we install every project's test-requirements16:38
mordredso linter variation fights - and pip check fails16:39
mordredyou only notice when you run devstack with additional services - my senlin functional job failed - and the one with octavia/designate/neutron-fwaas16:39
mordredbut - we install, uninstall and reinstall flake8 like 8 times16:39
mordredit's impressive16:40
mordredclarkb, gmann: https://review.opendev.org/#/c/715243/ is an sdk patch that now depends-on the devstack patch - so also should show that the patch fixes the issue16:42
*** yamamoto has quit IRC16:43
gmannmordred: clarkb 'requires-python' works fine ? or its 'python-requires'. in  PEP345 it used to be 'requires-python ' and then  'python-requires'. i am not finding if former one still supported. see this - https://github.com/mtreinish/stestr/issues/29216:46
gmannmost of place in openstack we used 'python-requires' and few 'requires-python' including pbr.16:47
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing  https://review.opendev.org/71533916:48
mordredgmann: zomg - is it python-requires?16:56
mordredgmann: if requires-python doesn't work - we should have pbr translate it to python-requires16:56
gmannmordred: 'python-requires' works fine. pbr support that - https://opendev.org/openstack/pbr/src/branch/master/pbr/util.py#L11616:57
mordredgmann: yah - just saying if requires-python is now bogus but in use, we should maybe add support to pbr for translating it16:58
gmannthat also make sense or just use get rid of requires-python as these two name confuse ?16:59
mordredlike: remote:   https://review.opendev.org/715484 Map requires-python to python-requires17:00
mordredyeah - we should also get rid of requires-python17:00
*** lucasagomes has quit IRC17:00
gmann+1.17:02
mtreinishgmann: I guess that means I should merge https://github.com/mtreinish/stestr/pull/293 and push out 3.0.117:11
gmannmtreinish: yeah, or wait for this but for old pbr env you anyways need pull/293 -  https://review.opendev.org/#/c/715484/17:13
mtreinishwell either way I'm gonna need to push out a new release with the correct metadata set17:14
mtreinishand I'd like to do that asap17:15
gmannyeah, i used same way in tempest but not released yet. all other place we are good like os-testr, hacking etc17:15
mtreinishIt's not a good situation either way because now people on python < 3.5 will have to manually exclude 3.0.0, I was planning for it to be done gracefully by pip17:16
gmannmtreinish: anything breaking on 3.0.0 or just you declare no support for <3.5.17:17
gmanni think you removed six rigjt17:17
*** jpena is now known as jpena|off17:28
*** evrardjp has quit IRC17:36
*** evrardjp has joined #openstack-qa17:36
mtreinishgmann: 3.0.0 will not work for anything < 3.517:42
mtreinishit removed six and also the subunit.run based path which provided compat for 2.7-3.417:43
gmann yeah i remember now17:43
mordredmtreinish: someone earlier suggested you could tag the release before 3.0.0 as 4.0.0 and then make a 5.0.0 with the python-requires17:54
mordredI don't know if that's a happy making thought or not though17:54
mtreinishmordred: That would probably work, or things still on < 3.5 could just cap or exclude. stestr has been emitting a warning for 8 months on 2.7 that >=3.0.0 will break you, and it's not like dropping 2.x support ungracefully is new (sphinx for example).18:15
mtreinishthe bigger concern for me though is we've been advertising for months that the 2.x.x release series is the one that will support < py3.5 and doing that would make 4.x.x that instead18:16
mtreinishespecially if I do need to push out a backport release for any reason18:16
gmannmtreinish: i think README and major version change is also enough info to get notice of py2 drop. relying on pip automation of doing magic of fallback or error is ok but reading doc is also user should do. so i do not think it is too bad situation.18:32
*** ramishra has quit IRC18:33
openstackgerritGhanshyam Mann proposed openstack/tempest master: Replace old requires-python with python-requires  https://review.opendev.org/71551318:41
*** rpittau is now known as rpittau|afk18:57
*** gmann is now known as gmann_lunch18:59
mordredgmann_lunch, clarkb: fwiw - the "stop installing test-requirements" patch made some red non-voting jobs green19:05
mordredand when I looked at why they were red - it was related to installing unneeded test-requirements :)19:05
mordredhttps://zuul.opendev.org/t/openstack/build/924593413290440985328502406d6251 is an example19:05
mordredhttps://review.opendev.org/#/c/715469/ fwiw19:06
*** dtantsur is now known as dtantsur|afk19:13
*** ralonsoh has quit IRC19:16
*** gmann_lunch is now known as gmann19:29
gmannmordred: wow, +2.19:30
*** rf0lc0 has joined #openstack-qa19:43
*** whoami-rajat has quit IRC19:44
*** soniya29 has quit IRC19:44
*** whoami-rajat has joined #openstack-qa19:44
*** rfolco has quit IRC19:46
*** elod has quit IRC19:46
*** iurygregory has quit IRC19:46
*** elod has joined #openstack-qa19:46
openstackgerritAndreas Jaeger proposed openstack/bashate master: Update hacking for Python3  https://review.opendev.org/71553520:05
openstackgerritAndreas Jaeger proposed openstack/stackviz master: Update hacking for Python3  https://review.opendev.org/71553820:06
openstackgerritSofia Enriquez proposed openstack/tempest master: WIP: Retype to encrypted type  https://review.opendev.org/71554220:15
*** gcheresh has joined #openstack-qa20:20
*** paras333 has quit IRC20:42
*** nweinber has quit IRC21:07
*** early has quit IRC21:55
*** early has joined #openstack-qa21:58
*** gcheresh has quit IRC22:02
*** raildo has quit IRC22:02
*** early` has joined #openstack-qa22:03
*** early has quit IRC22:04
openstackgerritMerged openstack/devstack master: Revert "Run pip check at end of devstack"  https://review.opendev.org/71546722:08
openstackgerritMerged openstack/tempest master: Fixes race condition in test_add_remove_fixed_ip  https://review.opendev.org/69216822:19
*** rh-jelabarre has quit IRC22:25
*** yamamoto has joined #openstack-qa22:41
*** yamamoto has quit IRC22:45
*** paras333 has joined #openstack-qa22:49

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