<div dir="ltr"><div>Hi Ghanshyam,</div><div><br></div><div>Thanks for your suggestion, and sorry for keeping you waiting.</div><div><br></div><div>Reversions to enable py2 tests are mostly done[1], but currently we are hitting a connection leak issues</div><div>with swiftclient 3.9.0 on Python2.7 (I'll file a bug later).</div><div>Because usage swiftclient 3.9.0 is forced by upper-constrants.txt, now I'm trying to implement</div><div>some workaround to make sure we install 3.8.1 in func tests tox environment.<br></div><div> [1] <a href="https://review.opendev.org/#/q/status:open+project:openstack/storlets+branch:master+topic:py27">https://review.opendev.org/#/q/status:open+project:openstack/storlets+branch:master+topic:py27</a></div><div><br></div><div>I'll let you know once I complete this work, but please allow us to take some time</div><div>to find possible solutions so far.</div><div><br></div><div>Thank you,</div><div>Takashi<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 16, 2020 at 12:35 AM Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com">gmann@ghanshyammann.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> ---- On Wed, 15 Apr 2020 09:41:07 -0500 Takashi Kajinami <<a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a>> wrote ----<br>
 > Hi Ghanshyam,<br>
 > <br>
 > > > Sorry to be late to raise this up, but I'd like to request to postpone completion of "Dropping Python2.7 Support" work in Storlets during this cycle.<br>
 > >  > Storlets is heavily dependent on swift and swift won't drop python 2.7 support during this cycle.This means that we still expect some users who want to run storlets with their swiftrunning on python2 during Ussuri.<br>
 > > <br>
 > > Yeah, this request is too late when we almost done to drop the py2 except swift. You should have brought this during planning phase.<br>
 > > But close interaction with swift and the use case you described make sense too.Again so sorry for our late request.We initially thought that we would require users to use python3 to if they want to have Storlets with Swift,but we found that concerns regarding possible known issues with python3 in swift which Kota pointed out <br>
 > while discussing this in irc.<br>
 > IPython3 should be our first option, so we won't add python2 tests back.<br>
 > However we are still looking for the possibility to keep python2 support mark so thatusers can run Storlets with python2 if they still need to run swift because of its known issues.We haven't dropped any compatibility codes for python2 yet, so adding the partial revert[1] toremove restrictions for Python3 should let operators use Storlets with python2 at their own risk. [1] <a href="https://review.opendev.org/#/c/720131/2" rel="noreferrer" target="_blank">https://review.opendev.org/#/c/720131/2</a><br>
 > > Before I check the possible options, I would like to know is there any OpenStack dependency (library or any other project) in Storlets for python2?From a functional perspective, Storlets depends on the same packages as swift does.(Precisely speaking it uses a subset of swift's dependencies)It doesn't depend on either any libraries like oslo.foo or other projects.<br>
 > The packages listed in test-requirements are all mostly for functional tests, andused to prepare required resources in keystone, swift, etc.<br>
 > <br>
 > I'd really appreciate your help and consideration about this topic.<br>
<br>
Thanks, Takashi for explaining, it looks good to me as this is much closure to swift in terms of deps and use case. <br>
I am ok with keeping py2 support but I think saying py2 is supported without testing, is very confusing and risky. <br>
Instead of breaking users for untested-supported things, we should either test it or drop completely. <br>
In that case, can you also revert the patch dropping the py2 testing/requirements? which is this - <a href="https://review.opendev.org/#/c/692277/5" rel="noreferrer" target="_blank">https://review.opendev.org/#/c/692277/5</a><br>
<br>
After that, I will propose adding the Storlets in python2-keeping-projects list in goal doc.<br>
<br>
-gmann<br>
<br>
<br>
<br>
 > Thank you,Takashi<br>
 > <br>
 > On Wed, Apr 15, 2020 at 10:53 PM Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com" target="_blank">gmann@ghanshyammann.com</a>> wrote:<br>
 > <br>
 > <br>
 > -- <br>
 > ----------Takashi Kajinami<br>
 > Senior Software Maintenance Engineer<br>
 > Customer Experience and Engagement<br>
 > Red Hate-mail: <a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a><br>
 > ---- On Wed, 15 Apr 2020 02:30:08 -0500 Takashi Kajinami <<a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a>> wrote ----<br>
 >  > Hi,<br>
 >  > <br>
 >  > Sorry to be late to raise this up, but I'd like to request to postponecompletion of "Dropping Python2.7 Support" work in Storlets during this cycle.<br>
 >  > Storlets is heavily dependent on swift and swift won't drop python 2.7 support during this cycle.This means that we still expect some users who want to run storlets with their swiftrunning on python2 during Ussuri.<br>
 > <br>
 > Yeah, this request is too late when we almost done to drop the py2 except swift. You should have brought this during planning phase.<br>
 > But close interaction with swift and the use case you described make sense too.<br>
 > <br>
 > Before I check the possible options, I would like to know is there any OpenStack dependency (library or any other project) in Storlets for python2?<br>
 > <br>
 > Few I can see in test requirements need few[1]. <br>
 > <br>
 > [1] <a href="https://github.com/openstack/storlets/blob/a8f1fce69ab236314ae62316482da2f11fe19865/test-requirements.txt" rel="noreferrer" target="_blank">https://github.com/openstack/storlets/blob/a8f1fce69ab236314ae62316482da2f11fe19865/test-requirements.txt</a><br>
 > <br>
 > -gmann<br>
 > <br>
 >  > <br>
 >  > Currently we've completed all python3 compatibility work, and py2 tests are already dropped.However we would like to postpone marking storlets as python3 only(and dropping all py2-3 compatibility codes)<br>
 >  > so that we don't prohibit users from using it in swift with python2.<br>
 >  > <br>
 >  > See the following patch to remove python3 only mark at the moment.(It was initially added, but we want to revert it)<br>
 >  >  [1] <a href="https://review.opendev.org/#/c/720131/2" rel="noreferrer" target="_blank">https://review.opendev.org/#/c/720131/2</a><br>
 >  > Thank you,Takashi<br>
 >  > -- <br>
 >  > ----------Takashi Kajinami<br>
 >  > Senior Software Maintenance Engineer<br>
 >  > Customer Experience and Engagement<br>
 >  > Red Hate-mail: <a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a><br>
 >  > <br>
 > <br>
 > <br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>----------</div><div>Takashi Kajinami<br></div><div>Senior Software Maintenance Engineer<span style="font-family:arial,helvetica,sans-serif"></span><br><div style="outline:currentcolor none medium"><div dir="ltr">Customer Experience and Engagement<br>Red Hat</div><div>e-mail: <a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a></div><div dir="ltr"><br></div></div></div></div></div></div></div>