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

failed to create or get service for metrics #461

Closed
jpkrohling opened this issue Jun 5, 2019 · 2 comments · Fixed by #530
Closed

failed to create or get service for metrics #461

jpkrohling opened this issue Jun 5, 2019 · 2 comments · Fixed by #530

Comments

@jpkrohling
Copy link
Contributor

Currently, the Jaeger Operator fails to start in OpenShift due to the following message:

time="2019-06-05T12:29:54Z" level=fatal msg="failed to create or get service for metrics: services \"jaeger-operator\" is forbidden: cannot set blockOwnerDeletion if an ownerReference refers to a resource you can't set finalizers on: , <nil>"

This log message is recorded with fatal level, making the operator to simply exit upon this failure. Operators created with newer versions of the SDK have this set to info instead. This of course doesn't fix the underlying problem, but I think we should not crash the operator because of this failure.

@lilic
Copy link

lilic commented Jun 19, 2019

@jpkrohling Hello, you need to add the following to your role.yaml file, and it should fix this problem, as we are putting the Deployment of the operator as the owner to the OwnerRef of the Service object on creation:

  resources:
  - deployments/finalizers

Docs on this in operator-sdk. See full role.yaml file here -> https://github.com/operator-framework/operator-sdk-samples/blob/bd30254f3a7e3943fdbae33e9b6992bce1e07755/memcached-operator/deploy/role.yaml

Let me know if this doesn't work for you!

@jpkrohling
Copy link
Contributor Author

Spot on, @lilic! Would you be open to review the PR that reinstates the metrics service?

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

Successfully merging a pull request may close this issue.

2 participants