Saturday, 2020-04-04

*** paras333 has quit IRC00:09
*** rm_work has quit IRC00:42
*** rm_work has joined #openstack-qa00:45
*** yamamoto has joined #openstack-qa01:01
*** brinzhang has joined #openstack-qa01:05
*** rh-jelabarre has joined #openstack-qa01:53
*** yamamoto has quit IRC02:00
*** yamamoto has joined #openstack-qa02:00
*** rh-jelabarre has quit IRC02:05
*** yamamoto has quit IRC02:10
*** yamamoto has joined #openstack-qa02:14
*** yamamoto has quit IRC02:20
*** yamamoto has joined #openstack-qa02:25
*** yamamoto has quit IRC03:55
*** yamamoto has joined #openstack-qa04:00
*** yamamoto has quit IRC04:05
*** yamamoto has joined #openstack-qa04:16
*** yamamoto has quit IRC04:21
*** igordc has quit IRC04:23
*** evrardjp has quit IRC04:36
*** evrardjp has joined #openstack-qa04:37
*** rfolco has quit IRC04:53
*** yamamoto has joined #openstack-qa05:13
*** yamamoto has quit IRC05:25
*** slaweq has joined #openstack-qa06:16
*** gcheresh has joined #openstack-qa06:45
*** slaweq has quit IRC07:02
*** gcheresh has quit IRC07:17
*** gcheresh has joined #openstack-qa07:18
*** gcheresh has quit IRC07:57
*** gcheresh has joined #openstack-qa08:18
openstackgerritAndreas Jaeger proposed openstack/tempest master: Update hacking for Python3  https://review.opendev.org/71568208:34
openstackgerritAndreas Jaeger proposed openstack/hacking master: Small cleanups  https://review.opendev.org/71744108:44
*** brinzhang has quit IRC08:55
openstackgerritAndreas Jaeger proposed openstack/hacking master: Small cleanups  https://review.opendev.org/71744108:56
openstackgerritAndreas Jaeger proposed openstack/hacking master: Remove usage of six  https://review.opendev.org/71744208:56
*** gcheresh has quit IRC10:09
*** ramishra has quit IRC10:22
*** tosky has joined #openstack-qa10:48
yoctozeptoAJaeger: no idea if you will get my comment from a merged change, so: https://docs.openstack.org/hacking/latest/user/usage.html - still advocates local-check-factory usage11:09
*** redrobot has quit IRC11:15
*** yamamoto has joined #openstack-qa11:38
AJaegeryoctozepto: yes, I know - https://review.opendev.org/71589411:57
*** yamamoto has quit IRC12:26
*** yamamoto has joined #openstack-qa12:34
*** yamamoto has quit IRC12:39
*** yamamoto has joined #openstack-qa12:57
*** yamamoto has quit IRC13:03
*** yamamoto has joined #openstack-qa13:07
*** stephenfin has quit IRC13:25
*** yamamoto has quit IRC14:02
*** yamamoto has joined #openstack-qa14:03
yoctozeptoAJaeger: thanks, I commented on your patches (not core [yet] there)14:08
*** yamamoto has quit IRC14:08
yoctozeptoAJaeger: I have a question regarding https://review.opendev.org/717382 - I did not see it failing on any - could you cross check?14:08
openstackgerritAndreas Jaeger proposed openstack/hacking master: Document new way of registering local plugins  https://review.opendev.org/71589414:21
openstackgerritAndreas Jaeger proposed openstack/hacking master: Small cleanups  https://review.opendev.org/71744114:22
openstackgerritRadosÅ‚aw Piliszek proposed openstack/hacking master: Remove usage of six  https://review.opendev.org/71744214:22
AJaegeryoctozepto: answered - removing "ignore" gives you the default ignore list ;(14:25
yoctozeptoAJaeger: ack! then the recommendation was missing in kolla...14:25
yoctozeptoAJaeger: I did not know that, thanks!14:25
AJaegeryoctozepto: learned it only recently - some repo had it as comment ;)14:26
yoctozeptoAJaeger: haha, good one; definitely against python way but what can one do? :-)14:27
AJaegergmann: could you review core group of hacking, please? Most of the direct members of https://review.opendev.org/#/admin/groups/153,members have left the community. I just pinged the infra team for review and they told me that they in there more as "insurance". Still, got some reviews...14:29
AJaegerclarkb: FYI ^14:30
AJaegergmann: I think we should merge those open changes and release a 3.0.114:30
*** slaweq has joined #openstack-qa14:30
* yoctozepto would not mind being able to +2 in hacking14:31
AJaegeryoctozepto: will you fix kolla-ansible further? I propose to revert to my version and do an update on top of it. But if you want to take over, fine for me as well.14:33
yoctozeptoAJaeger: yeah, I'm precisely doing that; just need to choose new style to pursue14:34
openstackgerritMerged openstack/hacking master: Clean up "is" conditionals  https://review.opendev.org/67860714:34
AJaegerthanks, yoctozepto !14:36
yoctozeptoAJaeger: np, I changed the comment to make me remember14:37
AJaegeryoctozepto: good ideae14:38
AJaegeryoctozepto: strange, I thought there were errors - seems I enabled in any case ;( Not my usual way of doing it. You seemed to have caught the one odd case :( Thanks!14:41
yoctozeptoAJaeger: thanks to you too for handling these matters14:44
yoctozeptohmm, hacking reports to both channels14:44
yoctozeptoAJaeger: for the gods of lolz - it fails in CI but passes here locally... so you were right with using them both14:53
yoctozeptonow why it does not see the problem...14:53
yoctozeptolet's regenerate pep814:54
yoctozeptowell, that made it show them14:56
yoctozeptoso need to decide on one hmm hmm14:56
yoctozeptoended up reverting to yours and adding comment 😂15:01
yoctozeptotoo many non-trivial reformat to satisfy either rule15:02
yoctozeptobest to merge early and follow up when time allows15:02
yoctozeptolast week is oh so much fire fighting...15:03
AJaegeryeah, can be done later - but going to hacking 3 is nice to take in IMHO for python 3 support15:10
*** mordred has quit IRC15:20
*** stephenfin has joined #openstack-qa15:20
*** mordred has joined #openstack-qa15:21
*** stephenfin has quit IRC15:22
*** slaweq has quit IRC15:22
*** stephenfin has joined #openstack-qa15:23
yoctozeptoAJaeger: agreed; weeding out old (usually bad) practice15:27
*** camelCaser has quit IRC15:29
*** stephenfin has quit IRC15:32
*** stephenfin has joined #openstack-qa15:33
openstackgerritAndreas Jaeger proposed openstack/tempest master: Remove universal wheel building  https://review.opendev.org/71749015:36
*** rfolco has joined #openstack-qa15:59
*** rfolco has quit IRC16:04
*** evrardjp has quit IRC16:36
*** evrardjp has joined #openstack-qa16:37
*** yamamoto has joined #openstack-qa16:44
*** yamamoto has quit IRC16:49
AJaegergmann: FYI, I pushed a couple of py27 cleanups with topic:cleanup-py27-support17:21
*** igordc has joined #openstack-qa18:09
*** camelCaser has joined #openstack-qa18:30
*** camelCaser has quit IRC18:30
*** camelCaser has joined #openstack-qa18:31
*** slaweq has joined #openstack-qa19:24
*** slaweq_ has joined #openstack-qa19:37
*** slaweq has quit IRC19:39
*** slaweq_ has quit IRC20:51
gmannAJaeger: sure, review done. I will wait for final U release and then cut the release for hacking. I will review the core list too.22:53
openstackgerritMerged openstack/hacking master: Stop building universal wheels  https://review.opendev.org/71739022:56
openstackgerritMerged openstack/hacking master: Small cleanups  https://review.opendev.org/71744122:56
openstackgerritMerged openstack/hacking master: Remove usage of six  https://review.opendev.org/71744222:56
openstackgerritMerged openstack/hacking master: Document new way of registering local plugins  https://review.opendev.org/71589422:57
*** tosky has quit IRC23:00
*** slaweq_ has joined #openstack-qa23:01
*** slaweq_ has quit IRC23:07
openstackgerritArtom Lifshitz proposed openstack/tempest master: Pass clients to compute base's create_test_server  https://review.opendev.org/71735523:43

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