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

chore: Add dotnet SDKv8 support for examples #3493

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

simonswine
Copy link
Contributor

@simonswine simonswine commented Aug 16, 2024

Unfortunately this problem #3288 has come back to me. 🙂

I think I have a solution now based on (dotnet/dotnet-docker#3832): It uses a multi-stage build in order to overcome build problems on dotnet sdk7+ on Rosetta based macs, when emulating amd64.

It will cross compile using the linux/arm64 dotnet sdk for the linux/amd64 target. (which is the only arch we support)

This uses a multi-stage build in order to overcome build problems on
sdk7+ on Rosetta based macs.

It will cross compile using the linxu/arm64 dotnet sdk for the linux/amd64 target.
@simonswine
Copy link
Contributor Author

Wonder if you guys could give it a try @bryanhuhta @korniltsev

@simonswine simonswine marked this pull request as ready for review August 16, 2024 13:06
@simonswine simonswine requested a review from a team as a code owner August 16, 2024 13:06
@@ -22,6 +41,7 @@ ENV PYROSCOPE_PROFILING_ENABLED=1
ENV PYROSCOPE_PROFILING_ALLOCATION_ENABLED=true
ENV PYROSCOPE_PROFILING_CONTENTION_ENABLED=true
ENV PYROSCOPE_PROFILING_EXCEPTION_ENABLED=true
ENV PYROSCOPE_PROFILING_HEAP_ENABLED=true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was something I would like to test, in order for it to work we need SDKv7+


ADD example .
# We hardcode linux-x64 here, as the profiler doesn't support any other platform
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am fairly certain this is the case, but just double checking

@simonswine simonswine merged commit ed5162d into grafana:main Aug 19, 2024
18 checks passed
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