lldp_neighbors API failing with itemNotFound

Bug #2054707 reported by Lucas Ratusznei Fonseca
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Lucas Ratusznei Fonseca

Bug Description

Brief Description
-----------------
Get /v1/lldp_neighbors/​{lldp_neighbor_id} through restAPI failed with itemNotFound.

Severity
--------
Standard

Steps to Reproduce
------------------
Call API via curl:
curl -g -i -X GET http://controller:6385/v1/lldp_neighbors/​{lldp_neighbor_id} -H "Accept: application/json" -H "Content-Type: application/json" -H "User-Agent: cgtsclient" -H "X-Auth-Token: {token}"

Expected Behavior
------------------
HTTP/1.1 200 OK

Actual Behavior
----------------
HTTP/1.1 404 Not Found

Reproducibility
---------------
100% reproducible

System Configuration
--------------------
AIO-SX

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

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/metal/+/909922

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

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/909924

Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/c/starlingx/config/+/909924
Committed: https://opendev.org/starlingx/config/commit/804677c0d7f501cefb78852a77ec64bbd198f841
Submitter: "Zuul (22348)"
Branch: master

commit 804677c0d7f501cefb78852a77ec64bbd198f841
Author: Lucas Ratusznei Fonseca <email address hidden>
Date: Thu Feb 22 16:05:46 2024 -0300

    Fix links to LLDP neighbors in API responses for host and port

    Responses from calls to host and port APIs return links to LLDP
    neighbors with incorrect paths. The correct path is
    /lldp_neighbours, but the links point to /lldp_neighbors. This commit
    fixes the paths.

    Test plan

    System: AIO-SX

    [PASS] Call /v1/ihosts/​{host_id}​ API, check that links are correct
    [PASS] Call /v1/ports/​{port_id} API, check that links are correct

    Partial-Bug: #2054707
    Change-Id: Ib70fd5f325b0e6d493e507869977ad34e6ae7a3d
    Signed-off-by: Lucas Ratusznei Fonseca <email address hidden>

Ghada Khalil (gkhalil)
tags: added: stx.9.0 stx.networking
Changed in starlingx:
assignee: nobody → Lucas Ratusznei Fonseca (lratuszn)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to metal (master)

Reviewed: https://review.opendev.org/c/starlingx/metal/+/909922
Committed: https://opendev.org/starlingx/metal/commit/33a4be37bcceac912dc2e4d9f3621b189a20eb0b
Submitter: "Zuul (22348)"
Branch: master

commit 33a4be37bcceac912dc2e4d9f3621b189a20eb0b
Author: Lucas Ratusznei Fonseca <email address hidden>
Date: Thu Feb 22 15:37:45 2024 -0300

    Fix reference to LLDP neighbors API path in the documentation

    This commit fixes the documentation so that the API path to get LLDP
    neighbors reflect the spelling used in the code. The path in the code
    is lldp_neighbours, and in the documentation was lldp_neighbors.

    Test plan

    [PASS] API paths in documentation are shown as */lldp_neighbours/*
           instead of */lldp_neighbors/*

    Partial-Bug: #2054707
    Change-Id: If1b1dc205b398cf050585c995bbc2ce985b29c40
    Signed-off-by: Lucas Ratusznei Fonseca <email address hidden>

Ghada Khalil (gkhalil)
Changed in starlingx:
status: In Progress → Fix Released
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.