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

Re-add test runtime hints #13511

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

jeanbisutti
Copy link
Member

@jeanbisutti jeanbisutti commented Mar 13, 2025

Fix #13508

The native tests have worked in #13501 but the native daily build has failed.

@jeanbisutti jeanbisutti requested a review from a team as a code owner March 13, 2025 14:47
@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Mar 13, 2025
@jeanbisutti jeanbisutti force-pushed the re-add-test-runtime-hints branch from 0d1d732 to 275a9e4 Compare March 13, 2025 14:54
@trask
Copy link
Member

trask commented Mar 13, 2025

The native tests have worked in #13501 but the native daily build has failed.

what's the difference between the two? thanks

@trask trask added this to the v2.14.0 milestone Mar 13, 2025
@jeanbisutti
Copy link
Member Author

The native tests have worked in #13501 but the native daily build has failed.

what's the difference between the two? thanks

Also the GraalVM version but I don't think it's related to the issue.

@trask
Copy link
Member

trask commented Mar 13, 2025

thanks, can we add that to the PR build now also?

@jeanbisutti
Copy link
Member Author

thanks, can we add that to the PR build now also?

@zeitlinger What do you think?

@zeitlinger
Copy link
Member

zeitlinger commented Mar 13, 2025

thanks, can we add that to the PR build now also?

testing all graal versions in the PR seems excessive (if that is what you're suggesting)

@trask trask merged commit 76dc8aa into open-telemetry:main Mar 13, 2025
87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test native This label can be applied to PRs to trigger them to run native tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow failed: Daily GraalVM native tests (#670)
3 participants