Skip to content
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.

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

🌱 Use GITHUB_RUN_ID for the specName of e2e tests. #1424

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

guettli
Copy link
Collaborator

@guettli guettli commented Aug 6, 2024

Then bm machines will get this name, and then it is easier to understand a failing e2e test.

It would be nice to see the ID here:

image

Unfortunately the JOB_ID is not available: actions/runner#324

Then bm machines will get this name, and then it is easier to understand
a failing e2e test.
@guettli guettli changed the title seedling: Use GITHUB_RUN_ID for the specName of e2e tests. 🌱 Use GITHUB_RUN_ID for the specName of e2e tests. Aug 6, 2024
@guettli guettli requested a review from batistein August 6, 2024 14:09
@batistein batistein marked this pull request as ready for review August 9, 2024 14:11
@syself-bot syself-bot bot added the area/test Changes made in the test directory label Aug 9, 2024
@batistein batistein merged commit 0e29498 into main Aug 9, 2024
16 of 17 checks passed
@batistein batistein deleted the tg/rename-e2e-cluster-to-e2e branch August 9, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test Changes made in the test directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants