search

redpanda-data / helm-charts

Redpanda Helm Chart
http://redpanda.com
Apache License 2.0
75 stars 96 forks source link

Convert the Redpanda chart into go code #1092

Open chrisseto opened 6 months ago

chrisseto commented 6 months ago 
### Tasks
- [ ] https://github.com/redpanda-data/helm-charts/pull/1063
- [ ] https://github.com/redpanda-data/helm-charts/issues/1111
- [ ] https://github.com/redpanda-data/helm-charts/issues/1081
- [ ] https://github.com/redpanda-data/helm-charts/issues/1222
- [ ] https://github.com/redpanda-data/helm-charts/issues/1213
- [ ] https://github.com/redpanda-data/helm-charts/issues/1202
- [ ] https://github.com/redpanda-data/helm-charts/issues/1224
- [ ] https://github.com/redpanda-data/helm-charts/issues/1223
- [ ] https://github.com/redpanda-data/helm-charts/issues/1225
- [ ] https://github.com/redpanda-data/helm-charts/issues/1226
- [ ] https://github.com/redpanda-data/helm-charts/issues/1227
- [ ] https://github.com/redpanda-data/helm-charts/issues/1228
- [ ] https://github.com/redpanda-data/helm-charts/issues/1229
- [ ] https://github.com/redpanda-data/helm-charts/issues/1220
- [ ] https://github.com/redpanda-data/helm-charts/issues/1219
- [ ] https://github.com/redpanda-data/helm-charts/issues/1218
- [ ] https://github.com/redpanda-data/helm-charts/issues/1217
- [ ] https://github.com/redpanda-data/helm-charts/issues/1216
- [ ] https://github.com/redpanda-data/helm-charts/issues/1215
- [ ] https://github.com/redpanda-data/helm-charts/issues/1214
- [x] https://github.com/redpanda-data/helm-charts/issues/1132
- [ ] https://github.com/redpanda-data/helm-charts/issues/1168
- [ ] https://github.com/redpanda-data/helm-charts/issues/1221
- [ ] redpanda: Convert subcharting to go code

JIRA Link: K8S-118

JIRA Link: K8S-304

chrisseto commented 6 months ago

I've converted a subset of the _helpers.tpl file. We might want to save that one for last?

chrisseto commented 6 months ago

Some other notes:

To the best of our ability, we want to keep inputs (values.yaml) and the outputs stable or at least backwards compatible. @alejandroEsc has pointed out that there are places where this will be impossible due to existing bugs. In that case, make sure to add regression tests.

I also accidentally duplicated the values.go file from the operator. We'll need to consolidate those eventually but we can probably wait until we finish the conversion and move the go definitions into the operator.