Kuryr tries to query the existing subnet with the given CIDR

Bug #1496727 reported by Taku Fukushima
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr
Fix Committed
Undecided
Taku Fukushima

Bug Description

Kuryr creates the subnets with their subnet CIDRs, e.g., 10.0.1.0/24, rather than the given CIDRs as it is, e.g., 10.0.1.1/24. However, current Kuryr queries with the given CIDRs and that leads multiple subnets are created with the same CIDR.

The following logic should be put outside of the block:

https://github.com/openstack/kuryr/blob/master/kuryr/controllers.py#L81-L83

Tags: ipam
Changed in kuryr:
assignee: nobody → Taku Fukushima (tfukushima)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kuryr (master)

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

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

Reviewed: https://review.openstack.org/224479
Committed: https://git.openstack.org/cgit/openstack/kuryr/commit/?id=582607236fe2e5a0a1c7c649d1758434d53c0bf9
Submitter: Jenkins
Branch: master

commit 582607236fe2e5a0a1c7c649d1758434d53c0bf9
Author: Taku Fukushima <email address hidden>
Date: Thu Sep 17 17:19:16 2015 +0900

    Fix subnet querying to use the subnet CIDR

    This patch fixes Kuryr not to use the given CIDRs as they are but the
    subnet CIDR. Before applying this patch to Kuryr queried the existing
    subnets with the CIDRs for interfaces in the requests and that led
    duplicated subnets created.

    Change-Id: I8fc4b32ce8a4e016554d8a76e8063e916a69c370
    Signed-off-by: Taku Fukushima <email address hidden>
    Closes-Bug: #1496727

Changed in kuryr:
status: In Progress → Fix Committed
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.