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

feat(konnect): add KongCredentialSecretReconciler to reconcile consumer Secrets with basic-auth credentials and create KongCredentialBasicAuth in response #1120

Merged
merged 14 commits into from
Feb 12, 2025

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Feb 4, 2025

What this PR does / why we need it:

This adds KongCredentialSecretReconciler which reconciles Secrets used as KongConsumers' credentials and managed KongCredentialBasicAuth resources in response.

For each KongConsumer credential this reconciler will create a corresponding credential resource.

Which issue this PR fixes

Part of #618
Closes #1122

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect significant changes

Sorry, something went wrong.

@pmalek pmalek self-assigned this Feb 4, 2025
@pmalek pmalek force-pushed the credentials-basic-auth branch 4 times, most recently from ddddee8 to 50a7e26 Compare February 4, 2025 21:39
@pmalek pmalek added this to the KGO v1.5.x milestone Feb 4, 2025
@pmalek pmalek force-pushed the credentials-basic-auth branch 2 times, most recently from 4a3e524 to 72501c6 Compare February 5, 2025 13:17
@pmalek pmalek marked this pull request as ready for review February 5, 2025 13:37
@pmalek pmalek requested a review from a team as a code owner February 5, 2025 13:37
@pmalek pmalek changed the title feat(konnect): add KongCredentialSecretReconciler to reconcile consumer Secrets and create Credential resources in response feat(konnect): add KongCredentialSecretReconciler to reconcile consumer Secrets with basic-auth credentials and create KongCredentialBasicAuth in response Feb 6, 2025
@pmalek pmalek force-pushed the credentials-basic-auth branch from 72501c6 to 380ee46 Compare February 7, 2025 12:26
Copy link
Member

@mlavacca mlavacca left a comment

Choose a reason for hiding this comment

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

In addition to the comments inline, I do have a point about consumer status: I feel we should include some condition about the secrets properly or improperly referenced there. WDYT?

@pmalek pmalek force-pushed the credentials-basic-auth branch from 9eb2180 to b0e50d6 Compare February 11, 2025 11:12
@pmalek pmalek requested a review from randmonkey February 11, 2025 11:22
@pmalek pmalek enabled auto-merge February 12, 2025 09:15
@pmalek pmalek added this pull request to the merge queue Feb 12, 2025
Merged via the queue into main with commit 7fbef3f Feb 12, 2025
23 checks passed
@pmalek pmalek deleted the credentials-basic-auth branch February 12, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Konnect entities: KongConsumer secrets reconciler for basic-auth
3 participants