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: make konnect entities spec.konnect.authRef immutable only when entity is already programmed #24

Merged
merged 2 commits into from
Aug 9, 2024

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Aug 8, 2024

What this PR does / why we need it:

This PR makes it possible to fix a broken authRef instead of requiring users to delete the object and recreate it with a fixed authRef.

This change makes the authRef mutable when

  • entity does not have a Programmed condition with status set to True
  • entity does not have a APIAuthValid condition with status set to True

Which issue this PR fixes

Fixes #

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

@pmalek pmalek self-assigned this Aug 8, 2024
@pmalek pmalek requested a review from a team as a code owner August 8, 2024 10:38
@pmalek pmalek force-pushed the make-konnect-authRef-immutable-only-when-programmed branch from 4fbea76 to 9fccfac Compare August 8, 2024 15:56
@pmalek pmalek force-pushed the make-konnect-authRef-immutable-only-when-programmed branch 2 times, most recently from e34d5b0 to 0eb947e Compare August 9, 2024 10:21
Copy link
Contributor

@czeslavo czeslavo left a comment

Choose a reason for hiding this comment

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

As we've already got the test suite added by Mattia, could we cover the added validations in it? I've created an issue to cover all existing CEL rules in this repo: #29.

@pmalek pmalek force-pushed the make-konnect-authRef-immutable-only-when-programmed branch from 0eb947e to c180eb4 Compare August 9, 2024 13:11
@pmalek
Copy link
Member Author

pmalek commented Aug 9, 2024

As we've already got the test suite added by Mattia, could we cover the added validations in it? I've created an issue to cover all existing CEL rules in this repo: #29.

Added those in ab619db

PTAL

@pmalek pmalek force-pushed the make-konnect-authRef-immutable-only-when-programmed branch from c180eb4 to ab619db Compare August 9, 2024 13:17
@pmalek pmalek merged commit fb07a55 into main Aug 9, 2024
6 checks passed
@pmalek pmalek deleted the make-konnect-authRef-immutable-only-when-programmed branch August 9, 2024 14:06
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.

2 participants