Skip to content

test: run kaggle-environments tests in jenkins pipeline#1535

Merged
KeijiBranshi merged 1 commit intomainfrom
keijibranshi/test/kaggle-environments
Mar 3, 2026
Merged

test: run kaggle-environments tests in jenkins pipeline#1535
KeijiBranshi merged 1 commit intomainfrom
keijibranshi/test/kaggle-environments

Conversation

@KeijiBranshi
Copy link
Copy Markdown
Member

BUG=b/477947477

A bit of a quick and dirty solution to the ask. It's brittle against things like simple file moves in the kaggle-environments repo, but wasn't too sure how much we really wanted to couple these two repos. Open to alternative implementations.

@KeijiBranshi KeijiBranshi requested a review from rosbo March 3, 2026 20:10
@rosbo
Copy link
Copy Markdown
Contributor

rosbo commented Mar 3, 2026

A bit of a quick and dirty solution to the ask. It's brittle against things like simple file moves in the kaggle-environments repo, but wasn't too sure how much we really wanted to couple these two repos. Open to alternative implementations.

Let's start with this. If it ends up being too brittle, we can switch to refactoring the existing cicd Cloud Build config in the kaggle-environments repo to optionally support running against a specific Docker image rather than building it from scratch.

@KeijiBranshi KeijiBranshi force-pushed the keijibranshi/test/kaggle-environments branch from 2720332 to 3f74a1d Compare March 3, 2026 21:13
@KeijiBranshi KeijiBranshi marked this pull request as ready for review March 3, 2026 21:33
@KeijiBranshi KeijiBranshi merged commit 8d207bb into main Mar 3, 2026
4 checks passed
@KeijiBranshi KeijiBranshi deleted the keijibranshi/test/kaggle-environments branch March 3, 2026 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants