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

Deploy service monitor automatically on operatorhub/catalog install #125

Open
trent-s opened this issue Sep 4, 2024 · 0 comments
Open

Comments

@trent-s
Copy link
Collaborator

trent-s commented Sep 4, 2024

This should be super easy, but it isn't.

So far, when enabling automatic service monitor deployment, the CI build/test caused remotely by a PR to operatorhub.io tend to require prometheus (which should be OK), but then things usually (but not always) fail. The current (temporary?) solution is to require the user to hand install the service monitor from susql-operator/hack/susql-servicemonitor.yaml when installing from operatorhub. It isn't a huge burden because it seems that the servicemonitor only needs to be installed once, but it is not elegant to require the user to take that extra step....

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