Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v22.2.x] rpk: do not modify redpanda.rpc_server_tls field (manual backport) #9367

Merged
merged 2 commits into from
Mar 15, 2023

Conversation

twmb
Copy link
Contributor

@twmb twmb commented Mar 10, 2023

Manual backport of PR #7719
Fixes #7644

Release Notes

Bug Fixes

  • rpk will not modify redpanda.rpc_server_tls property when decoding the redpanda.yaml, which means that it will leave the field as a list or as an element depending on what the user has configured before executing rpk commands.

rpk now will treat this field like an unmanaged
field, which means that we can have either a list,
or an element, and rpk will respect that.
@twmb twmb added area/rpk kind/backport PRs targeting a stable branch labels Mar 10, 2023
@twmb twmb requested review from a team, r-vasquez, gene-redpanda and BenPope as code owners March 10, 2023 00:56
@twmb twmb requested review from andrewhsu and removed request for a team March 10, 2023 00:56
@twmb twmb changed the base branch from dev to v22.2.x March 10, 2023 00:56
@twmb twmb requested a review from 0x5d as a code owner March 10, 2023 00:56
@CLAassistant
Copy link

CLAassistant commented Mar 10, 2023

CLA assistant check
All committers have signed the CLA.

@twmb twmb removed request for gene-redpanda, BenPope, a team, andrewhsu and 0x5d March 10, 2023 00:56
@twmb twmb force-pushed the v22.2.x-rpc-tls-fix-v2 branch 2 times, most recently from 94493fe to 2b7a0ec Compare March 10, 2023 01:07
@piyushredpanda piyushredpanda added this to the v22.2.11 milestone Mar 10, 2023
@twmb twmb force-pushed the v22.2.x-rpc-tls-fix-v2 branch 4 times, most recently from 3f32172 to afb06b2 Compare March 10, 2023 07:03
If the rpc_tls_server config is a list rpk will
do a TLS config validation and print a warning if
the config is invalid.
@twmb twmb force-pushed the v22.2.x-rpc-tls-fix-v2 branch from a6c25be to f1fc4c2 Compare March 10, 2023 22:46
@twmb
Copy link
Contributor Author

twmb commented Mar 15, 2023

Failure is #7756

@twmb twmb merged commit 6275698 into redpanda-data:v22.2.x Mar 15, 2023
@twmb twmb deleted the v22.2.x-rpc-tls-fix-v2 branch March 15, 2023 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rpk kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v22.3.x] rpk: do not modify input redpanda.rpc_server_tls field
4 participants