Conversation
pat-s
changed the title
jaeger
pat-s
changed the title
jaegerjaeger
Co-authored-by: Patrick Schratz <patrick.schratz@gmail.com>
Co-authored-by: Patrick Schratz <patrick.schratz@gmail.com>
pat-s
left a comment
pat-s
left a comment
There was a problem hiding this comment.
Please also run just helm-docs (or how it's called) to update the docs :)
|
@pat-s can you double check this? I want to make sure I did this correctly |
pat-s
left a comment
pat-s
left a comment
There was a problem hiding this comment.
Please run just docs to generate docs.
| # ricochet-config.toml unless config.'otel' is already set. | ||
| enabled: false | ||
| # jaeger.image -- Jaeger v2 image to use (avoid latest in production) | ||
| image: jaegertracing/jaeger:2.15.1 |
There was a problem hiding this comment.
Let's check this into renovate so we don't have to take care of it.
| limits: | ||
| memory: 256Mi | ||
| # jaeger.initImage -- Busybox image used by the init container to write the Jaeger config | ||
| initImage: busybox:1.37 |
There was a problem hiding this comment.
Let's check this into renovate so we don't have to take care of it.
|
@pat-s I've updated the resources. Both myself and claude have no idea how to handle renovate. If we'll just update based on every new tag, why not just use |
…ates Extract hardcoded busybox:1.37 from deployment template into values.yaml and add renovate: image comments to enable automated monthly dependency updates via Renovate regex manager.
This question could start a paper-like reply :) One should never use |
pat-s
left a comment
pat-s
left a comment
There was a problem hiding this comment.
@JosiahParry If you're good with my recent changes, please merge.
2 participants
Requires https://github.com/ricochet-rs/ricochet/pull/700 to be merged.
Adds an optional jaeger sidecar container. If enabled, injects
[otel]into ricochet-config.toml