Textual definition of "subnet" is not correct

Bug #1168159 reported by Lothar.Reith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Maruti Kamat

Bug Description

The introduction section of the Quantum API states: "The v2.0 API does this by introducing a new entity, called a Subnet. Subnets can represent either a IPv4 or IPv6 address block, ..."

This text should be corrected in the following way:
The introduction section of the Quantum API states: "The v2.0 API does this by introducing a new entity, called a Subnet. Subnets can represent the binding of either a IPv4 or IPv6 address block to a previously created Quantum Network, ..."

Proof Point: The High-Level flow is flawed, as it states that a subnet gets associated with a network, however, there is no API "associate Subnet", only an API "create Subnet, which associates (binds) a CIDR to a network. Therefore, the nature of the object "subnet" is not correctly reflected in the definitions. While this may sound like a minor negligence in the introductory text, this could lead to high cost as this object is so central in the data model. Some implementers may be mislead to believe, that a subnet can indeed exists without binding to a previsouly created network, i.e. that an address range already constitutes a subnet.

Here is the current text describing the high level flow

• Tenant creates a network (e.g., "net1")
• Tenant associates a subnet with that network (e.g., "10.0.0.0/24")

Here is a proposal to correct this text and bring it inline with the API "create subnet":

• Tenant creates a network (e.g., "net1")
• Tenant creates a subnet (e.g., "10.0.0.0/24"), which implicitly associates the address space with that network. This association creates a binding of the adress space to the previously created network. While it is common to say, that a subnet is represented by an IPv4 or IPv6 address range, the adress range alone is not a subnet. Rather it is the binding of the address space to the network, which creates the subnet.

Tags: api
Revision history for this message
Mark McClain (markmcclain) wrote :

This text is in the and not a manual.

tags: added: api
Changed in quantum:
importance: Undecided → Low
status: New → Triaged
assignee: nobody → Salvatore Orlando (salvatore-orlando)
Changed in neutron:
assignee: Salvatore Orlando (salvatore-orlando) → nobody
Revision history for this message
Talusani Mani Shanker (shanker-mani0) wrote :

I would like to work on this bug. Could anyone help me how to fix this bug. This is my first attempt at Bug fixing.

Changed in neutron:
assignee: nobody → Talusani Mani Shanker (shanker-mani0)
Changed in neutron:
assignee: Talusani Mani Shanker (shanker-mani0) → nobody
Maruti Kamat (marutik)
Changed in neutron:
assignee: nobody → Maruti Kamat (marutik)
Revision history for this message
Maruti Kamat (marutik) wrote :

I have fixed the bug at https://wiki.openstack.org/wiki/Neutron/APIv2-specification#High-level_flow

Mark/Salvatore, please change the status of this defect accordingly.

Changed in neutron:
status: Triaged → In Progress
Changed in neutron:
milestone: none → juno-rc1
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-rc1 → 2014.2
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.