Skip to content

Add smoke test to Docker publish workflow#298

Merged
goneall merged 2 commits into
spdx:masterfrom
bact:ci-docker-smoke-test
Jul 19, 2026
Merged

Add smoke test to Docker publish workflow#298
goneall merged 2 commits into
spdx:masterfrom
bact:ci-docker-smoke-test

Conversation

@bact

@bact bact commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator
  • Add smoke test to check if @@VERSION@@ substitution in tools-java wrapper is success.
  • Removed dead type=raw,value=main tag rule at meta_base step
    • github.ref is never refs/heads/main here since push only triggers on tags (not branches) and the repo's default branch is master, not main (we don't have main).
  • Update out-dated GitHub Actions

Will close #297

 docker_push step needs successful smoke_test

Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
@bact bact added the test Unit test, code coverage, test case label Jul 16, 2026
@bact

bact commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator Author

The workflow failed as expected, error messages from the run log confirms #295 as reported by @kzantow:

Error: Unable to access jarfile /usr/lib/java/spdx/tools-java--jar-with-dependencies.jar

That error should be fixed once #296 (Dockerfile fix) is merged.

--

We should merge #296 first and re-run the CI in this PR again

@goneall

goneall commented Jul 19, 2026

Copy link
Copy Markdown
Member

I merged #296 and reran the CI.

It failed with the message:

Run output=$(docker run --rm tools-java:ci-test 2>&1) || true
Error: Unable to access jarfile /usr/lib/java/spdx/tools-java--jar-with-dependencies.jar
Error: Container failed to find its jar file
Error: Process completed with exit code 1.

@bact

bact commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator Author

@goneall the CI is now passing.

@goneall
goneall merged commit 4857e5b into spdx:master Jul 19, 2026
3 checks passed
@bact
bact deleted the ci-docker-smoke-test branch July 19, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Unit test, code coverage, test case

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Dockerfile smoke test

2 participants