Skip to content

[wip] bats suite in GH actions: PoC on custom runner #610

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

[wip] bats suite in GH actions: PoC on custom runner #610

wants to merge 26 commits into from

Conversation

@jgehrcke
Copy link
Collaborator

@jgehrcke jgehrcke commented Sep 22, 2025 •
edited
Loading

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 22, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@jgehrcke jgehrcke force-pushed the jp/dev-gh-runner-tests branch from d3a6993 to 3dbca3e Compare September 22, 2025 21:41
@jgehrcke
Copy link
Collaborator Author

jgehrcke commented Sep 22, 2025

Saw a successful test suite execution (logs). Good night 💤!

@jgehrcke jgehrcke changed the title [wip] run tests in GH actions PoC [wip] bats suite in GH actions: PoC on custom runner Sep 22, 2025
@jgehrcke jgehrcke mentioned this pull request Sep 23, 2025
Merged
@jgehrcke jgehrcke force-pushed the jp/dev-gh-runner-tests branch from c80485b to 6d3dd1c Compare September 23, 2025 14:21
@klueska klueska added this to the unscheduled milestone Sep 30, 2025
@klueska klueska added the ci/testing issue/PR related to CI and/or testing label Sep 30, 2025
@jgehrcke jgehrcke force-pushed the jp/dev-gh-runner-tests branch 3 times, most recently from 1c8bd98 to 2e298a1 Compare October 15, 2025 17:22
@jgehrcke
add workflow scaffold
640d0e1 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

adjust

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

remove tea, add coffee

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
debug
6784c01 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
respect TMPDIR if set
e07fbd6 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
ci: tmpdir in checkout dir
a353e55 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
explicit ./ as TMPDIR
b3f37e9 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
fix CI detection
afa2d9c 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

., not ./

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

try not setting TMPDIR to RUNDIR

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
pretty in ci, too
c76c737 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

explicit TERM for GHA -- pretty?

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

nothing pretty here

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
resolve conflict
28a56ad 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke jgehrcke force-pushed the jp/dev-gh-runner-tests branch from 1914d54 to 28a56ad Compare November 21, 2025 17:58
@jgehrcke
makefile: let us see
f1595c5 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
test gpu operator installed
fae4e70 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
bump tested chart
7fa5d5f 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
container image: use pre-built bash binaries
6a824e9 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
dockerfile: more cleanup
0fad1d8 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
Makefile: image-build-and-copy-to-nodes
8ca121b 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
workflow: use image-build-and-copy-to-nodes, TEST_CHART_LOCAL
903bbc4 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
toolkit version script: remove reliance on go tooling
d669880 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
squash into last
a9d8495 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
workflow: start after artifacts are pushed
7d3c479 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
workflow: build image before basic checks are done
a28761e 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
Copy link
Collaborator Author

jgehrcke commented Nov 21, 2025

/ok to test a28761e

@jgehrcke
workflows: do not install go for building image
2a02e0c 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
workflows: run ci on push
939e2f0 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
squash into last
63a91f3 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
build image on GHA runners
e916296 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
build image on GHA runners (2)
38ef5ff 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
workflows: dt not rely on GH VERSION var
70469b4 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
@jgehrcke
squash into last
b40ee10 
Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jgehrcke jgehrcke

Labels

ci/testing issue/PR related to CI and/or testing

Projects

Planning Board: k8s-dra-driver-gpu
Status: Backlog

Milestone

unscheduled

Development

Successfully merging this pull request may close these issues.

2 participants

@jgehrcke @klueska