[requirements][os-ken][neutron] update of tinyrpc from 1.0.3 to 1.0.4 breaks neutron-grenade

Slawek Kaplonski skaplons at redhat.com
Tue Dec 3 08:02:40 UTC 2019


Hi,

I opened story board for this [1] but it seems for me that this isn’t os-ken problem. It seems that version 1.0.4 of tinyrpc is not compatible with python 2.7 and this is failing because of [2].
Neutron-grenade job is old py27 job which I already proposed to remove. See [3] for details. So IMO we should simply wait a bit more, remove this old py27 job and than we should be good with new tinyrpc version.

[1] https://storyboard.openstack.org/#!/story/2006975
[2] https://github.com/mbr/tinyrpc/commit/e67c368dd65e03a055d93f38d313d97e69092920
[3] https://review.opendev.org/#/q/owner:%22Slawek+Kaplonski%22+branch:master+topic:drop-py27-support

> On 2 Dec 2019, at 16:16, Matthew Thode <mthode at mthode.org> wrote:
> 
> The only explicit depenency I can find is for os-ken
> 
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> | Repository                                       | Filename                                                 | Line | Text                                               |
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> | openstack/os-ken                                 | requirements.txt                                         |   13 | tinyrpc>=0.6  # RPC library, BGP speaker(net_cntl) |
> | openstack/requirements                           | openstack_requirements/tests/files/upper-constraints.txt |  187 | tinyrpc===0.5                                      |
> | openstack/upstream-institute-virtual-environment | elements/upstream-training/static/tmp/requirements.txt   |  292 | tinyrpc==1.0.3                                     |
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> 
> Though it's included in constraints for a bunch of neutron packages though lower-constraints.txt
> 
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> | Repository                                       | Filename                                                 | Line | Text                                               |
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> | openstack/dragonflow                             | lower-constraints.txt                                    |  151 | tinyrpc==0.8                                       |
> | openstack/networking-bagpipe                     | lower-constraints.txt                                    |  143 | tinyrpc==0.6                                       |
> | openstack/networking-baremetal                   | lower-constraints.txt                                    |  139 | tinyrpc==0.8                                       |
> | openstack/networking-bgpvpn                      | lower-constraints.txt                                    |  165 | tinyrpc==0.6                                       |
> | openstack/networking-generic-switch              | lower-constraints.txt                                    |  134 | tinyrpc==0.6                                       |
> | openstack/networking-hyperv                      | lower-constraints.txt                                    |  131 | tinyrpc==0.6                                       |
> | openstack/networking-l2gw                        | lower-constraints.txt                                    |  137 | tinyrpc==0.6                                       |
> | openstack/networking-odl                         | lower-constraints.txt                                    |  192 | tinyrpc==0.8                                       |
> | openstack/networking-ovn                         | lower-constraints.txt                                    |  145 | tinyrpc==0.6                                       |
> | openstack/networking-powervm                     | lower-constraints.txt                                    |  138 | tinyrpc==0.6                                       |
> | openstack/networking-sfc                         | lower-constraints.txt                                    |  146 | tinyrpc==0.8                                       |
> | openstack/neutron                                | lower-constraints.txt                                    |  147 | tinyrpc==0.6                                       |
> | openstack/neutron-dynamic-routing                | lower-constraints.txt                                    |  141 | tinyrpc==0.6                                       |
> | openstack/neutron-fwaas                          | lower-constraints.txt                                    |  141 | tinyrpc==0.6                                       |
> | openstack/neutron-vpnaas                         | lower-constraints.txt                                    |  144 | tinyrpc==0.6                                       |
> | openstack/os-ken                                 | lower-constraints.txt                                    |  144 | tinyrpc==0.6                                       |
> | openstack/os-ken                                 | requirements.txt                                         |   13 | tinyrpc>=0.6  # RPC library, BGP speaker(net_cntl) |
> | openstack/requirements                           | openstack_requirements/tests/files/upper-constraints.txt |  187 | tinyrpc===0.5                                      |
> | openstack/requirements                           | upper-constraints.txt                                    |  270 | tinyrpc===1.0.3                                    |
> | openstack/tricircle                              | lower-constraints.txt                                    |  149 | tinyrpc==0.6                                       |
> | openstack/upstream-institute-virtual-environment | elements/upstream-training/static/tmp/requirements.txt   |  292 | tinyrpc==1.0.3                                     |
> | x/flame                                          | upper-constraints.txt                                    |  208 | tinyrpc===0.9.3                                    |
> | x/networking-arista                              | lower-constraints.txt                                    |  146 | tinyrpc==0.6                                       |
> | x/networking-fujitsu                             | lower-constraints.txt                                    |  142 | tinyrpc==0.8                                       |
> | x/networking-omnipath                            | lower-constraints.txt                                    |  144 | tinyrpc==0.6                                       |
> | x/networking-opencontrail                        | lower-constraints.txt                                    |  149 | tinyrpc==0.6                                       |
> | x/networking-vsphere                             | lower-constraints.txt                                    |  142 | tinyrpc==0.6                                       |
> | x/neutron-interconnection                        | lower-constraints.txt                                    |  145 | tinyrpc==0.6                                       |
> | x/tap-as-a-service                               | lower-constraints.txt                                    |  139 | tinyrpc==0.8                                       |
> +--------------------------------------------------+----------------------------------------------------------+------+----------------------------------------------------+
> 
> https://github.com/mbr/tinyrpc/compare/1.0.3...1.0.4 isn't too big but does seem to have a refactor in it.
> 
> If you want to depend on a requirements patch for testing, I've created https://review.opendev.org/692915
> for testing the update.
> 
> Thanks,
> 
> -- 
> Matthew Thode

— 
Slawek Kaplonski
Senior software engineer
Red Hat




More information about the openstack-discuss mailing list