Skip to content

Implement core pkg/runtime/secrets migration #1846

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

Closed

Conversation

cappyzawa
Copy link
Contributor

@cappyzawa cappyzawa commented Jul 3, 2025

Summary

Implements core migration to pkg/runtime/secrets v0.63.0 for source-controller as part of fluxcd/flux2#5433.
Migrates basic auth, TLS config, and OCI registry authentication while removing internal wrapper functions.

  • Update pkg/runtime from v0.60.0 to v0.63.0
  • Migrate Bucket and OCIRepository controllers to use pkg/runtime/secrets
  • Remove internal/helm/getter wrapper functions
  • Add OCI registry authentication using pkg/runtime/secrets

Follow-up Work

Remaining controllers (GitRepository, HelmRepository) and cloud provider authentication will be migrated in subsequent PRs.

@cappyzawa cappyzawa force-pushed the feat-runtime-secrets-migration branch from 3cd259e to e742d5a Compare July 4, 2025 04:31
@cappyzawa cappyzawa marked this pull request as draft July 4, 2025 04:39
@cappyzawa cappyzawa force-pushed the feat-runtime-secrets-migration branch 7 times, most recently from 4e903f7 to a1bab65 Compare July 4, 2025 23:55
@cappyzawa cappyzawa marked this pull request as ready for review July 5, 2025 00:06
@cappyzawa
Copy link
Contributor Author

cappyzawa commented Jul 10, 2025

Similar to fluxcd/notification-controller#1139, this PR has multiple secret retrieval patterns that cause unnecessary K8s API calls.

@cappyzawa cappyzawa marked this pull request as draft July 10, 2025 11:54
This is a foundational migration that establishes runtime/secrets usage
patterns for the source-controller.

Signed-off-by: cappyzawa <[email protected]>
@cappyzawa cappyzawa force-pushed the feat-runtime-secrets-migration branch from a1bab65 to b80909a Compare July 10, 2025 12:41
@cappyzawa
Copy link
Contributor Author

Closing since we're handling each repo separately like #1849.

@cappyzawa cappyzawa closed this Jul 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant