operator-lifecycle-manager icon indicating copy to clipboard operation
operator-lifecycle-manager copied to clipboard

admissionReviewVersion's services are not properly reconciled

Open nunnatsa opened this issue 3 years ago • 1 comments

Bug Report

What did you do? Deployed an operator using a CSV file. The CSV file contains several admissionReviewVersions sections to define webhooks. For each one of these admissionReviewVersions, OLM creates a service. e.g. for this (partial) admissionReviewVersions:

- admissionReviewVersions:
  - v1beta1
  - v1
  containerPort: 4343
  deploymentName: my-operator

A new service named "my-operator-services is created.

When I manually delete this service, it's never re-created.

What did you expect to see? I would expect OLD to reconcile these services, and re-create them if they are deleted.

What did you see instead? Under which circumstances? The service is no created again and there is no way to trigger OLM to create it.

Environment

  • operator-lifecycle-manager version:

4.12.0-ec.4

  • Kubernetes version information:
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.2", GitCommit:"8b5a19147530eaac9476b0ab82980b4088bbc1b2", GitTreeState:"clean", BuildDate:"2021-09-15T21:38:50Z", GoVersion:"go1.16.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.0+3ef6ef3", GitCommit:"ae0ef8bfe60052956dc4152a4aaf903a0e4e3775", GitTreeState:"clean", BuildDate:"2022-09-28T13:13:16Z", GoVersion:"go1.19.1", Compiler:"gc", Platform:"linux/amd64"}

nunnatsa avatar Nov 02 '22 07:11 nunnatsa

Hello! I may need a little more information to confirm this is what's going on, but it looks as though the webhook service you're referring to here is being created as part of the InstallPlan OLM is using to deploy your operator. If that's the case, OLM currently does not support recreating those resources. If possible, would you be able to share the installation manifest you're providing to OLM?

dtfranz avatar Nov 09 '22 22:11 dtfranz