-
Notifications
You must be signed in to change notification settings - Fork 7
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
[DPE-2935] Integrating Prometheus Exporter #145
Conversation
b24661b
to
3f9254e
Compare
0e6819c
to
df7afa0
Compare
3362bd7
to
b90884a
Compare
46e2ca2
to
f05e7a7
Compare
Hi @juditnovak, I have found a way to manage the removal of a given plugin, without neither removing the method What happens now is that we defer the relation-broken event until there is no more relations between opensearch and grafana-agent. I see that as a valid solution for the relation-broken issue. |
@phvalguima Sure, changes from your diff applied as a separate commit |
41ad4d0
to
4f04de2
Compare
* Charm config * OpenSearch version With these options, it is possible to have an external URL passed as configuration parameter to the charm. The plugin class loads that config, resolves the version and returns a custom install_path. Original functionality persists: if the charm does not implement new install_path property, then it returns plugin name.
04928a5
to
df083e9
Compare
be76844
to
2ed9511
Compare
2ed9511
to
16ee479
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.