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

Implement Kapitan plugin as ConfigMamangementPlugin sidecar #128

Merged
merged 3 commits into from
Nov 17, 2023

Conversation

simu
Copy link
Member

@simu simu commented Nov 16, 2023

See also https://argo-cd.readthedocs.io/en/stable/operator-manual/config-management-plugins/#sidecar-plugin

With the sidecar plugin architecture, we can use the upstream Kapitan docker image, since we don't need to be able to mount a single kapitan binary into the argocd-repo-server container to implement the plugin.

Follow-up to #90

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@simu simu force-pushed the feat/update-kapitan-plugin-v2.9 branch from 0c2ebee to ad5f8f4 Compare November 16, 2023 13:00
@simu simu requested a review from a team November 16, 2023 16:02
@simu simu force-pushed the feat/update-kapitan-plugin-v2.9 branch from 5a24e31 to fc5b7f1 Compare November 16, 2023 16:03
@simu simu added the change label Nov 16, 2023
@simu simu force-pushed the feat/update-kapitan-plugin-v2.9 branch 2 times, most recently from 6a5a28f to d410b60 Compare November 16, 2023 16:56
@simu simu force-pushed the feat/update-kapitan-plugin-v2.9 branch from d410b60 to b5d5966 Compare November 17, 2023 08:08
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants