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

[.NET Core Debugging] Add support for Alpine images #771

Merged
merged 5 commits into from
Feb 14, 2019

Conversation

philliphoff
Copy link
Member

@philliphoff philliphoff commented Jan 31, 2019

Adds support for debugging Linux images based on the "Alpine" set of .NET Core base images.

Alpine images require a different variant of the .NET Core debugger (vsdbg) than used for the "standard" images currently supported. Determining which variant is necessary must be done after the container is running, by executing a script on the container which returns the image type.

Currently, the debugging provider volume mounts only the folder of a specific variant of debugger, and has the debugging engine use that variant. This change updates the debugging provider such that containers volume mount the root folder for all cached debugging variants (as that must happen during container creation), then starts the container, then runs the script to determine the needed variant, and then has the debugging engine use that variant. Because the volume mount is the root of the debugger cache, the container can use any of the cached variants as needed.

@philliphoff philliphoff requested a review from a team as a code owner January 31, 2019 01:06
@philliphoff
Copy link
Member Author

Resolves #765.

Copy link
Contributor

@StephenWeatherford StephenWeatherford left a comment

Choose a reason for hiding this comment

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

Have you added a test case for CTI?

@philliphoff
Copy link
Member Author

@StephenWeatherford Thanks for the reminder; I've added a variant of the main scenario but with Alpine-based images.

@StephenWeatherford
Copy link
Contributor

Thx.

@philliphoff philliphoff merged commit eed20a4 into master Feb 14, 2019
@philliphoff philliphoff deleted the philliphoff-alpine branch February 14, 2019 19:20
gdziadkiewicz added a commit to gdziadkiewicz/vscode-docker that referenced this pull request Feb 16, 2019
@microsoft microsoft locked and limited conversation to collaborators Oct 27, 2021
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.

2 participants