On 19-12-03 09:02:40, Slawek Kaplonski wrote:

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/e67c368dd65e03a055d93f38d313d97e690929... [3] https://review.opendev.org/#/q/owner:%22Slawek+Kaplonski%22+branch:master+to...

...

On 2 Dec 2019, at 16:16, Matthew Thode mthode@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

Your right, I forgot about that, thanks for the reminder.