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

Add a CreationTimePatchConfiguration CRD #41

Open
QuingKhaos opened this issue Aug 3, 2022 · 0 comments
Open

Add a CreationTimePatchConfiguration CRD #41

QuingKhaos opened this issue Aug 3, 2022 · 0 comments

Comments

@QuingKhaos
Copy link

Instead of manually configuring the creation-time patch webhook, the patch-operator controller could provide a CreationTimePatch CRD and a controller, which watches those CRs and configures the webhook accordingly.

apiVersion: redhatcop.redhat.io/v1alpha1
kind: CreationTimePatchConfiguration
metadata:
  name: example
spec:
  rules:
    - apiVersion: v1
      resources:
        - configmaps
        - secrets
    - apiVersion: binding.operators.coreos.com/v1alpha1
      resources:
        - servicebindings

I would suggest to support multiple CreationTimePatchConfiguration CRs, which are then consolidated into a single webhook configuration, but having a singleton CRD is also okay.

Multiple CRs would support the use-case were I can configure a new CRD created by some operator installed in the same GitOps subproject/subdir which installed the other operator and don't have to modify another place to support that. But this may be an advanced use-case and not sure how easy it is to map multiple CRs to one resource in the end.

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

No branches or pull requests

1 participant