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

Add support for iface-regex options in flannel/canal #2996

Open
germanoeich opened this issue Jul 28, 2022 · 22 comments
Open

Add support for iface-regex options in flannel/canal #2996

germanoeich opened this issue Jul 28, 2022 · 22 comments

Comments

@germanoeich
Copy link

germanoeich commented Jul 28, 2022

flanneld has support for the iface-regex option, but this is not available in the rancher network config options. This forces operators to either modify the DaemonSet manually, which is very fragile, or enforce a consistent interface name across all nodes within the cluster, which may not always be feasible and is error prone.

This has been asked multiple times:
#2577 - Marked as stale and closed
rancher/rancher#17180 - Opened since 2018

Can I get a confirmation on if you plan to support this and/or if you would accept PRs relating to this?

@steve-todorov
Copy link

Yes! YES PLEASE!!!!

I have a custom bash script which starts rke and then does a very nasty patch to fix some issues:

  echo "Patch flannel deployment to add iface-regex so it uses the correct interfaces for flannel external traffic"  > /dev/null
  IFACE1_REGEX="192\.0\.0\.*"
  IFACE2_REGEX="192\.8\.1\.*"
  (kubectl get daemonsets.apps --namespace=kube-system canal -o yaml | yq e '(.spec.template.spec.containers.[] | select(.name == "kube-flannel")).command |= ["/opt/bin/flanneld", "--ip-masq", "--kube-subnet-mgr", "--iface-regex='$IFACE1_REGEX'", "--iface-regex='$IFACE2_REGEX'"]' - | kubectl apply -f -)

It would make my life so much, much, MUCH easier if there was an option you could put in the services section that would ... maybe something like:

network:
  plugin: canal
  options:
    flannel_backend_type: vxlan
    flannel_iface: flannel.1
    flannel_iface_regex: [ "192\.0\.0\.*", "192\.8\.1\.*", ... ]

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Still relevant.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2022

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@germanoeich
Copy link
Author

Still relevant

@steve-todorov
Copy link

@ibuildthecloud sorry for tagging you directly, but it looks like this issue (as well as rancher/rancher#17180) have been under the radar for more than 4 years now. Are there any plans to add this simple improvement?

Unfortunately Go lang is not my area of expertise, otherwise I would have opened a PR a long time ago.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2023

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Still relevant. :)

@aimbot31
Copy link

still

@steve-todorov
Copy link

Relevant.

@github-actions
Copy link
Contributor

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Still

@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Not stale.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Bump.

Copy link
Contributor

github-actions bot commented Dec 3, 2023

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Still relevant.

Copy link
Contributor

github-actions bot commented Feb 3, 2024

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

@steve-todorov
Copy link

Lol... still relevant :D

@manuelbuil
Copy link
Contributor

hey, RKE2 engineer here. We have this feature ready in RKE2 if you would consider using RKE2 instead: https://github.com/rancher/rke2-charts/blob/main-source/packages/rke2-canal/charts/values.yaml#L15

Copy link
Contributor

This repository uses an automated workflow to automatically label issues which have not had any activity (commit/comment/label) for 60 days. This helps us manage the community issues better. If the issue is still relevant, please add a comment to the issue so the workflow can remove the label and we know it is still valid. If it is no longer relevant (or possibly fixed in the latest release), the workflow will automatically close the issue in 14 days. Thank you for your contributions.

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

No branches or pull requests

5 participants