Update app Zuul Check Jobs.

Modify code to conform to flake8 and pylint.

Jobs are now flake8, pylint, py39 and metadata.

Test Plan
PASS - All zuul jobs pass as expected.

Story: 2010929
Task: 49264

Change-Id: I2a9e5d5b9eb7099efaa7700aab0185756e3428c5
Signed-off-by: Reed, Joshua <Joshua.Reed@windriver.com>
This commit is contained in:
Reed, Joshua 2023-12-14 12:30:24 -07:00
parent 81a379ee0d
commit 4b2a17cb17
6 changed files with 44 additions and 7 deletions

2
.gitignore vendored Normal file
View File

@ -0,0 +1,2 @@
.tox
venv/

View File

@ -9,6 +9,7 @@
- k8sapp-power-metrics-tox-flake8
- k8sapp-power-metrics-tox-pylint
- k8sapp-power-metrics-tox-bandit
- k8sapp-power-metrics-tox-metadata
gate:
jobs:
- openstack-tox-linters
@ -16,6 +17,7 @@
- k8sapp-power-metrics-tox-flake8
- k8sapp-power-metrics-tox-pylint
- k8sapp-power-metrics-tox-bandit
- k8sapp-power-metrics-tox-metadata
- job:
name: k8sapp-power-metrics-tox-py39
@ -48,6 +50,25 @@
tox_envlist: flake8
tox_extra_args: -c python3-k8sapp-power-metrics/k8sapp_power_metrics/tox.ini
- job:
name: k8sapp-power-metrics-tox-metadata
parent: tox
description: |
Run metadata test for k8sapp_power_metrics
required-projects:
- starlingx/config
- starlingx/fault
- starlingx/root
- starlingx/update
- starlingx/utilities
nodeset: debian-bullseye
files:
- python3-k8sapp-power-metrics/*
vars:
tox_envlist: metadata
tox_extra_args: -c python3-k8sapp-power-metrics/k8sapp_power_metrics/tox.ini
tox_constraints_file: '{{ ansible_user_dir }}/src/opendev.org/starlingx/root/build-tools/requirements/debian/upper-constraints.txt'
- job:
name: k8sapp-power-metrics-tox-pylint
parent: tox

View File

@ -1,5 +1,5 @@
[tox]
envlist = flake8,py39,pylint,bandit
envlist = flake8,py39,pylint,bandit,metadata
minversion = 2.9
skipsdist = True
@ -11,6 +11,8 @@ distshare={toxworkdir}/.tox/distshare
[testenv]
allowlist_externals = bash
find
echo
basepython = python3.9
sitepackages = False
@ -86,3 +88,11 @@ commands =
deps = pip_missing_reqs
-rrequirements.txt
commands=pip-missing-reqs -d --ignore-file=/k8sapp_power_metrics/tests k8sapp_power_metrics
[testenv:metadata]
install_command = pip install -v -v -v \
-c{env:UPPER_CONSTRAINTS_FILE:https://opendev.org/starlingx/root/raw/branch/master/build-tools/requirements/debian/upper-constraints.txt} \
{opts} {packages}
# Pass top level app folder to 'sysinv-app tox' command.
commands =
bash -c "echo $(dirname $(dirname $(pwd))) | xargs -n 1 sysinv-app tox"

View File

@ -30,9 +30,9 @@ override_dh_auto_build:
cp helm-charts/*.tgz $(STAGING)/charts
# Populate metadata
sed -i 's/@APP_NAME@/$(APP_NAME)/g' $(STAGING)/metadata.yaml
sed -i 's/@APP_VERSION@/$(APP_VERSION)/g' $(STAGING)/metadata.yaml
sed -i 's/@HELM_REPO@/$(HELM_REPO)/g' $(STAGING)/metadata.yaml
sed -i 's/APP_REPLACE_NAME/$(APP_NAME)/g' $(STAGING)/metadata.yaml
sed -i 's/APP_REPLACE_VERSION/$(APP_VERSION)/g' $(STAGING)/metadata.yaml
sed -i 's/HELM_REPLACE_REPO/$(HELM_REPO)/g' $(STAGING)/metadata.yaml
# Copy the plugins: installed in the buildroot
mkdir -p $(STAGING)/plugins

View File

@ -1,8 +1,8 @@
maintain_user_overrides: true
app_name: @APP_NAME@
app_version: @APP_VERSION@
helm_repo: @HELM_REPO@
app_name: APP_REPLACE_NAME
app_version: APP_REPLACE_VERSION
helm_repo: HELM_REPLACE_REPO
upgrades:
auto_update: true

View File

@ -47,6 +47,10 @@ description = Dummy environment to allow flake8 to be run in subdir tox
basepython = python3
description = Dummy environment to allow pylint to be run in subdir tox
[testenv:metadata]
basepython = python3
description = Dummy environment to allow sysinv-app to be run in subdir tox
[testenv:bandit]
basepython = python3
description = Dummy environment to allow bandit to be run in subdir tox