Skip to content

Support ClusterNetworkingConfig and AuthPreference in --apply-on-startup#34524

Merged
lxea merged 2 commits intomasterfrom
lxea/apply-on-startup-cnc-ap
Nov 14, 2023
Merged

Support ClusterNetworkingConfig and AuthPreference in --apply-on-startup#34524
lxea merged 2 commits intomasterfrom
lxea/apply-on-startup-cnc-ap

Conversation

@lxea
Copy link
Copy Markdown
Contributor

@lxea lxea commented Nov 13, 2023

@lxea lxea added backport/branch/v14 no-changelog Indicates that a PR does not require a changelog entry labels Nov 13, 2023
@github-actions github-actions Bot requested review from fheinecke and r0mant November 13, 2023 13:07
@r0mant r0mant requested review from hugoShaka and removed request for fheinecke November 13, 2023 16:02
Copy link
Copy Markdown
Contributor

@hugoShaka hugoShaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add the kinds to the ResourceApplyPriority map above? This will also dynamically update the CLI help strings. I think the docs CLI reference needs to be manually edited.

Could you also add a couple cases to the existing tests in init_test.go?

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Vercel preview here: https://docs-i199aj8ae-goteleport.vercel.app/docs/ver/preview

@lxea lxea added this pull request to the merge queue Nov 14, 2023
Merged via the queue into master with commit 1861fb5 Nov 14, 2023
@lxea lxea deleted the lxea/apply-on-startup-cnc-ap branch November 14, 2023 11:32
@public-teleport-github-review-bot
Copy link
Copy Markdown

@lxea See the table below for backport results.

Branch Result
branch/v14 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants