| tkajinam | kota_, thanks for your review about sbus client patch :-) | 01:25 |
|---|---|---|
| openstackgerrit | Merged openstack/storlets master: Use SBusClient for execute command https://review.opendev.org/676962 | 01:38 |
| openstackgerrit | Merged openstack/storlets master: Cleanup py27 support https://review.opendev.org/718076 | 01:53 |
| kota_ | tkajinam: b | 06:37 |
| tkajinam | kota_, do you want to land https://review.opendev.org/#/c/709357/ during this cycle ? | 06:38 |
| tkajinam | though I should resolve its conflict now :-P | 06:38 |
| kota_ | sounds nice | 06:39 |
| tkajinam | let me update the patch to fix conflict. I'll add you as a reviewer on the patch, so please have a look when you have time | 06:39 |
| kota_ | but wait... currently, swift upstream still reccomends py2, isn't it? | 06:40 |
| kota_ | it may be too early to cleanup all compatibility code | 06:41 |
| tkajinam | I thought swift should support py3 but haven't drop py2, but I might be wrong if you say that | 06:41 |
| tkajinam | kota_, I think we can hold that patch in this cycle, and then merge it in next cycle. | 06:41 |
| kota_ | let me walk through the patch | 06:42 |
| kota_ | swift currently support py3 already but for some known issue, swift doesn't drop py2 yet IIRC. | 06:43 |
| tkajinam | https://review.opendev.org/#/c/692277/ | 06:44 |
| kota_ | if storlets drop py2 compatible code NOW, storlets should go into the hole of known issue | 06:44 |
| tkajinam | https://review.opendev.org/#/c/715612/ | 06:44 |
| tkajinam | https://review.opendev.org/#/c/718076/ | 06:44 |
| tkajinam | ^^^ As far as I see these 3 can affect here | 06:44 |
| tkajinam | maybe we need to revert the last one ? Sorry I put that forward based on my misunderstanding of py3 support status in swift | 06:45 |
| kota_ | wait a sec, meeting now | 06:46 |
| tkajinam | kota_, ok | 06:46 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets master: Drop Py27 support in storlet agent https://review.opendev.org/709357 | 06:46 |
| kota_ | back | 06:53 |
| kota_ | https://review.opendev.org/#/c/718076/7/setup.cfg@9 :/ | 06:55 |
| kota_ | We should NOT merge that | 06:55 |
| kota_ | suggesting py3 migration, mainly support py3 is good but DONOT require >=py3 yet. | 06:56 |
| kota_ | IMO, we may drop py2 testing taht would be broken eventually but... | 06:58 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets master: Revert "Cleanup py27 support" https://review.opendev.org/720131 | 06:59 |
| tkajinam | kota_, ^^^ | 07:00 |
| tkajinam | kota_, yeah I understand the risk of un-tested py2 :-( | 07:00 |
| kota_ | It looks Andreas Jaeger is in reviewers so I just leave my +2 | 07:10 |
| openstackgerrit | Andreas Jaeger proposed openstack/storlets master: Revert "Cleanup py27 support" https://review.opendev.org/720131 | 07:14 |
| tkajinam | kota_, I'll take to gmann as suggested by Andreas. | 07:17 |
| tkajinam | talk to * | 07:17 |
| tkajinam | kota_, do you have any handy ling describing known issue in swift with py3 ? | 07:18 |
| tkajinam | link, sorry | 07:18 |
| kota_ | i don't have that and it's still maintained. | 07:19 |
| kota_ | but the link i pasted to your patch mentioned Swift will keep the py2 gate for a while. | 07:20 |
| kota_ | it's official docs for Ussuri OpenStack community goal. | 07:20 |
| kota_ | tkajinam: https://governance.openstack.org/tc/goals/selected/ussuri/drop-py27.html#projects-keeping-py2-7-support | 07:21 |
| kota_ | unless the state changed in that page, I assume Swift keeps py2. | 07:21 |
| tkajinam | kota_, ahh thanks. I was looking at the wrong line. | 07:21 |
| tkajinam | kota_, I'll send an e-mail on openstack-discuss to request to include storlets there | 07:22 |
| kota_ | tkajinam: IMO, we *could* drop py2 support but not require py3 at the moment. | 07:22 |
| kota_ | how do you think? | 07:23 |
| tkajinam | kota_, yeah I'll mention that point. | 07:24 |
| kota_ | I know it would be gray (unclear) area on the stance. | 07:24 |
| tkajinam | kota_, so we won't test py2, but we won't prohibit py2 | 07:24 |
| kota_ | it's my opinion for Storlets Ussuri. | 07:25 |
| kota_ | until Swift drops py2 | 07:25 |
| kota_ | users can use storlets with py2 by their own risks and maintains. | 07:25 |
| tkajinam | kota_, let's see how gmann responds. | 07:29 |
| tkajinam | kota_, I sent an e-mail on openstack-discuss | 07:31 |
| kota_ | tks | 07:41 |
| *** tkajinam has quit IRC | 08:21 | |
| *** tkajinam has joined #openstack-storlets | 14:43 | |
| *** tkajinam has quit IRC | 15:53 | |
| *** tkajinam has joined #openstack-storlets | 22:59 | |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets master: Revert "[ussuri][goal] Drop python 2.7 support and testing" https://review.opendev.org/720314 | 23:19 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!