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

Add dotnet multi stage image example #52

Merged
merged 4 commits into from
Dec 4, 2023

Conversation

RassK
Copy link
Contributor

@RassK RassK commented Nov 30, 2023

What

Provides an example demonstrating how to setup instrumentation using a heavy build image and a barebone runtime image.

Copy link

github-actions bot commented Nov 30, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@greatestusername-splunk
Copy link
Collaborator

Thanks for your contribution @RassK !
Please respond with comments for CLA and CoC.

I'll review your submission as soon as I can.
Really appreciate you adding this to the repo!

@RassK
Copy link
Contributor Author

RassK commented Nov 30, 2023

I guess currently doing the CLA once is ok. If there will be more PRs, I'll check with the internal process.
Therefore:

I have read the CLA Document and I hereby sign the CLA

@RassK
Copy link
Contributor Author

RassK commented Nov 30, 2023

@github-actions recheck

@@ -0,0 +1,42 @@
#See https://aka.ms/customizecontainer to learn how to customize your debug container and how Visual Studio uses this Dockerfile to build your images for faster debugging.

FROM mcr.microsoft.com/dotnet/aspnet:6.0 AS base
Copy link

Choose a reason for hiding this comment

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

Consider to upgrade to 8.0 (together with csproj and other places).

It will be longer valid example.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@RassK
Copy link
Contributor Author

RassK commented Dec 1, 2023

I have read the CLA Document and I hereby sign the CLA

rdoshi1 added a commit to splunk/cla-agreement that referenced this pull request Dec 1, 2023
@greatestusername-splunk
Copy link
Collaborator

I have read the CLA Document and I hereby sign the CLA

please also respond with a comment to get the CoC check passing:

I have read the Code of Conduct and I hereby accept the Terms

Thanks @RassK and @Kielek !

Copy link
Collaborator

@greatestusername-splunk greatestusername-splunk left a comment

Choose a reason for hiding this comment

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

Left a couple suggestions in the docs but otherwise LGTM.

2023-12-01 15:34:32
```

## Common known issues
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thank you for this detailed readme and common issue fixes!
This is great!

Comment on lines +14 to +15
1. Build the Docker image.
1. Map a random host port (e.g.: 8181) to the container port (8080).
Copy link
Collaborator

Choose a reason for hiding this comment

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

if you wanted to be really hand holdy you could have the docker commands here for folks to copy and paste.
Just a thought.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good idea. Added script, removed hard coded env from the dockerfile just in case.


### instrument.sh is not found

Linux distributions that do not recognize CRLF (Carriage Return Line Feed) file endings may encounter issues when executing the `entrypoint.sh` script. It is necessary to ensure that on such systems, `entrypoint.sh` uses LF (Line Feed) file endings, especially when the file has been modified in a Windows environment.
Copy link
Collaborator

Choose a reason for hiding this comment

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

worth linking to some other resource on the difference between CRLF / LF? maybe https://en.wikipedia.org/wiki/Newline#Issues_with_different_newline_formats

Just a thought.

@RassK
Copy link
Contributor Author

RassK commented Dec 1, 2023

I have read the Code of Conduct and I hereby accept the Terms

Copy link
Collaborator

@greatestusername-splunk greatestusername-splunk left a comment

Choose a reason for hiding this comment

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

lgtm

@greatestusername-splunk greatestusername-splunk merged commit 89e1368 into splunk:main Dec 4, 2023
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2023
@RassK RassK deleted the dotnet-docker branch December 4, 2023 13:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants