Skip to content

Change signature for diagnostic binaries - #74322

Merged
hoyosjs merged 6 commits into
dotnet:mainfrom
hoyosjs:juhoyosa/dac-signing-rc1
Aug 22, 2022
Merged

Change signature for diagnostic binaries#74322
hoyosjs merged 6 commits into
dotnet:mainfrom
hoyosjs:juhoyosa/dac-signing-rc1

Conversation

@hoyosjs

Copy link
Copy Markdown
Member
  • Add DAC signature infrastructure
  • Try using a .NET 6 SDK to enable signing

Use a signing infrastructure orthogonal to microbuild to use other certs for diagnostic binaries.

cc: @dotnet/dotnet-diag @leculver

@hoyosjs
hoyosjs requested review from a team, jkoritzinsky and mmitcheAugust 22, 2022 02:36
@ghostghost assigned hoyosjsAug 22, 2022
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Add DAC signature infrastructure
  • Try using a .NET 6 SDK to enable signing

Use a signing infrastructure orthogonal to microbuild to use other certs for diagnostic binaries.

cc: @dotnet/dotnet-diag @leculver

Author:hoyosjs
Assignees:-
Labels:

area-Diagnostics-coreclr

Milestone:-

@hoyosjs

hoyosjs commented Aug 22, 2022

Copy link
Copy Markdown
MemberAuthor

@dotnetdotnet deleted a comment from github-actionsBotAug 22, 2022
@dotnetdotnet deleted a comment from github-actionsBotAug 22, 2022
@dotnetdotnet deleted a comment from github-actionsBotAug 22, 2022
@hoyosjs
hoyosjsforce-pushed the juhoyosa/dac-signing-rc1 branch from 975ec67 to ab5bf51CompareAugust 22, 2022 07:18
@hoyosjs

Copy link
Copy Markdown
MemberAuthor

/backport to release/7.0-rc1

@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/7.0-rc1: https://github.com/dotnet/runtime/actions/runs/2901995280

@hoyosjs

Copy link
Copy Markdown
MemberAuthor

Failure is #73688

@noahfalknoahfalk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM
I'm not an expert with these build files, but it all looked reasonable. I assume we'll be manually validating after the build that the binaries got signed appropriately

@hoyosjs
hoyosjs merged commit 8b25e31 into dotnet:mainAug 22, 2022
@hoyosjs
hoyosjs deleted the juhoyosa/dac-signing-rc1 branch August 22, 2022 21:20
@ghostghost locked as resolved and limited conversation to collaborators Sep 22, 2022
Sign up for freeto 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.

2 participants

@hoyosjs@noahfalk