chart-redpanda: cut release v5.9.19 #393
Conversation
|
This will need to be synced back to the helm-charts repository to officially be released. |
chrisseto
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
from
ed12c75 to
0760379
Compare
RafalKorepta
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
2 times, most recently
from
bae1edb to
28e5e3d
Compare
|
I found the problem in unit test. The unreleased version 5.9.19 is not available in official helm repository index ( |
chrisseto
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
2 times, most recently
from
82ac982 to
2abe951
Compare
RafalKorepta
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
6 times, most recently
from
e0cd2c0 to
2aff951
Compare
chrisseto
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
from
2aff951 to
35fdf22
Compare
This commit cuts release v5.9.19 of the redpanda chart, fixes `TestChartYAMLVersions` to work with changie and the new repo structure, and disables operator tests that depend on the redpanda chart version matching the most recently published redpanda chart as there is currently a manual step required to finalize the release.
chrisseto
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
5 times, most recently
from
0c8a7b9 to
4a4c15f
Compare
chrisseto
force-pushed
the
chris/p/chart-redpanda-5.9.19
branch
from
4a4c15f to
0eaf6ca
Compare
This commit pins the charts module version to the commit closest to v5.9.18 of the redpanda helm chart. There is no exact commit to pin to due to the monorepo migration. Additional this commit includes a small fix to the redpanda controller that prevents the status updated from failing due to unexpected usage of optimistic locking.
| - task: :test:integration | ||
| vars: | ||
| GO_TEST_RUNNER: gotestsum | ||
| CLI_ARGS: --junitfile=integration-tests.xml -- {{.CLI_ARGS}} | ||
| GO_TEST_RUNNER: gotestsum --raw-command --junitfile=integration-tests-%.xml -- go test |
There was a problem hiding this comment.
NIT: the % character might be hard to understand when I would look at this line in month time. A comment, that explain xargs usage in test:integration could help (at least me).
There was a problem hiding this comment.
Good call out. I can push up some notes. It's very hacky 🙈
| licenseupdater: | ||
| dir: licenseupdater | ||
| cmds: | ||
| - go build -o ../.build/licenseupdater . |
There was a problem hiding this comment.
If the update-licenses target has the following command
go run -C licenseupdater ./... -config ../.licenseupdater.yaml
then I don't understand why licenseupdater needs to be build.
There was a problem hiding this comment.
Ah, I think this is just due to a bad merge resolution on my behalf. Sorry! This can be removed or we can update the go run. No strong feelings.
|
The integration tests would be fixed in next commits. This PR needs to be merged in order to release operator. |
This commit cuts release v5.9.19 of the redpanda chart and fixes
TestChartYAMLVersionsto work with changie and the new repo structure.