docs(build): clarify UBI9 compiler requirement and remove redundant test

- Added a comment in build_container.sh to explain why we install python3.11‑devel, gcc, and make on UBI9.
- Removed tests/integration/test_ubi9_toolchain.py and pytest.ini, as the test provided little value
This commit is contained in:
reluctantfuturist 2025-04-17 17:02:36 -07:00
parent 0de530e99f
commit d4ec593a35
3 changed files with 3 additions and 32 deletions

View file

@ -72,6 +72,9 @@ if [[ $container_base == *"registry.access.redhat.com/ubi9"* ]]; then
FROM $container_base FROM $container_base
WORKDIR /app WORKDIR /app
# We install the Python 3.11 dev headers and build tools so that any
# Cextension wheels (e.g. polyleven, faisscpu) can compile successfully.
RUN dnf -y update && dnf install -y iputils net-tools wget \ RUN dnf -y update && dnf install -y iputils net-tools wget \
vim-minimal python3.11 python3.11-pip python3.11-wheel \ vim-minimal python3.11 python3.11-pip python3.11-wheel \
python3.11-setuptools python3.11-devel gcc make && \ python3.11-setuptools python3.11-devel gcc make && \

View file

@ -1,29 +0,0 @@
# Copyright (c) Meta Platforms, Inc. and affiliates.
# All rights reserved.
#
# This source code is licensed under the terms described in the LICENSE file in
# the root directory of this source tree.
# tests/integration/test_ubi9_toolchain.py
from pathlib import Path
def test_ubi9_compiler_packages_present():
"""
Verify that the UBI9 dnf install line in build_container.sh includes
python3.11setuptools, python3.11devel, gcc, and make.
"""
script = (
Path(__file__).parents[2] # moves from tests/integration up to repo root
/ "llama_stack"
/ "distribution"
/ "build_container.sh"
)
content = script.read_text(encoding="utf-8")
expected = "python3.11-setuptools python3.11-devel gcc make"
assert expected in content, (
f"Expected to find '{expected}' in the UBI9 install line, but it was missing.\n\n"
f"Content of {script}:\n{content}"
)

View file

@ -1,3 +0,0 @@
[pytest]
markers =
integration: slow / dockerdependent tests