Discrepancy between API and CLI on how to enable PD

Bug #1513894 reported by Baodong (Robert) Li
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Medium
Unassigned
python-neutronclient
Won't Fix
Undecided
Unassigned
python-openstackclient
Fix Released
Undecided
Unassigned

Bug Description

Checking api/v2/attributes.py, user can provide a subnetpool_id as "prefix_delegation" to enable PD. refer to https://github.com/openstack/neutron/blob/master/neutron/api/v2/attributes.py#L376. However, the neutron CLI doesn't allow this sincee it's now retrieving the subnetpool before submitting the subnet create. The get of subnetpool id for "prefix_delegation" will not return a valid id. This causes discrepancy between API and CLI on how to enable PD

Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Medium
status: New → Triaged
Henry Gessau (gessau)
Changed in neutron:
status: Triaged → Confirmed
tags: added: ipv6
Changed in neutron:
assignee: nobody → John Davidge (john-davidge)
Revision history for this message
Dr. Jens Harbott (j-harbott) wrote :

Looking at http://docs.openstack.org/mitaka/networking-guide/adv-config-ipv6.html I wonder whether this is really an issue. The usual procedure seems to be to set

ipv6_pd_enabled = True

in neutron.conf which will have the side effect of making the "--use_default_subnetpool" option to automatically set the subnetpool_id to "prefix_delegation".

So what would be the use case where for subnet creation one would need to set the pool_id explicitly instead of using --use_default_subnetpool?

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

This bug is > 180 days without activity. We are unsetting assignee and milestone and setting status to Incomplete in order to allow its expiry in 60 days.

If the bug is still valid, then update the bug status.

Changed in neutron:
assignee: John Davidge (john-davidge) → nobody
status: Confirmed → Incomplete
Revision history for this message
Akihiro Motoki (amotoki) wrote :

The neutron API allows to specify 'prefix_delegation' as subnet-pool ID when creating a subnet [1][2].
Thus it is reasonable that CLI accepts 'prefix_delegation' as subnet pool ID for subnet-create.

[1] https://github.com/openstack/neutron/blob/5e2fa751e830a0ad09e4c6843b78a91ad3533500/neutron/api/v2/attributes.py#L157-L162
[2] validator definition https://github.com/openstack/neutron-lib/blob/f2eb0ecb89001fb403c907b6d96b0e24385a9eef/neutron_lib/api/validators.py#L658-L667

Changed in neutron:
status: Incomplete → Confirmed
status: Confirmed → Incomplete
Changed in python-neutronclient:
status: New → Confirmed
Revision history for this message
Akihiro Motoki (amotoki) wrote :

I believe this affects openstackclient as well.

'prefix_delegation' is a special subnetpool ID and it is not listed in subnetpool list, so we need a special handling when checking a specified subnet pool is available or not.

Changed in python-openstackclient:
assignee: nobody → Reedip (reedip-banerjee)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-openstackclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/491664

Changed in python-openstackclient:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-openstackclient (master)

Reviewed: https://review.openstack.org/491664
Committed: https://git.openstack.org/cgit/openstack/python-openstackclient/commit/?id=04ef8a41acbd45ef03253240934fa07ec170d7f4
Submitter: Jenkins
Branch: master

commit 04ef8a41acbd45ef03253240934fa07ec170d7f4
Author: Reedip <email address hidden>
Date: Tue Aug 8 09:23:44 2017 +0530

    Allow PD as Subnetpool during Subnet creations

    This commit now allows user to specify 'prefix_delegation'
    as a Subnetpool during Subnet creation by using the new
    --use-prefix-delegation option so that the IPv6
    prefixes can be delegated to routers set up by the
    cloud admins.

    Change-Id: I67e5d81c4155db2e3c5c41ee1df77f2d77a17689
    Closes-Bug: #1513894

Changed in python-openstackclient:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-openstackclient (feature/osc4)

Fix proposed to branch: feature/osc4
Review: https://review.openstack.org/504105

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-openstackclient (feature/osc4)
Download full text (9.3 KiB)

