Skip to content

Set up a8c-secrets identities - #25885

Open
mokagio wants to merge 1 commit into
trunkfrom
mokagio/a8c-secrets-identities
Open

Set up a8c-secrets identities#25885
mokagio wants to merge 1 commit into
trunkfrom
mokagio/a8c-secrets-identities

Conversation

@mokagio

@mokagio mokagio commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Description

Groundwork extracted from #25772 (AINFRA-1538) so the a8c-secrets migration lands as small, independently reviewable pieces.

Adds the repository's a8c-secrets identity: the repo id and the public keys authorised to encrypt for it. Nothing reads these files yet — no encrypted blob is committed, no build phase or CI job consults them — so this is inert until the store swap lands.

Testing instructions

Nothing to run. The review is a check that the two values are right:

Copilot AI lite review requested due to automatic review settings August 10, 2026 03:20
@mokagio mokagio added this to the 27.2 milestone Aug 10, 2026
@mokagio mokagio self-assigned this Aug 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Introduces the repository identity scaffolding for the upcoming a8c-secrets migration by adding the repo identity string and the set of Age recipients allowed to encrypt secrets for this repository (no code paths consume these files yet).

Changes:

  • Adds .a8c-secrets/repo-id to define the repository’s a8c-secrets identity.
  • Adds .a8c-secrets/keys.pub with Age recipients (dev + CI) authorized for this identity.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.a8c-secrets/repo-id Adds the repo identity string used to scope a8c-secrets secrets for this repository.
.a8c-secrets/keys.pub Adds the Age recipient public keys to encrypt secrets for the repo identity.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .a8c-secrets/repo-id
Comment thread .a8c-secrets/keys.pub
# dev
age1srcq3hl92ym9jk3ezj5prwhche3w8szc0cssy8t7afrkjmtkxu2qkjsdfn
# ci
age1a7xcr6qzwnzgcxq95sq33p58xdzsmful8w7mp2zktvuy5434yuss9rmv8f

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

See also #25772 (comment)

Image

@mokagio mokagio added Tooling Build, Release, and Validation Tools and removed [Type] Tooling labels Aug 10, 2026
@mokagio
mokagio requested a review from a team August 10, 2026 03:32
@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number33678
VersionPR #25885
Bundle IDorg.wordpress.alpha
Commit5fbb0a6
Installation URL3huoe319pbphg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number33678
VersionPR #25885
Bundle IDcom.jetpack.alpha
Commit5fbb0a6
Installation URL17t956kjkjopo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@mokagio
mokagio enabled auto-merge August 10, 2026 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tooling Build, Release, and Validation Tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants