Skip to content

Commit

Permalink
test(itest): ensure correct latest built image is tested (#1572)
Browse files Browse the repository at this point in the history
* test(itest): never pull Cryostat container from registry, require one present locally

* test(itest): use os/arch-specific image tag for testing

* test(run.sh): include image os/arch in default image to run

* remove bad image version override
  • Loading branch information
andrewazores authored Jun 30, 2023
1 parent c675a2e commit 367f2c4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-build-image.yml
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ jobs:
with:
step: restore
- name: Run integration tests
run: POD_NAME=cryostat-itests CONTAINER_NAME=cryostat-itest ITEST_IMG_VERSION=latest bash repeated-integration-tests.bash
run: POD_NAME=cryostat-itests CONTAINER_NAME=cryostat-itest bash repeated-integration-tests.bash
- name: Print itest logs
if: failure()
run: ls -1dt target/cryostat-itest-*.log | head -n1 | xargs cat
Expand Down
3 changes: 2 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -550,6 +550,7 @@
<executable>${imageBuilder}</executable>
<arguments>
<argument>run</argument>
<argument>--pull=never</argument>
<argument>--pod=${cryostat.itest.podName}</argument>
<argument>--name=${cryostat.itest.containerName}</argument>
<argument>--health-cmd</argument>
Expand Down Expand Up @@ -614,7 +615,7 @@
<argument>GRAFANA_DASHBOARD_URL=http://${cryostat.itest.webHost}:${cryostat.itest.grafana.port}</argument>
<argument>--detach</argument>
<argument>--rm</argument>
<argument>${cryostat.imageStream}:${cryostat.imageVersionLower}</argument>
<argument>${cryostat.imageStream}:${cryostat.imageVersionLower}-${build.os}-${build.arch}</argument>
</arguments>
<skip>${skipITs}</skip>
</configuration>
Expand Down
2 changes: 1 addition & 1 deletion run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ cleanup() {
trap cleanup EXIT

if [ -z "$CRYOSTAT_IMAGE" ]; then
CRYOSTAT_IMAGE="quay.io/cryostat/cryostat:$(${MVN} validate help:evaluate -o -B -q -DforceStdout -Dexpression=cryostat.imageVersionLower)"
CRYOSTAT_IMAGE="quay.io/cryostat/cryostat:$(${MVN} validate help:evaluate -o -B -q -DforceStdout -Dexpression=cryostat.imageVersionLower)-$(getPomProperty build.os)-$(getPomProperty build.arch)"
fi

printf "\n\nRunning %s ...\n\n", "$CRYOSTAT_IMAGE"
Expand Down

0 comments on commit 367f2c4

Please sign in to comment.