Skip to content

[WIP] Convert flannel bools to single stringslice#6897

Closed
dereknola wants to merge 2 commits intok3s-io:masterfrom
dereknola:flannel_config
Closed

[WIP] Convert flannel bools to single stringslice#6897
dereknola wants to merge 2 commits intok3s-io:masterfrom
dereknola:flannel_config

Conversation

@dereknola
Copy link
Member

Signed-off-by: Derek Nola derek.nola@suse.com

Proposed Changes

  • Convert the growing list of flannel booleans to a single flannel-options flag.

Types of Changes

Verification

Testing

Linked Issues

User-Facing Change


Further Comments

Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.com>
@dereknola dereknola requested a review from a team as a code owner February 6, 2023 23:42
@brandond
Copy link
Member

brandond commented Feb 9, 2023

This sounds a lot like what @manuelbuil was attempting in #6557 and we ultimately decided against. I still thing it's worth doing an ADR for the Flannel CLI stuff since we keep revisiting it.

@dereknola dereknola mentioned this pull request Feb 16, 2023
@dereknola dereknola marked this pull request as draft March 8, 2023 17:40
@caroline-suse-rancher
Copy link
Contributor

Hey @dereknola should we keep this one open or close?

@dereknola dereknola closed this Feb 5, 2024
@dereknola
Copy link
Member Author

I closed it.... This never went anywhere

@dereknola dereknola deleted the flannel_config branch December 20, 2024 18:50
@rdesaintleger rdesaintleger mentioned this pull request Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants