netaddr dependency is overweight

Bug #1007151 reported by Gabriel Hurley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Won't Fix
Wishlist
Gabriel Hurley

Bug Description

netaddr has added dependencies on packages such as IPython which are pretty heavyweight. I don't think we're using netaddr for very much (mostly just in test), so if we can replace it and drop the dependencies we should.

Devin Carlen (devcamcar)
Changed in horizon:
status: New → Confirmed
importance: Undecided → Wishlist
milestone: none → folsom-3
assignee: nobody → Nebula (nebula)
Revision history for this message
Tihomir Trifonov (ttrifonov) wrote :

netaddr is used for validation in the IPField in forms that require entering of IP value/range. It is added to test-requires only, because nova already has it as a requirement and installs it. I'm not sure if this is correct that way.. it just left kind of undecided some time ago.. However there is another good library for IP validation - ipaddr that could be used, in case when netaddr is not a good enough...

Revision history for this message
Anton V. Yanchenko (simplylizz) wrote :

Note that author of netaddr agreed to remove this extra dependecies. But this was 2 monthes ago:
https://github.com/drkjam/netaddr/issues/25

Changed in horizon:
assignee: Nebula (nebula) → Gabriel Hurley (gabriel-hurley)
Revision history for this message
Gabriel Hurley (gabriel-hurley) wrote :

Completely non-critical, can wait for RC1

Changed in horizon:
milestone: folsom-3 → folsom-rc1
Changed in horizon:
milestone: folsom-rc1 → none
Revision history for this message
Devin Carlen (devcamcar) wrote :

This issue has been addressed by the netaddr author:

https://github.com/drkjam/netaddr/issues/25#issuecomment-8108308

This is fixed in trunk and will be fixed in release 0.7.8.

Changed in horizon:
milestone: none → folsom-rc1
Revision history for this message
Gabriel Hurley (gabriel-hurley) wrote :

I'm gonna close this out since the change upstream will eventually be picked up on its own.

Changed in horizon:
milestone: folsom-rc1 → none
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.