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

Fix indexing of correlated diagnostic binaries #80652

Merged
merged 1 commit into from
Jan 19, 2023
Merged

Conversation

hoyosjs
Copy link
Member

@hoyosjs hoyosjs commented Jan 14, 2023

Since 66af857 it looks like the runtime is not creating the clr <-> dac correlation index in symbol servers. publishInstallersAndChecksums is no longer needed, so skip passing that to the in-stage publish.

Since 66af857 it looks like the runtime is not creating the clr <-> dac correlation index in symbol servers. `publishInstallersAndChecksums` is no longer needed, so skip passing that to the in-stage publish.
@ghost ghost assigned hoyosjs Jan 14, 2023
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Jan 17, 2023

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Since 66af857 it looks like the runtime is not creating the clr <-> dac correlation index in symbol servers. publishInstallersAndChecksums is no longer needed, so skip passing that to the in-stage publish.

Author: hoyosjs
Assignees: hoyosjs
Labels:

area-Infrastructure

Milestone: -

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

LGTM although I didn't look at what this switch exactly does

@hoyosjs
Copy link
Member Author

hoyosjs commented Jan 19, 2023

Failure is #80666

@hoyosjs hoyosjs merged commit 0daf880 into main Jan 19, 2023
@hoyosjs hoyosjs deleted the juhoyosa/fix-dac-indexing branch January 19, 2023 19:53
mdh1418 pushed a commit to mdh1418/runtime that referenced this pull request Jan 24, 2023
Since dotnet@66af857 it looks like the runtime is not creating the clr <-> dac correlation index in symbol servers. `publishInstallersAndChecksums` is no longer needed, so skip passing that to the in-stage publish.
@ghost ghost locked as resolved and limited conversation to collaborators Feb 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants