-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This commit updates `RedpandaClusterSpec` to support the `TrustStore` option that was added to the helm chart in redpanda-data/helm-charts@f539d8f. Additionally, it adds a regression test to find divergences between the CRD and helm chart. The majority of this test case is currently disabled due to many such divergences. Finally, this commit also contains a handful of small fixes that were found in the above test case: - Added `omitempty` to `Auth.SASL`, `RedpandaMemory.{Memory,ReserveMemory}`. - Deprecated `ListenerTLS.SecretRef` as it's not a real value. - Deprecated `UsageStats.Organization` as it's been removed from the helm chart. - Removed unused types `Limits`, `Requests`, `TopologySpreadConstraintsItems`. - Corrected `TopologySpreadConstraint` to be a slice. (See note) Technically, changing the type of `TopologySpreadConstraint` is a breaking change. However, specifying it would have caused deployments to immediately break. Therefore it's reasonably safe to assume this change will not affect any users. Changing the casing of `FullNameOverride` is being deferred as it is possible to specify it without breaking deployments and changing the casing could result in breakages as that would be seen as a field addition and removal from the API perspective.
- Loading branch information
Showing
6 changed files
with
1,458 additions
and
342 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.