Re: [openstack-dev] [magnum][requirements][release] The introduction of package py2-ipaddress

2016-04-11 Thread Davanum Srinivas
Thierry, Hongbin,

Looked at current docker-py and came up with this:
https://github.com/docker/docker-py/pull/1033/

Thanks,
Dims

On Mon, Apr 11, 2016 at 3:43 PM, Hongbin Lu <hongbin...@huawei.com> wrote:
> Hi Thierry,
>
> Thanks for your advice. I submitted a patch [1] to downgrade docker-py to 
> 1.7.2. In long term, we will negotiate with upstream maintainers to resolve 
> the module conflicting issue.
>
> [1] https://review.openstack.org/#/c/304296/
>
> Best regards,
> Hongbin
>
>> -Original Message-
>> From: Thierry Carrez [mailto:thie...@openstack.org]
>> Sent: April-11-16 5:28 AM
>> To: openstack-dev@lists.openstack.org
>> Subject: Re: [openstack-dev] [magnum][requirements][release] The
>> introduction of package py2-ipaddress
>>
>> Hongbin Lu wrote:
>> > Hi requirements team,
>> >
>> > In short, the recently introduced package py2-ipaddress [1] seems to
>> > break Magnum. In details, Magnum gate recently broke by an error:
>> > "'\xac\x18\x05\x07' does not appear to be an IPv4 or IPv6 address" [2]
>> > (the gate breakage has been temporarily fixed but we are looking for
>> a
>> > permanent fix [3]). After investigation, I opened a ticket in
>> > Cryptography for help [4]. According to the feedback from
>> Cryptography
>> > community, the problem is from py2-address, which was introduced to
>> > OpenStack recently [1].
>> >
>> > I wonder if we can get any advice from requirements team in this
>> > regards. In particular, what is the proper way to handle the
>> > problematic package?
>> >
>> > [1] https://review.openstack.org/#/c/302539/
>> > [2] https://bugs.launchpad.net/magnum/+bug/1568212
>> > [3] https://bugs.launchpad.net/magnum/+bug/1568427
>> > [4] https://github.com/pyca/cryptography/issues/2870
>>
>> py2-ipaddress was introduced as a dependency by docker-py 1.8.0.
>> Short-term solution would be to cap <1.8.0 in global-requirements
>> (which will make us fallback to 1.7.2 and remove py2-ipaddress).
>>
>> If the two modules are conflicting we should determine which one is the
>> best and converge to it. ipaddress seems a lot more used and pulled by
>> a lot of packages. So long-term solution would be to make docker-py
>> upstream depend on ipaddress instead...
>>
>> --
>> Thierry Carrez (ttx)
>>
>> ___
>> ___
>> OpenStack Development Mailing List (not for usage questions)
>> Unsubscribe: OpenStack-dev-
>> requ...@lists.openstack.org?subject:unsubscribe
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev



-- 
Davanum Srinivas :: https://twitter.com/dims

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [magnum][requirements][release] The introduction of package py2-ipaddress

2016-04-11 Thread Hongbin Lu
Hi Thierry,

Thanks for your advice. I submitted a patch [1] to downgrade docker-py to 
1.7.2. In long term, we will negotiate with upstream maintainers to resolve the 
module conflicting issue.

[1] https://review.openstack.org/#/c/304296/

Best regards,
Hongbin

> -Original Message-
> From: Thierry Carrez [mailto:thie...@openstack.org]
> Sent: April-11-16 5:28 AM
> To: openstack-dev@lists.openstack.org
> Subject: Re: [openstack-dev] [magnum][requirements][release] The
> introduction of package py2-ipaddress
> 
> Hongbin Lu wrote:
> > Hi requirements team,
> >
> > In short, the recently introduced package py2-ipaddress [1] seems to
> > break Magnum. In details, Magnum gate recently broke by an error:
> > "'\xac\x18\x05\x07' does not appear to be an IPv4 or IPv6 address" [2]
> > (the gate breakage has been temporarily fixed but we are looking for
> a
> > permanent fix [3]). After investigation, I opened a ticket in
> > Cryptography for help [4]. According to the feedback from
> Cryptography
> > community, the problem is from py2-address, which was introduced to
> > OpenStack recently [1].
> >
> > I wonder if we can get any advice from requirements team in this
> > regards. In particular, what is the proper way to handle the
> > problematic package?
> >
> > [1] https://review.openstack.org/#/c/302539/
> > [2] https://bugs.launchpad.net/magnum/+bug/1568212
> > [3] https://bugs.launchpad.net/magnum/+bug/1568427
> > [4] https://github.com/pyca/cryptography/issues/2870
> 
> py2-ipaddress was introduced as a dependency by docker-py 1.8.0.
> Short-term solution would be to cap <1.8.0 in global-requirements
> (which will make us fallback to 1.7.2 and remove py2-ipaddress).
> 
> If the two modules are conflicting we should determine which one is the
> best and converge to it. ipaddress seems a lot more used and pulled by
> a lot of packages. So long-term solution would be to make docker-py
> upstream depend on ipaddress instead...
> 
> --
> Thierry Carrez (ttx)
> 
> ___
> ___
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-
> requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [magnum][requirements][release] The introduction of package py2-ipaddress

2016-04-11 Thread Thierry Carrez

Hongbin Lu wrote:

Hi requirements team,

In short, the recently introduced package py2-ipaddress [1] seems to
break Magnum. In details, Magnum gate recently broke by an error:
“'\xac\x18\x05\x07' does not appear to be an IPv4 or IPv6 address” [2]
(the gate breakage has been temporarily fixed but we are looking for a
permanent fix [3]). After investigation, I opened a ticket in
Cryptography for help [4]. According to the feedback from Cryptography
community, the problem is from py2-address, which was introduced to
OpenStack recently [1].

I wonder if we can get any advice from requirements team in this
regards. In particular, what is the proper way to handle the problematic
package?

[1] https://review.openstack.org/#/c/302539/
[2] https://bugs.launchpad.net/magnum/+bug/1568212
[3] https://bugs.launchpad.net/magnum/+bug/1568427
[4] https://github.com/pyca/cryptography/issues/2870


py2-ipaddress was introduced as a dependency by docker-py 1.8.0.
Short-term solution would be to cap <1.8.0 in global-requirements (which 
will make us fallback to 1.7.2 and remove py2-ipaddress).


If the two modules are conflicting we should determine which one is the 
best and converge to it. ipaddress seems a lot more used and pulled by a 
lot of packages. So long-term solution would be to make docker-py 
upstream depend on ipaddress instead...


--
Thierry Carrez (ttx)

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [magnum][requirements][release] The introduction of package py2-ipaddress

2016-04-10 Thread Hongbin Lu
Hi requirements team,

In short, the recently introduced package py2-ipaddress [1] seems to break 
Magnum. In details, Magnum gate recently broke by an error: "'\xac\x18\x05\x07' 
does not appear to be an IPv4 or IPv6 address" [2] (the gate breakage has been 
temporarily fixed but we are looking for a permanent fix [3]). After 
investigation, I opened a ticket in Cryptography for help [4]. According to the 
feedback from Cryptography community, the problem is from py2-address, which 
was introduced to OpenStack recently [1].

I wonder if we can get any advice from requirements team in this regards. In 
particular, what is the proper way to handle the problematic package?

[1] https://review.openstack.org/#/c/302539/
[2] https://bugs.launchpad.net/magnum/+bug/1568212
[3] https://bugs.launchpad.net/magnum/+bug/1568427
[4] https://github.com/pyca/cryptography/issues/2870

Best regards,
Hongbin
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev