Cannot fetch VnfcResourceInfo and VnfcInfo due to K8s resource name

Bug #1990793 reported by Masaki UENO
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tacker
Fix Released
Undecided
Unassigned

Bug Description

VnfcResourceInfo and VnfcInfo will be `None` in particular pattern of K8s resource name through v2 API. The bug will appear if K8s resource name contains `-`.

The code[1] contains `res.name.split("-")[0]`, so this part does not consider `-` in K8s resource names.

[1] https://opendev.org/openstack/tacker/src/branch/master/tacker/sol_refactored/infra_drivers/kubernetes/kubernetes_common.py#L113-L116

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tacker (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tacker/+/859206

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

Reviewed: https://review.opendev.org/c/openstack/tacker/+/859206
Committed: https://opendev.org/openstack/tacker/commit/2e92de26df0af79d400e7aa78b87c8987343e416
Submitter: "Zuul (22348)"
Branch: master

commit 2e92de26df0af79d400e7aa78b87c8987343e416
Author: Ken Fujimoto <email address hidden>
Date: Mon Sep 26 02:13:29 2022 +0000

    Fix k8s resource name handling in CNF v2 API

    Previously if k8s resource name contains "-",
    vnfcResourceInfo and vnfcInfo are not stored through
    CNF v2 API. This patch fixes this problem.

    Closes-Bug: #1990793
    Change-Id: If8ea964c62542df2e3d1e25034e58131480f2c37

Changed in tacker:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tacker (stable/zed)

Fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/tacker/+/859741

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tacker (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/tacker/+/859741
Committed: https://opendev.org/openstack/tacker/commit/1c23cb2911a665f048f4f5fab81b91ca772220fb
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 1c23cb2911a665f048f4f5fab81b91ca772220fb
Author: Ken Fujimoto <email address hidden>
Date: Mon Sep 26 02:13:29 2022 +0000

    Fix k8s resource name handling in CNF v2 API

    Previously if k8s resource name contains "-",
    vnfcResourceInfo and vnfcInfo are not stored through
    CNF v2 API. This patch fixes this problem.

    Closes-Bug: #1990793
    Change-Id: If8ea964c62542df2e3d1e25034e58131480f2c37

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tacker 8.0.0.0rc2

This issue was fixed in the openstack/tacker 8.0.0.0rc2 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tacker 9.0.0.0rc1

This issue was fixed in the openstack/tacker 9.0.0.0rc1 release candidate.

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.