Skip to content

Commit 9879882

Browse files
authored
Move the collector tests image to its own registry (#2150) (#2181)
We have been pushing our integration tests to a generic qa registry that is meant to have (mostly) static images. This somewhat pollutes the registry and might make it harder for people to find the tags they want, so we are moving this image to a new registry.
1 parent f5cc9f7 commit 9879882

File tree

6 files changed

+10
-10
lines changed

6 files changed

+10
-10
lines changed

.github/workflows/integration-test-containers.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ on:
2323
outputs:
2424
collector-tests-tag:
2525
description: The tag used for the integration test image
26-
value: ${{ jobs.build-test-image.outputs.collector-tests-tag || 'collector-tests-master' }}
26+
value: ${{ jobs.build-test-image.outputs.collector-tests-tag || 'master' }}
2727

2828
jobs:
2929
should-build-test-image:
@@ -63,7 +63,7 @@ jobs:
6363
collector-tests-tag: ${{ steps.tests-tag.outputs.collector-tests-tag }}
6464

6565
env:
66-
DEFAULT_TESTS_TAG: collector-tests-master
66+
DEFAULT_TESTS_TAG: master
6767

6868
steps:
6969
- uses: actions/checkout@v4
@@ -95,7 +95,7 @@ jobs:
9595
if [[ "${{ github.event_name }}" == 'pull_request' || \
9696
"${{ github.ref_type }}" == 'tag' || \
9797
"${{ github.ref_name }}" =~ ^release- ]]; then
98-
COLLECTOR_TESTS_TAG="collector-tests-${{ inputs.collector-tag }}"
98+
COLLECTOR_TESTS_TAG="${{ inputs.collector-tag }}"
9999
fi
100100
101101
echo "COLLECTOR_TESTS_TAG=${COLLECTOR_TESTS_TAG}" >> "$GITHUB_ENV"
@@ -130,7 +130,7 @@ jobs:
130130
# build_multi_arch passed in as json to ensure boolean type
131131
ansible-playbook \
132132
--connection local -i localhost, --limit localhost \
133-
-e test_image="quay.io/rhacs-eng/qa-multi-arch:${COLLECTOR_TESTS_TAG}" \
133+
-e test_image="quay.io/rhacs-eng/collector-tests:${COLLECTOR_TESTS_TAG}" \
134134
-e "{\"build_multi_arch\": $BUILD_MULTI_ARCH}" \
135135
-e @'${{ github.workspace }}/ansible/secrets.yml' \
136136
ansible/ci-build-tests.yml

.github/workflows/integration-tests-vm-type.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
env:
4848
COLLECTOR_IMAGE: ${{ inputs.collector-repo }}:${{ inputs.collector-tag }}
4949
COLLECTOR_QA_TAG: ${{ inputs.collector-qa-tag }}
50-
TEST_IMAGE: quay.io/rhacs-eng/qa-multi-arch:${{ inputs.collector-tests-tag }}
50+
TEST_IMAGE: quay.io/rhacs-eng/collector-tests:${{ inputs.collector-tests-tag }}
5151
steps:
5252
- uses: actions/checkout@v4
5353

.github/workflows/k8s-integration-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ on:
2626

2727
env:
2828
ANSIBLE_CONFIG: ${{ github.workspace }}/ansible/ansible.cfg
29-
COLLECTOR_TESTS_IMAGE: quay.io/rhacs-eng/qa-multi-arch:${{ inputs.collector-tests-tag }}
29+
COLLECTOR_TESTS_IMAGE: quay.io/rhacs-eng/collector-tests:${{ inputs.collector-tests-tag }}
3030
COLLECTOR_IMAGE: ${{ inputs.collector-repo }}:${{ inputs.collector-tag }}
3131

3232
jobs:

ansible/roles/run-test-target/tasks/test-cri.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
ansible.builtin.set_fact:
3434
logs_root: "{{ collector_root }}/integration-tests/container-logs"
3535
collector_image: "{{ lookup('env', 'COLLECTOR_IMAGE', default='quay.io/rhacs-eng/collector:' + collector_tag_result.stdout) }}"
36-
integration_tests_image: "{{ lookup('env', 'TEST_IMAGE', default='quay.io/rhacs-eng/qa-multi-arch:collector-tests-' + collector_tag_result.stdout) }}"
36+
integration_tests_image: "{{ lookup('env', 'TEST_IMAGE', default='quay.io/rhacs-eng/collector-tests:' + collector_tag_result.stdout) }}"
3737

3838
- name: Default args
3939
ansible.builtin.set_fact:

ansible/roles/run-test-target/tasks/test-docker.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
- set_fact:
2222
collector_image: "{{ lookup('env', 'COLLECTOR_IMAGE', default='quay.io/rhacs-eng/collector:' + collector_tag_result.stdout) }}"
23-
integration_tests_image: "{{ lookup('env', 'TEST_IMAGE', default='quay.io/rhacs-eng/qa-multi-arch:collector-tests-' + collector_tag_result.stdout) }}"
23+
integration_tests_image: "{{ lookup('env', 'TEST_IMAGE', default='quay.io/rhacs-eng/collector-tests:' + collector_tag_result.stdout) }}"
2424

2525
- set_fact:
2626
run_args: -test.run {{ collector_test }} -test.timeout 60m -test.count=1

integration-tests/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@ COLLECTOR_QA_TAG=$(shell cat container/QA_TAG | tr -d '\n')
1414
endif
1515

1616
ifeq ($(COLLECTOR_TESTS_REPO),)
17-
COLLECTOR_TESTS_REPO=quay.io/rhacs-eng/qa-multi-arch
17+
COLLECTOR_TESTS_REPO=quay.io/rhacs-eng/collector-tests
1818
endif
1919

2020
ifeq ($(COLLECTOR_TESTS_IMAGE),)
21-
COLLECTOR_TESTS_IMAGE=$(COLLECTOR_TESTS_REPO):collector-tests-$(COLLECTOR_TAG)
21+
COLLECTOR_TESTS_IMAGE=$(COLLECTOR_TESTS_REPO):$(COLLECTOR_TAG)
2222
endif
2323

2424
SHELL=/bin/bash

0 commit comments

Comments
 (0)