Skip to content

Conversation

@charludo
Copy link
Contributor

@charludo charludo commented Nov 18, 2025

Reads GH token from GCP, and uses it to create the secrets file in just populate.

We could just store the entire config in gcp, but this way it's easy to adjust and extend with additional configuration.

@charludo charludo added the no changelog PRs not listed in the release notes label Nov 18, 2025
@charludo charludo force-pushed the ch/just-use-registry-auth branch 4 times, most recently from d75f19f to 406e0cc Compare November 18, 2025 11:10
@charludo charludo marked this pull request as ready for review November 18, 2025 12:00
@charludo charludo requested a review from katexochen as a code owner November 18, 2025 12:00
@katexochen katexochen self-assigned this Nov 18, 2025
@charludo charludo force-pushed the ch/just-use-registry-auth branch from 406e0cc to 24be9f6 Compare November 19, 2025 09:25
@charludo charludo requested a review from katexochen November 19, 2025 09:25
@charludo charludo force-pushed the ch/just-use-registry-auth branch from 24be9f6 to 5b737e7 Compare November 19, 2025 09:29
@charludo charludo force-pushed the ch/just-use-registry-auth branch 2 times, most recently from ea7569f to f4d8100 Compare November 19, 2025 11:26
@charludo charludo force-pushed the ch/just-use-registry-auth branch from f4d8100 to 434771f Compare November 19, 2025 12:44
@charludo charludo requested a review from katexochen November 19, 2025 12:44
Copy link
Member

@katexochen katexochen left a comment

Choose a reason for hiding this comment

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

Code lgtm, but I want to test this when the e2e test is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PRs not listed in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants