From aa40ec3a8805df28b1358136d5b3a092b4868ac7 Mon Sep 17 00:00:00 2001 From: jkraitbewr Date: Fri, 8 Dec 2023 12:34:05 -0500 Subject: [PATCH] Updated base image plus Python to 3.11 The base image has been updated. As part of the base image update the Python version has changed from 3.10 to 3.11. See base image changes here: https://git.onap.org/integration/docker/onap-python/commit/?h=12.0.0&id=b5ccfd6b07b0b14dc738030c7f33ee242f1724b7 Additionally, the pip requirements had to be updated because of issues as noted by the comments. Some config values also needed to be updated because they no longer worked inside of docker-compose. TEST PLAN PASS: Run docker-compose unit testing PASS: Run compliance and quality robot tests, with mock SMO * Deploy oran-o2 application on StarlingX system controller (stx8) * Update/apply override for dev image * Ensure O2 pod is running using correct image Issue-ID: INF-437 Signed-off-by: jkraitbewr Change-Id: I321fe76fcf6f1617ca93bad88d3ea8984e877616 --- Dockerfile | 2 +- Dockerfile.localtest | 4 +++- constraints.txt | 5 +++-- o2common/config/config.py | 15 +++++---------- requirements.txt | 3 ++- 5 files changed, 14 insertions(+), 15 deletions(-) diff --git a/Dockerfile b/Dockerfile index 23f9add..a2628cd 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1,4 +1,4 @@ -FROM nexus3.onap.org:10001/onap/integration-python:10.1.0 +FROM nexus3.onap.org:10001/onap/integration-python:12.0.0 # https://nexus3.onap.org/#browse/search=keyword%3Dintegration-python:d406d405e4cfbf1186265b01088caf9a # https://git.onap.org/integration/docker/onap-python/tree/Dockerfile diff --git a/Dockerfile.localtest b/Dockerfile.localtest index 02ee361..33fcaa8 100644 --- a/Dockerfile.localtest +++ b/Dockerfile.localtest @@ -1,4 +1,4 @@ -FROM python:3.10-slim-buster +FROM python:3.11-slim-buster RUN apt-get update && apt-get install -y git gcc \ vim curl procps ssh @@ -52,4 +52,6 @@ RUN tar -zxvf helm-v3.3.1-linux-amd64.tar.gz; cp linux-amd64/helm /usr/local/bin RUN mkdir -p /etc/kubeconfig/ # COPY temp/kubeconfig/config /etc/kubeconfig/ +RUN mkdir -p /var/log/orano2 + WORKDIR /src diff --git a/constraints.txt b/constraints.txt index a1cc510..349ab5f 100644 --- a/constraints.txt +++ b/constraints.txt @@ -1,4 +1,5 @@ # -e git+https://opendev.org/starlingx/distcloud-client.git@master#egg=distributedcloud-client&subdirectory=distributedcloud-client # -e git+https://opendev.org/starlingx/config.git@master#egg=cgtsclient&subdirectory=sysinv/cgts-client/cgts-client -cryptography==3.3.2 -python-keystoneclient==3.21.0 \ No newline at end of file +# https://github.com/JoinMarket-Org/joinmarket-clientserver/issues/1452 +cryptography>=35 +python-keystoneclient==3.21.0 diff --git a/o2common/config/config.py b/o2common/config/config.py index 2e47414..c93d388 100644 --- a/o2common/config/config.py +++ b/o2common/config/config.py @@ -36,18 +36,15 @@ def get_smo_ca_config_path(): def get_postgres_uri(): - # host = os.environ.get("DB_HOST", "localhost") - # port = 54321 if host == "localhost" else 5432 - host = "localhost" - port = 5432 + host = os.environ.get("DB_HOST", "localhost") + port = int(os.environ.get("DB_PORT", 5432)) password = os.environ.get("DB_PASSWORD", "o2ims123") user, db_name = "o2ims", "o2ims" return f"postgresql://{user}:{password}@{host}:{port}/{db_name}" def get_api_url(): - # host_interal = os.environ.get("API_HOST", "localhost") - host_interal = "localhost" + host_interal = os.environ.get("API_HOST", "localhost") host_external = os.environ.get("API_HOST_EXTERNAL_FLOATING") if config.conf.OCLOUD.API_HOST_EXTERNAL_FLOATING is not None and \ config.conf.OCLOUD.API_HOST_EXTERNAL_FLOATING != '': @@ -87,10 +84,8 @@ def get_o2dms_api_base(): def get_redis_host_and_port(): - # host = os.environ.get("REDIS_HOST", "localhost") - # port = 63791 if host == "localhost" else 6379 - host = "localhost" - port = 6379 + host = os.environ.get("REDIS_HOST", "localhost") + port = int(os.environ.get("REDIS_PORT", 6379)) return dict(host=host, port=port) diff --git a/requirements.txt b/requirements.txt index f68a4f3..3462bae 100644 --- a/requirements.txt +++ b/requirements.txt @@ -1,4 +1,5 @@ -flask +# https://github.com/python-restx/flask-restx/issues/567 +flask<3.0.0 flask-restx sqlalchemy<2.0.0 redis -- 2.16.6