triggermesh / docs

Documentation for https://github.com/triggermesh/triggermesh
https://docs.triggermesh.io
Apache License 2.0
6 stars 11 forks source link

Update observability-metrics.md #390

Closed jmcx closed 1 year ago

jmcx commented 1 year ago

Added a warning for users installing Prometheus using the Operator.

image

tzununbekov commented 1 year ago

It is very subjective, but for me, the issue we're solving with this update is rather on the "special case" side, so maybe we could make this warning less intrusive. We can simply mention it in the config snippet with a short explanation, like this:


Screenshot 2023-06-15 at 7 59 45 PM

Explanation wording can be changed if you have something better in mind. Or we can uncomment those two lines, it shouldn't break anything. What do you think?

jmcx commented 1 year ago

Sure! I like it. I'll make that change and add the function examples too.

jmcx commented 1 year ago

Maybe we can merge this PR and do the function stuff in another one.