Skip to content

add test for Mac OS and Windows#1934

Merged
trask merged 21 commits intoopen-telemetry:mainfrom
zeitlinger:windows-test
Jun 29, 2025
Merged

add test for Mac OS and Windows#1934
trask merged 21 commits intoopen-telemetry:mainfrom
zeitlinger:windows-test

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

@zeitlinger zeitlinger requested a review from a team June 2, 2025 14:42
@zeitlinger
Copy link
Copy Markdown
Member Author

@trask can you help with the windows error?

@trask
Copy link
Copy Markdown
Member

trask commented Jun 3, 2025

@trask can you help with the windows error?

I think you need to add double quotes below

"-Porg.gradle.java.installations.paths=${{ steps.setup-test-java.outputs.path }}"

import org.junit.jupiter.api.condition.EnabledOnOs;
import org.junit.jupiter.api.io.TempDir;

@EnabledOnOs(LINUX) // Uses async-profiler, which is only supported on Linux
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

async profiler works on mac, see https://github.com/async-profiler/async-profiler

Comment thread .github/workflows/build.yml
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml
@zeitlinger
Copy link
Copy Markdown
Member Author

@trask please check again

@otelbot-java-contrib
Copy link
Copy Markdown
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@trask
Copy link
Copy Markdown
Member

trask commented Jun 28, 2025

@zeitlinger sorry for the delay, I finally realized the difference between this and the link checking is that this is using the GitHub Rest API, which is more heavily throttled for anonymous users, so instead I changed it to avoid the Rest API

@trask trask enabled auto-merge June 28, 2025 23:27
@trask trask added this pull request to the merge queue Jun 29, 2025
Merged via the queue into open-telemetry:main with commit 3c1269e Jun 29, 2025
29 of 30 checks passed
@zeitlinger zeitlinger deleted the windows-test branch June 30, 2025 09:21
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.

5 participants