<div dir="ltr"><div>By the way, do you want the other fixes included in this bug? I can continue to send the non-py2 fixes if you want them. Just let me know what's relevant for you.<br></div><div><br></div><div>Corey<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 6, 2018 at 1:16 PM, Corey Bryant <span dir="ltr"><<a href="mailto:corey.bryant@canonical.com" target="_blank">corey.bryant@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Fair enough. We'll be dual stack in Ubuntu until we can drop all py2 at once. That will likely be when upstream claims full supported for py3 - <a href="https://wiki.openstack.org/wiki/Python3" target="_blank">https://wiki.openstack.org/<wbr>wiki/Python3</a>.<br></div><div><br></div><div>Thanks,<br></div><div>Corey<br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jun 6, 2018 at 12:54 PM, Debian Bug Tracking System <span dir="ltr"><<a href="mailto:owner@bugs.debian.org" target="_blank">owner@bugs.debian.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">This is an automatic notification regarding your Bug report<br>
which was filed against the magnum package:<br>
<br>
#900904: magnum: Updates from Ubuntu (python2, autopkgtests)<br>
<br>
It has been closed by Thomas Goirand <<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>>.<br>
<br>
Their explanation is attached below along with your original report.<br>
If this explanation is unsatisfactory and you have not received a<br>
better one in a separate message then please contact Thomas Goirand <<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>> by<br>
replying to this email.<br>
<span class="m_-4156192953947364796HOEnZb"><font color="#888888"><br>
<br>
-- <br>
900904: <a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=900904" rel="noreferrer" target="_blank">https://bugs.debian.org/cgi-bi<wbr>n/bugreport.cgi?bug=900904</a><br>
Debian Bug Tracking System<br>
Contact <a href="mailto:owner@bugs.debian.org" target="_blank">owner@bugs.debian.org</a> with problems<br>
</font></span><br><br></div></div>---------- Forwarded message ----------<br>From: Thomas Goirand <<a href="mailto:zigo@debian.org" target="_blank">zigo@debian.org</a>><br>To: <a href="mailto:900904-done@bugs.debian.org" target="_blank">900904-done@bugs.debian.<wbr>org</a><br>Cc: <br>Bcc: <br>Date: Wed, 6 Jun 2018 18:52:10 +0200<br>Subject: Re: Bug#900904: magnum: Updates from Ubuntu (python2, autopkgtests)<br>On 06/06/2018 04:38 PM, Corey Bryant wrote:<br>
> Package: magnum<br>
> Version: 6.0.1-2<br>
> Severity: normal<br>
> Tags: patch<br>
> User: <a href="mailto:ubuntu-devel@lists.ubuntu.com" target="_blank">ubuntu-devel@lists.ubuntu.com</a><br>
> Usertags: origin-ubuntu cosmic ubuntu-patch<br>
> <br>
> Dear Maintainer,<br>
> <br>
> In Ubuntu, the attached patch was applied to achieve the following:<br>
> <br>
> * Merge from Debian unstable. Remaining changes:<br>
> - d/control, d/rules: Add Python 2.7 support and python-magnum package.<br>
> - d/control: Support Python 2.7 first and Python 3.x as alternative.<br>
> magnum-common depends on python-magnum by default and python3-magnum<br>
> as alternative. magnum-api does the same for python-openstackclient.<br>
> - d/control: Drop ${python3:Depends} from magnum-* binary packages<br>
> as it's covered in python3-magnum.<br>
> - d/control: move python(3)-openstackclient dependency from magnum-api<br>
> Depends to python(3)-magnum Depends.<br>
> - d/control: Enable autopkgtest-pkg-python testsuite.<br>
> - d/rules: Add Python 2.7 support.<br>
> - d/rules: Install to package build directories rather than debian/tmp.<br>
> - d/rules: Use pkgos-dh_auto_install instead of python setup.py install.<br>
> This helps name binaries for update-alternatives.<br>
> - d/python(3)-magnum.post*, d/python(3)-magnum.prerm: update-alternatives<br>
> is used to enable correct shebangs for Python binaries.<br>
> - d/python3-magnum.install: Dropped. Not needed.<br>
> - d/tests/*: Add autopkgtests for magnum daemons and shebangs.<br>
> - d/watch: Get tarball from <a href="http://tarballs.openstack.org" rel="noreferrer" target="_blank">tarballs.openstack.org</a>.<br>
> <br>
> Thanks for considering the patch.<br>
<br>
Hi Corey,<br>
<br>
Thanks for your patch.<br>
<br>
Once we've re-added python 2 support, then we would remove it? Is that<br>
the plan? I'm sorry, but I don't feel like moving to a dual-stack Python<br>
support is the way to go. Already, Debian packages are passing all<br>
functional tests using puppet-openstack (well, one single functional<br>
test fails, and I'm currently working on it). So I don't see the need to<br>
add extra complexity for no return.<br>
<br>
The current Debian plan is to completely remove Python 2 support when<br>
Buster is released (including in clients, and all...). Hopefully,<br>
there's going to be more visibility on the Ubuntu side of things by that<br>
time. Anyway, please let me know what's going on. :)<br>
<br>
Cheers,<br>
<br>
Thomas Goirand (zigo)<br><br>---------- Forwarded message ----------<br>From: Corey Bryant <<a href="mailto:corey.bryant@canonical.com" target="_blank">corey.bryant@canonical.com</a>><br>To: Debian Bug Tracking System <<a href="mailto:submit@bugs.debian.org" target="_blank">submit@bugs.debian.org</a>><br>Cc: <br>Bcc: <br>Date: Wed, 06 Jun 2018 10:38:28 -0400<br>Subject: magnum: Updates from Ubuntu (python2, autopkgtests)<br>Package: magnum<br>
Version: 6.0.1-2<br>
Severity: normal<br>
Tags: patch<br>
User: <a href="mailto:ubuntu-devel@lists.ubuntu.com" target="_blank">ubuntu-devel@lists.ubuntu.com</a><br>
Usertags: origin-ubuntu cosmic ubuntu-patch<br>
<br>
Dear Maintainer,<br>
<br>
In Ubuntu, the attached patch was applied to achieve the following:<br>
<br>
* Merge from Debian unstable. Remaining changes:<br>
- d/control, d/rules: Add Python 2.7 support and python-magnum package.<br>
- d/control: Support Python 2.7 first and Python 3.x as alternative.<br>
magnum-common depends on python-magnum by default and python3-magnum<br>
as alternative. magnum-api does the same for python-openstackclient.<br>
- d/control: Drop ${python3:Depends} from magnum-* binary packages<br>
as it's covered in python3-magnum.<br>
- d/control: move python(3)-openstackclient dependency from magnum-api<br>
Depends to python(3)-magnum Depends.<br>
- d/control: Enable autopkgtest-pkg-python testsuite.<br>
- d/rules: Add Python 2.7 support.<br>
- d/rules: Install to package build directories rather than debian/tmp.<br>
- d/rules: Use pkgos-dh_auto_install instead of python setup.py install.<br>
This helps name binaries for update-alternatives.<br>
- d/python(3)-magnum.post*, d/python(3)-magnum.prerm: update-alternatives<br>
is used to enable correct shebangs for Python binaries.<br>
- d/python3-magnum.install: Dropped. Not needed.<br>
- d/tests/*: Add autopkgtests for magnum daemons and shebangs.<br>
- d/watch: Get tarball from <a href="http://tarballs.openstack.org" rel="noreferrer" target="_blank">tarballs.openstack.org</a>.<br>
<br>
Thanks for considering the patch.<br>
<br>
<br>
-- System Information:<br>
Debian Release: buster/sid<br>
APT prefers cosmic<br>
APT policy: (500, 'cosmic'), (500, 'bionic-security')<br>
Architecture: amd64 (x86_64)<br>
Foreign Architectures: i386<br>
<br>
Kernel: Linux 4.15.0-22-generic (SMP w/4 CPU cores)<br>
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE=en_US.UTF-8 (charmap=UTF-8)<br>
Shell: /bin/sh linked to /bin/dash<br>
Init: systemd (via /run/systemd/system)<br>
LSM: AppArmor: enabled<br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>