controller-tools icon indicating copy to clipboard operation
controller-tools copied to clipboard

webhook generation hardcodes name and namespace

Open ellistarn opened this issue 5 years ago • 10 comments

https://github.com/kubernetes-sigs/controller-tools/blob/9a9db70f732af8f2ed69184a6e849f25d8e5a05e/pkg/webhook/parser.go#L243

I'd like this to be configurable so that I don't have to use Kustomize to add prefixes.

ellistarn avatar Sep 15 '20 17:09 ellistarn

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Dec 14 '20 17:12 fejta-bot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle rotten

fejta-bot avatar Jan 13 '21 18:01 fejta-bot

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-contributor-experience at kubernetes/community. /close

fejta-bot avatar Feb 12 '21 18:02 fejta-bot

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-contributor-experience at kubernetes/community. /close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Feb 12 '21 18:02 k8s-ci-robot

@DirectXMan12 any thoughts here?

ellistarn avatar Feb 12 '21 18:02 ellistarn

I run my webhook in a different namespace than system so also need to customize the namespace.

davidxia avatar Nov 26 '23 12:11 davidxia

Looks like the [Mutating|Validating]WebhookConfigurations' name itself is also hardcoded. I also need to customize this.

I need to customize these because if I deploy two different controllers to the same cluster, I can't use controller-gen's generated files without modification since the [Mutating|Validating]WebhookConfigurations' own names and that of their .webhooks[].clientConfig.name and .webhooks[].clientConfig.name conflict with each other.

davidxia avatar Nov 26 '23 14:11 davidxia

/reopen /lifecycle frozen

sbueringer avatar Apr 10 '24 05:04 sbueringer

@sbueringer: Reopened this issue.

In response to this:

/reopen /lifecycle frozen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Apr 10 '24 05:04 k8s-ci-robot

/remove-lifecycle rotten

sbueringer avatar Apr 10 '24 05:04 sbueringer