Skip to content

chore: improve basic reconciliation for KongPluginInstallation#560

Merged
programmer04 merged 1 commit intomainfrom
kpi-adjustments
Sep 9, 2024
Merged

chore: improve basic reconciliation for KongPluginInstallation#560
programmer04 merged 1 commit intomainfrom
kpi-adjustments

Conversation

@programmer04
Copy link
Member

@programmer04 programmer04 commented Sep 8, 2024

What this PR does / why we need it:

Follow-up for PRs #424 & #474, prerequisite for

it contains a handful of improvements

  • make Makefile aligned with skaffold configuration
  • labels created ConfigMaps with managed by specified to KongPluginInstallation (the same as other resources do)
  • introduce mapping with dedicated annotation to easily discover which KongPluginInstallation a particular ConfigMap responds (label can't be used for this due to length limitation)
  • setting a transitive state of Pending when it's not yet known if the plugin will be fetched successfully or not

Which issue this PR fixes

Part of #379

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

@programmer04 programmer04 added the enhancement New feature or request label Sep 8, 2024
@programmer04 programmer04 added this to the KGO v1.4.x milestone Sep 8, 2024
@programmer04 programmer04 self-assigned this Sep 8, 2024
@programmer04 programmer04 requested a review from a team as a code owner September 8, 2024 17:47
@programmer04 programmer04 enabled auto-merge (squash) September 8, 2024 17:49
@programmer04 programmer04 enabled auto-merge (squash) September 8, 2024 17:49
@programmer04 programmer04 merged commit 9d0f2dc into main Sep 9, 2024
@programmer04 programmer04 deleted the kpi-adjustments branch September 9, 2024 07:42
programmer04 pushed a commit that referenced this pull request Nov 12, 2025
…eason (#560)

* add controlPlaneOptionsValid condition type and reason

* Apply suggestions from code review

Co-authored-by: Patryk Małek <patryk.malek@konghq.com>

---------

Co-authored-by: Patryk Małek <patryk.malek@konghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants