Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,23 +3,28 @@
Follow these steps whenever you modify the charts. Keep answers concise and actionable.

## Always do

- Use supported, security-patched chart versions. Verify availability with `helm search repo <name> --versions` before pinning.
- All commits must be signed off (DCO). Use `git commit -s`.
- Update dependencies in `charts/jaeger/Chart.yaml`, then run `helm dependency update charts/jaeger` to vendor them.
- Bump `version` in `charts/jaeger/Chart.yaml` for any functional change (deps, templates, values, docs that affect behavior). Only change `appVersion` when updating the Jaeger image.
- Keep Bitnami `common` unless you also replace all `common.*` helpers in templates.
- Run `helm lint charts/jaeger` after changes; add `helm template ...` if you touched templates or defaults.

## Dependency specifics

- Elasticsearch: use `elastic/elasticsearch`; pin the latest safe patch; confirm with search before setting the version.
- Kafka: use `strimzi/strimzi-kafka-operator`; remember this installs the operator only—Kafka clusters require separate CRs.
- Avoid archived repos; replace incubator deps when possible.

## Docs and values

- Update `charts/jaeger/values.yaml` comments/examples to match new defaults or dependencies.
- Update `charts/jaeger/README.md` for any repo/command changes and note required manual steps (e.g., Strimzi Kafka CRs).
- Keep example hosts consistent with rendered service names (use `fullnameOverride` if you change them).

## Before you finish

- Dependencies vendored under `charts/jaeger/charts/*` are current.
- Chart version bumped appropriately.
- Lint passes; templates render for provisioned modes you changed.
Expand Down
Loading