<div dir="ltr"><div dir="ltr">I have posted a comment on the said patch but I prefer using pyvers in that specific patch because;</div><div> - The change seems to be a backport candidate and using pyvers helps us backport the change</div><div>   to older branches like Train which still supports python 2 IIRC.</div><div> - We still use pyvers in the other modules so it would make sense to have consistent implementation</div><div>   before we actually remove usage of pyvers from all modules.</div><div><br></div><div>Personally I prefer keeping pyvers because it doesn't make much effort but helps us just in case</div><div>we have Python4 in a future (though I don't know when it really comes) or any distro changes</div><div>package name format from python3- to python- or any different style.<br></div><div><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 27, 2021 at 9:59 PM Thomas Goirand <<a href="mailto:zigo@debian.org">zigo@debian.org</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 2/27/21 11:28 AM, Radosław Piliszek wrote:<br>
> Hi Zigo,<br>
> <br>
> On Sat, Feb 27, 2021 at 10:24 AM Thomas Goirand <<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>> wrote:<br>
>> Using the ${pyvers} so we can switch be between Python versions made<br>
>> sense 1 or 2 years ago. However, I'm in the opinion that we should stop<br>
>> using that, and switch to using python3 everywhere directly whenever<br>
>> possible. Though Tobias seems to not agree (see [1]), so I'm raising the<br>
>> topic in the list so we can discuss this together.<br>
>><br>
>> [1] <a href="https://review.opendev.org/c/openstack/puppet-swift/+/777564" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/puppet-swift/+/777564</a><br>
> <br>
> I had a quick look and I believe Tobias is not against not using that<br>
> var but the fact that you renamed the resource from<br>
> python-ceilometermiddleware to ceilometermiddleware which does not<br>
> make sense indeed and could be breaking.<br>
> <br>
> Kind regards,<br>
> <br>
> -yoctozepto<br>
<br>
I'll fix that. But the question remain: shall we deprecate the use of<br>
${pyvers} and use python3 instead?<br>
<br>
Cheers,<br>
<br>
Thomas Goirand (zigo)<br>
<br>
</blockquote></div><div dir="ltr"><div><div dir="ltr"><div><div style="outline:currentcolor none medium"><div dir="ltr"><br></div></div></div></div></div></div></div>