| *** iokiwi has joined #openstack-qa | 00:02 | |
| *** rfolco has joined #openstack-qa | 00:06 | |
| gmann | mtreinish: 'python-requires' support in there for sure pbr - https://opendev.org/openstack/pbr/src/branch/master/pbr/util.py#L116 | 00:18 |
|---|---|---|
| gmann | mtreinish: 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-python | 00:19 |
| gmann | i am checking if 'requires-python' is still supported or it is being removed. | 00:21 |
| gmann | pbr itself use 'requires-python' so i am still hoping that is supported - https://opendev.org/openstack/pbr/src/branch/master/setup.cfg#L14 | 00:23 |
| *** artom has quit IRC | 00:41 | |
| *** mvkr has quit IRC | 00:50 | |
| *** eandersson8 has joined #openstack-qa | 01:00 | |
| *** eandersson has quit IRC | 01:01 | |
| *** eandersson8 is now known as eandersson | 01:01 | |
| *** mvkr has joined #openstack-qa | 01:02 | |
| *** Liang__ has joined #openstack-qa | 01:03 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Changing the review policy to approve with single +2 https://review.opendev.org/715335 | 01:20 |
| *** paras333 has quit IRC | 01:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM: testing requires-python vs python-requires https://review.opendev.org/715339 | 01:40 |
| openstackgerrit | Liang Fang proposed openstack/devstack-plugin-open-cas master: Add tempest tests to zuul https://review.opendev.org/713772 | 01:44 |
| *** paras333 has joined #openstack-qa | 01:51 | |
| openstackgerrit | Liang Fang proposed openstack/devstack-plugin-open-cas master: Add tempest tests to zuul https://review.opendev.org/713772 | 02:14 |
| *** rh-jelabarre has quit IRC | 02:21 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Fixes race condition in test_add_remove_fixed_ip https://review.opendev.org/692168 | 02:42 |
| *** artom has joined #openstack-qa | 02:43 | |
| *** artom has quit IRC | 02:43 | |
| *** artom has joined #openstack-qa | 02:44 | |
| *** rfolco has quit IRC | 02:55 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM: testing https://review.opendev.org/715339 | 02:56 |
| *** gkadam has joined #openstack-qa | 03:03 | |
| *** gkadam_ has joined #openstack-qa | 03:16 | |
| *** gkadam has quit IRC | 03:16 | |
| *** gkadam_ has quit IRC | 03:17 | |
| *** Liang__ is now known as LiangFang | 03:46 | |
| LiangFang | hi QA exports, when running tempests test, it seems the volume created for VM would be some local qcow2 file | 03:54 |
| LiangFang | like: | 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=none | 03:54 |
| LiangFang | Is my understanding correct? thanks | 03:55 |
| LiangFang | I 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 |
| LiangFang | In 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 used | 03:59 |
| *** paras333 has quit IRC | 04:44 | |
| *** udesale has joined #openstack-qa | 05:06 | |
| *** evrardjp has quit IRC | 05:36 | |
| *** evrardjp has joined #openstack-qa | 05:36 | |
| *** paras333 has joined #openstack-qa | 05:52 | |
| *** gcheresh has joined #openstack-qa | 05:55 | |
| *** paras333 has quit IRC | 06:22 | |
| *** gcheresh has quit IRC | 06:50 | |
| *** paras333 has joined #openstack-qa | 06:52 | |
| *** paras333 has quit IRC | 06:56 | |
| *** gcheresh has joined #openstack-qa | 07:06 | |
| *** ralonsoh has joined #openstack-qa | 07:08 | |
| *** vishalmanchanda has joined #openstack-qa | 07:16 | |
| *** gcheresh has quit IRC | 07:28 | |
| *** paras333 has joined #openstack-qa | 07:46 | |
| *** slaweq has joined #openstack-qa | 08:00 | |
| *** elod has quit IRC | 08:13 | |
| *** jpena|off is now known as jpena | 08:14 | |
| *** paras333 has quit IRC | 08:16 | |
| *** rpittau|afk is now known as rpittau | 08:23 | |
| *** paras333 has joined #openstack-qa | 08:52 | |
| *** AJaeger has joined #openstack-qa | 08:58 | |
| *** AJaeger has left #openstack-qa | 09:00 | |
| *** lucasagomes has joined #openstack-qa | 09:07 | |
| *** paras333 has quit IRC | 09:22 | |
| *** dtantsur|afk is now known as dtantsur | 09:28 | |
| *** tosky has joined #openstack-qa | 09:29 | |
| *** udesale_ has joined #openstack-qa | 09:35 | |
| *** udesale has quit IRC | 09:38 | |
| *** ccamposr__ has joined #openstack-qa | 09:47 | |
| *** ccamposr has quit IRC | 09:49 | |
| *** paras333 has joined #openstack-qa | 09:52 | |
| *** LiangFang has quit IRC | 09:53 | |
| *** jpena has quit IRC | 09:59 | |
| *** gcheresh has joined #openstack-qa | 10:00 | |
| *** jpena has joined #openstack-qa | 10:06 | |
| *** paras333 has quit IRC | 10:22 | |
| *** kopecmartin has quit IRC | 10:22 | |
| *** kopecmartin has joined #openstack-qa | 10:27 | |
| *** paras333 has joined #openstack-qa | 10:52 | |
| *** paras333 has quit IRC | 10:57 | |
| *** ccamposr__ has quit IRC | 10:59 | |
| *** rnoriega_ has quit IRC | 11:00 | |
| *** rnoriega_ has joined #openstack-qa | 11:03 | |
| *** arxcruz is now known as arxcruz|off | 11:19 | |
| *** rpittau is now known as rpittau|bbl | 11:32 | |
| *** yamamoto has joined #openstack-qa | 11:35 | |
| *** ccamposr has joined #openstack-qa | 11:43 | |
| *** ccamposr__ has joined #openstack-qa | 11:46 | |
| *** ccamposr has quit IRC | 11:47 | |
| *** ccamposr__ has quit IRC | 11:49 | |
| *** ccamposr__ has joined #openstack-qa | 11:49 | |
| *** jpena is now known as jpena|lunch | 12:05 | |
| openstackgerrit | Merged openstack/devstack master: Run pip check at end of devstack https://review.opendev.org/714941 | 12:10 |
| *** dtantsur is now known as dtantsur|brb | 12:15 | |
| *** rh-jelabarre has joined #openstack-qa | 12:17 | |
| *** elod_ has joined #openstack-qa | 12:20 | |
| *** raildo has joined #openstack-qa | 12:25 | |
| *** paras333 has joined #openstack-qa | 12:40 | |
| *** rfolco has joined #openstack-qa | 12:48 | |
| *** yamamoto has quit IRC | 12:51 | |
| *** elod_ is now known as elod | 12:53 | |
| *** openstackstatus has quit IRC | 13:01 | |
| *** openstack has joined #openstack-qa | 13:02 | |
| *** ChanServ sets mode: +o openstack | 13:02 | |
| *** yamamoto has joined #openstack-qa | 13:06 | |
| *** rpittau|bbl is now known as rpittau | 13:10 | |
| *** paras333 has quit IRC | 13:10 | |
| *** nweinber has joined #openstack-qa | 13:19 | |
| openstackgerrit | Merged openstack/patrole master: Changing the review policy to approve with single +2 https://review.opendev.org/715335 | 13:22 |
| *** rcernin has quit IRC | 13:23 | |
| *** dtantsur|brb is now known as dtantsur | 13:40 | |
| *** yamamoto has quit IRC | 13:43 | |
| *** yamamoto has joined #openstack-qa | 13:44 | |
| *** yamamoto has quit IRC | 13:44 | |
| *** yamamoto has joined #openstack-qa | 13:45 | |
| *** paras333 has joined #openstack-qa | 13:48 | |
| *** yamamoto has quit IRC | 13:50 | |
| *** yamamoto has joined #openstack-qa | 13:51 | |
| *** gcheresh has quit IRC | 14:08 | |
| *** gcheresh has joined #openstack-qa | 14:17 | |
| *** vishalmanchanda has quit IRC | 14:32 | |
| *** yamamoto has quit IRC | 14:41 | |
| *** yamamoto has joined #openstack-qa | 14:41 | |
| *** yamamoto has quit IRC | 14:46 | |
| *** udesale_ has quit IRC | 15:20 | |
| *** yamamoto has joined #openstack-qa | 15:20 | |
| openstackgerrit | Merged openstack/patrole master: Update compute tests to adopt new policies https://review.opendev.org/711367 | 15:34 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM: testing https://review.opendev.org/715339 | 15:53 |
| openstackgerrit | Monty Taylor proposed openstack/devstack master: Revert "Run pip check at end of devstack" https://review.opendev.org/715467 | 16:10 |
| *** melwitt is now known as jgwentworth | 16:12 | |
| openstackgerrit | Monty Taylor proposed openstack/devstack master: Revert "Run pip check at end of devstack" https://review.opendev.org/715467 | 16:12 |
| *** slaweq has quit IRC | 16:18 | |
| *** gcheresh has quit IRC | 16:23 | |
| openstackgerrit | Monty Taylor proposed openstack/devstack master: Stop installing test-requirements with projects https://review.opendev.org/715469 | 16:23 |
| *** mordred has joined #openstack-qa | 16:30 | |
| mordred | gmann: so .... the pip_check patch I added the other day has uncovered a giant can of worms | 16:30 |
| gmann | mordred: ohk, you want fast approve that revert ? | 16:32 |
| mordred | gmann: yeah - we should re-revert it and add it back - because it turns out we ARE going to have a problem with new pip | 16:33 |
| mordred | but right now it's a spread out problem with no easy solution - so we're going to have to think about it | 16:33 |
| gmann | mordred: done | 16:34 |
| mordred | gmann: 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 |
| mordred | gmann: thanks!!! | 16:35 |
| gmann | mordred: as long as project list requirements in right place. but yes on integrated testing we should not need those. | 16:35 |
| clarkb | mordred: problems due to the dep resolver? | 16:36 |
| gmann | mordred: waiting for gate results and then i am +2 on that | 16:36 |
| mordred | clarkb: yeah | 16:38 |
| mordred | clarkb: becuase we don't apply constraints to linters | 16:38 |
| clarkb | ah | 16:38 |
| mordred | clarkb: and we install every project's test-requirements | 16:38 |
| mordred | so linter variation fights - and pip check fails | 16:39 |
| mordred | you only notice when you run devstack with additional services - my senlin functional job failed - and the one with octavia/designate/neutron-fwaas | 16:39 |
| mordred | but - we install, uninstall and reinstall flake8 like 8 times | 16:39 |
| mordred | it's impressive | 16:40 |
| mordred | clarkb, 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 issue | 16:42 |
| *** yamamoto has quit IRC | 16:43 | |
| gmann | mordred: 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/292 | 16:46 |
| gmann | most of place in openstack we used 'python-requires' and few 'requires-python' including pbr. | 16:47 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: DNM: testing https://review.opendev.org/715339 | 16:48 |
| mordred | gmann: zomg - is it python-requires? | 16:56 |
| mordred | gmann: if requires-python doesn't work - we should have pbr translate it to python-requires | 16:56 |
| gmann | mordred: 'python-requires' works fine. pbr support that - https://opendev.org/openstack/pbr/src/branch/master/pbr/util.py#L116 | 16:57 |
| mordred | gmann: yah - just saying if requires-python is now bogus but in use, we should maybe add support to pbr for translating it | 16:58 |
| gmann | that also make sense or just use get rid of requires-python as these two name confuse ? | 16:59 |
| mordred | like: remote: https://review.opendev.org/715484 Map requires-python to python-requires | 17:00 |
| mordred | yeah - we should also get rid of requires-python | 17:00 |
| *** lucasagomes has quit IRC | 17:00 | |
| gmann | +1. | 17:02 |
| mtreinish | gmann: I guess that means I should merge https://github.com/mtreinish/stestr/pull/293 and push out 3.0.1 | 17:11 |
| gmann | mtreinish: yeah, or wait for this but for old pbr env you anyways need pull/293 - https://review.opendev.org/#/c/715484/ | 17:13 |
| mtreinish | well either way I'm gonna need to push out a new release with the correct metadata set | 17:14 |
| mtreinish | and I'd like to do that asap | 17:15 |
| gmann | yeah, i used same way in tempest but not released yet. all other place we are good like os-testr, hacking etc | 17:15 |
| mtreinish | It'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 pip | 17:16 |
| gmann | mtreinish: anything breaking on 3.0.0 or just you declare no support for <3.5. | 17:17 |
| gmann | i think you removed six rigjt | 17:17 |
| *** jpena is now known as jpena|off | 17:28 | |
| *** evrardjp has quit IRC | 17:36 | |
| *** evrardjp has joined #openstack-qa | 17:36 | |
| mtreinish | gmann: 3.0.0 will not work for anything < 3.5 | 17:42 |
| mtreinish | it removed six and also the subunit.run based path which provided compat for 2.7-3.4 | 17:43 |
| gmann | yeah i remember now | 17:43 |
| mordred | mtreinish: 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-requires | 17:54 |
| mordred | I don't know if that's a happy making thought or not though | 17:54 |
| mtreinish | mordred: 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 |
| mtreinish | the 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 instead | 18:16 |
| mtreinish | especially if I do need to push out a backport release for any reason | 18:16 |
| gmann | mtreinish: 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 IRC | 18:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Replace old requires-python with python-requires https://review.opendev.org/715513 | 18:41 |
| *** rpittau is now known as rpittau|afk | 18:57 | |
| *** gmann is now known as gmann_lunch | 18:59 | |
| mordred | gmann_lunch, clarkb: fwiw - the "stop installing test-requirements" patch made some red non-voting jobs green | 19:05 |
| mordred | and when I looked at why they were red - it was related to installing unneeded test-requirements :) | 19:05 |
| mordred | https://zuul.opendev.org/t/openstack/build/924593413290440985328502406d6251 is an example | 19:05 |
| mordred | https://review.opendev.org/#/c/715469/ fwiw | 19:06 |
| *** dtantsur is now known as dtantsur|afk | 19:13 | |
| *** ralonsoh has quit IRC | 19:16 | |
| *** gmann_lunch is now known as gmann | 19:29 | |
| gmann | mordred: wow, +2. | 19:30 |
| *** rf0lc0 has joined #openstack-qa | 19:43 | |
| *** whoami-rajat has quit IRC | 19:44 | |
| *** soniya29 has quit IRC | 19:44 | |
| *** whoami-rajat has joined #openstack-qa | 19:44 | |
| *** rfolco has quit IRC | 19:46 | |
| *** elod has quit IRC | 19:46 | |
| *** iurygregory has quit IRC | 19:46 | |
| *** elod has joined #openstack-qa | 19:46 | |
| openstackgerrit | Andreas Jaeger proposed openstack/bashate master: Update hacking for Python3 https://review.opendev.org/715535 | 20:05 |
| openstackgerrit | Andreas Jaeger proposed openstack/stackviz master: Update hacking for Python3 https://review.opendev.org/715538 | 20:06 |
| openstackgerrit | Sofia Enriquez proposed openstack/tempest master: WIP: Retype to encrypted type https://review.opendev.org/715542 | 20:15 |
| *** gcheresh has joined #openstack-qa | 20:20 | |
| *** paras333 has quit IRC | 20:42 | |
| *** nweinber has quit IRC | 21:07 | |
| *** early has quit IRC | 21:55 | |
| *** early has joined #openstack-qa | 21:58 | |
| *** gcheresh has quit IRC | 22:02 | |
| *** raildo has quit IRC | 22:02 | |
| *** early` has joined #openstack-qa | 22:03 | |
| *** early has quit IRC | 22:04 | |
| openstackgerrit | Merged openstack/devstack master: Revert "Run pip check at end of devstack" https://review.opendev.org/715467 | 22:08 |
| openstackgerrit | Merged openstack/tempest master: Fixes race condition in test_add_remove_fixed_ip https://review.opendev.org/692168 | 22:19 |
| *** rh-jelabarre has quit IRC | 22:25 | |
| *** yamamoto has joined #openstack-qa | 22:41 | |
| *** yamamoto has quit IRC | 22:45 | |
| *** paras333 has joined #openstack-qa | 22:49 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!