Exception recovering ports in use without K8S_ANNOTATION_VIF

Bug #1721194 reported by Luis Tomas Bolivar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr-kubernetes
Fix Released
Undecided
Luis Tomas Bolivar

Bug Description

When the port_debug is disabled and the ports belonging to the pools need to be recovered, we make use of the kubernetes API to discover what ports are being used by the running pods and obtain its IDs based on the K8S_ANNOTATION_VIF information. However, it may happen that some pods do not have the K8S_ANNOTATION_VIF as they are not associated to a neutron port (for instance, the router pod in OpenShift) as they are associated to the host instead.

Changed in kuryr-kubernetes:
assignee: nobody → Luis Tomas Bolivar (ltomasbo)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kuryr-kubernetes (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kuryr-kubernetes (master)

Reviewed: https://review.openstack.org/509364
Committed: https://git.openstack.org/cgit/openstack/kuryr-kubernetes/commit/?id=a9cb42897aee292753bb618cfe4344cda9bcd56d
Submitter: Jenkins
Branch: master

commit a9cb42897aee292753bb618cfe4344cda9bcd56d
Author: Luis Tomas Bolivar <email address hidden>
Date: Wed Oct 4 09:26:05 2017 +0200

    Add exception handling to _get_in_use_ports

    This patch skips the pods without kuryr annotations when
    recovering the IDs of the ports that are being used by
    running pods.

    Moreover, as part of the unit test additions, it also moves
    some tests to the BaseVIFPOOL as they are independent of the
    Pool driver

    Closes-Bug: 1721194
    Change-Id: I70828c1459b4100975bf20d6193db69303451cae

Changed in kuryr-kubernetes:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kuryr-kubernetes 0.3.0

This issue was fixed in the openstack/kuryr-kubernetes 0.3.0 release.

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.