starlingx/fault
Code Issues Proposed changes

Add sysinv dependency to fm-rest-api image

Browse Source 
A recent change added the sysinv module to a
file in the cgtsclient package [1]. That caused the
fm-rest-api to fail when using the stx-fm-rest-api
docker image.

This review adds the sysinv package as a DIS_PACKAGE
in the stx-fm-rest-api dockerfile. It also adds the
dependency to the tox file.

[1] - https://review.opendev.org/c/starlingx/config/+/952590

Test Plan:
PASS - Build fm-rest-api package and stx-fm-rest-api image
PASS - Use new docker image in the STX-O deployment
PASS - STX-O successfully applied
PASS - fm-rest-api pod running normally

Closes-Bug: #2116944

Change-Id: Ib561c6989b8763aabf43c3fdc03c502d9e119014
Signed-off-by: Daniel Caires <DanielMarques.Caires@windriver.com>
This commit is contained in:
Daniel Caires
2025-07-15 15:08:34 -03:00
parent 0d36c58f6a
commit fccaea1914
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
fm-rest-api/debian/stx-fm-rest-api.stable_docker_image
View File

@@ -3,7 +3,7 @@ LABEL=stx-fm-rest-api
# Set PROJECT=infra and PROJECT_REPO=nil because we are not cloning a repo
PROJECT=infra
PROJECT_REPO=nil
DIST_PACKAGES="postgresql fm-common"
DIST_PACKAGES="postgresql fm-common sysinv"
DIST_REPOS="OS"
PIP_PACKAGES="

1
fm-rest-api/fm/tox.ini
View File

@@ -15,6 +15,7 @@ deps = -chttps://opendev.org/starlingx/root/raw/branch/master/build-tools/requir
-r{toxinidir}/test-requirements.txt
-e{[tox]stxdir}/config/tsconfig/tsconfig
-e{[tox]stxdir}/config/sysinv/cgts-client/cgts-client
-e{[tox]stxdir}/config/sysinv/sysinv/sysinv
-e{[tox]stxdir}/fault/fm-api/source
-e{[tox]stxdir}/fault/fm-rest-api/fm