Reviewed: https://review.openstack.org/504105
Committed: https://git.openstack.org/cgit/openstack/python-openstackclient/commit/?id=5309bf5f873aba4e994c9011a893987c40d56eda
Submitter: Jenkins
Branch: feature/osc4

commit c7671dd9a41a7e3dcf5086bb180630c3b4ee0978
Author: OpenStack Proposal Bot <email address hidden>
Date: Wed Sep 13 13:02:59 2017 +0000

    Updated from global requirements

    Change-Id: Ibeed9c623a6ef1c641c121a2b5ee75920346056f

commit 95378bb37d2ba29c7055c9510dd2b5e5513e281f
Author: OpenStack Proposal Bot <email address hidden>
Date: Wed Sep 13 00:15:31 2017 +0000

    Updated from global requirements

    Change-Id: I30be8748ef78604eb63c61e1dcf5fc5a740822aa

commit 760e91abcf01180651447373e37dc99fafba8f30
Author: Gage Hugo <email address hidden>
Date: Tue Sep 12 16:19:49 2017 -0500

    Correct import of keystoneauth1 session

    keystoneclient.session has been long deprecated in favor of
    keystoneauth1.session. This change corrects the import in the
    tests to use the correct library's session.

    Change-Id: Ic24ebde59e4b9eb70d6f14c1e0536f8d24f0de73

commit ce468209c4664eb3680abcaddecc103e0a636e72
Author: Dean Troyer <email address hidden>
Date: Sat Sep 9 08:48:58 2017 -0500

    Skip Volume v1 functional tests if v1 not present

    Volume v1 is gone in Queens

    Just skip it for now until DevStack does not create a v1 endpoint

    Change-Id: I2aa2f78b0d5c8ac2048c922c7835e5c4574028cc

commit 104d6d0e316e74cb8a700dea09aa3566a6d8e069
Author: Jens Harbott <email address hidden>
Date: Tue Sep 12 14:21:49 2017 +0000

    Fix output for subnet show

    The use_default_subnet_pool column is never returned by the API, it is
    only being used in the request to create a subnet. So make sure that we
    do not show it when displaying a subnet.

    Change-Id: Ie021149cceb8f89b779ad0f3c13ac60420509671
    Related-Bug: 1668223

commit 8f4440a6de4425afb77f647ae2dca9452b1166f0
Author: OpenStack Proposal Bot <email address hidden>
Date: Wed Sep 6 03:15:36 2017 +0000

    Updated from global requirements

    Change-Id: I1c5bc67d82b7a123530a7e8a6af5ca4f7faf53f0

commit de2af66c1622115dcb28aca88aa62ce5b177c771
Author: OpenStack Proposal Bot <email address hidden>
Date: Fri Sep 1 12:47:11 2017 +0000

    Updated from global requirements

    Change-Id: If52bacff8458010d2762bfac5ecea29f5fb54f01

commit ff91e26983578bf1b77e1d68fb1ae26067eed7ea
Author: lihaijing <email address hidden>
Date: Wed Aug 23 11:23:27 2017 +0800

    Update image cli doc and fix some typos

    Change-Id: I0a8d095e51a96804c97612e28fac2d00aa94c638
    Closes-Bug: #1711284

commit e62587625f80b4496b2b092be7efb00153d43a23
Author: OpenStack Proposal Bot <email address hidden>
Date: Thu Aug 24 11:48:43 2017 +0000

    Updated from global requirements

    Change-Id: I45705bad68f0615dbf09d46a2a487b9fed1abb10

commit c1404f14b88c38e3a6fb0a9dc12946241de2fd80
Author: Akihiro Motoki <email address hidden>
Date: Wed Aug 23 21:15:24 2017 +0000

    Convert remaining network functional tests to JSON

    Change-Id: Ib7dff...

Read more...

tags: added: in-feature-osc4
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-openstackclient 3.13.0

This issue was fixed in the openstack/python-openstackclient 3.13.0 release.

Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Bug closed due to lack of activity, please feel free to reopen if needed.

Changed in neutron:
status: Incomplete → Won't Fix
Changed in python-neutronclient:
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.