Need to remove usage of novaclient.v2.security_group_rules

Bug #1676298 reported by Javier Peña
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Undecided
Unassigned
python-openstackclient
Confirmed
High
Dean Troyer

Bug Description

python-novaclient has removed the deprecated security_group_rules APIs in https://review.openstack.org/447707 . These are still being used by openstackclient (see [1]), so it will fail when a new novaclient version is released.

[1] https://github.com/openstack/python-openstackclient/blob/f63a9f402dc3761a1f7e358d92b7e1aa33098c7a/openstackclient/network/v2/security_group_rule.py#L19-L22

Revision history for this message
Javier Peña (jpena-c) wrote :

This also affects Horizon, which uses novaclient.v2.security_group_rules in https://github.com/openstack/horizon/blob/master/openstack_dashboard/api/nova.py#L36

Changed in python-openstackclient:
assignee: nobody → pawnesh kumar (pawnesh.kumar)
Revision history for this message
Akihiro Motoki (amotoki) wrote :

Question to nova team from horizon side.
From nova team perspective, what is the recommended way if horizon supports Nova security group?
If horizon needs to talk nova micro-versioning API directly, it makes nova-network support difficult. Is it time to drop nova-network support in horizon?

Revision history for this message
Akihiro Motoki (amotoki) wrote :

I sent a mail to -dev ML regarding nova-network deprecation policy.
http://lists.openstack.org/pipermail/openstack-dev/2017-March/114674.html

Changed in python-openstackclient:
status: New → Confirmed
status: Confirmed → New
Changed in horizon:
status: New → Confirmed
Revision history for this message
Dean Troyer (dtroyer) wrote :

This really can not be removed as unlike Horizon, we support clouds going back quite a ways (officially only those not EOL but OSC works back past kilo for many things)

We will need to re-implement these functions to support clouds deployed with nova-net.

Changed in python-openstackclient:
status: New → Confirmed
importance: Undecided → High
assignee: pawnesh kumar (pawnesh.kumar) → Dean Troyer (dtroyer)
Revision history for this message
Ying Zuo (yingzuo) wrote :
Changed in horizon:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers