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

[ci] Use DotNetCoreCLI to sign macOS files #9102

Merged
merged 4 commits into from
Jul 15, 2024
Merged

Conversation

pjcollins
Copy link
Member

We've been having issues with the signing steps that run doing the macOS
build. Migration to a new post-build workflow is in progress, however
this will hopefully fix things more quickly and be safer to backport.

We've been having issues with the signing steps that run doing the macOS
build.  Migration to a new post-build workflow is in progress, however
this will hopefully fix things more quickly and be safer to backport.
@pjcollins pjcollins requested a review from jonpryor as a code owner July 11, 2024 20:44
@pjcollins
Copy link
Member Author

pjcollins commented Jul 11, 2024

Test run is here - https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=9869333&view=logs&s=1afc3bfe-122c-538b-e9ad-2a86c2efcfef&j=96fd57f5-f69e-53c7-3d47-f67e6cf9b93e, looks like we've made it further than recent builds on main. Looking to rework this a bit in #9091 as well but this should be safer to backport as needed if it works.

@pjcollins pjcollins marked this pull request as draft July 12, 2024 01:35
@pjcollins
Copy link
Member Author

This doesn't crash, but it's timed out twice now so I guess it doesn't really work either...

@pjcollins
Copy link
Member Author

Looks like we got a successful run with CodeQL explicitly disabled. Not yet sure if signing services are running better this week or the CodeQL changes sped things up, but I've moved those changes into #9111.

I think this should be good to go to at least fix our mono MSBuild crash in signing.

@pjcollins pjcollins marked this pull request as ready for review July 15, 2024 18:43
@pjcollins pjcollins merged commit 6b3b480 into main Jul 15, 2024
55 of 58 checks passed
@pjcollins pjcollins deleted the dev/pjc/sign-dotnet branch July 15, 2024 21:58
pjcollins added a commit that referenced this pull request Jul 15, 2024
We've been having issues with the signing steps that run doing the macOS
build.  Migration to a new post-build workflow is in progress, however
this will hopefully fix things more quickly and be safer to backport.
grendello added a commit that referenced this pull request Jul 23, 2024
* main: (23 commits)
  Localized file check-in by OneLocBuild Task (#9129)
  [ci] Disable CodeQL on CI/PR pipelines (#9128)
  Refine 16k page alignment support (#9075)
  [build] fix `ConfigureLocalWorkload` target (#9124)
  Bump to NDK r27 (#9020)
  [ci] Use drop service for SDK insertion artifacts  (#9116)
  Fix up all mapping paths (#9121)
  [ci] Fix maestro publishing for stable packages (#9118)
  Bump to dotnet/sdk@2f14fea98b 9.0.100-preview.7.24367.21 (#9108)
  Missing androidx.window.[extensions|sidecar] warnings (#9085)
  [ci] Use sign-artifacts template for macOS signing (#9091)
  [ci] Use DotNetCoreCLI to sign macOS files (#9102)
  [ci] Disable CodeQL on macOS, Linux, non-main jobs (#9111)
  [tests] re-enable `JavaAbstractMethodTest` (#9097)
  [Microsoft.Android.Sdk.ILLink] preserve types with `IJniNameProviderAttribute` (#9099)
  [Mono.Android] Data sharing and Close() overrides (#9103)
  [AndroidManifest] Add `Android.App.PropertyAttribute` (#9016)
  [Mono.Android] Add support for AndroidMessageHandler ClientCertificates (#8961)
  [Mono.Android] Bind and enumify API-35 (#9043)
  Bump to dotnet/java-interop@7a058c0e (#9066)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Aug 16, 2024
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