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

Please include calico-apiserver #207

Open
cbarbian-sap opened this issue Aug 26, 2022 · 2 comments
Open

Please include calico-apiserver #207

cbarbian-sap opened this issue Aug 26, 2022 · 2 comments
Labels
lifecycle/rotten Nobody worked on this for 12 months (final aging stage)

Comments

@cbarbian-sap
Copy link

cbarbian-sap commented Aug 26, 2022

Please include calico-apiserver (https://projectcalico.docs.tigera.io/maintenance/install-apiserver) into the shoot deployment (calico-apiserver went GA with calico v3.20.0). If you don't want to make this default it would be nice to have it as an optional component, configurable via shoot spec.

Not sure, maybe this can even go into the seed ...

Background: without the calico-apiserver, the projectcalico.org/v3 api group is not available in the shoots, and people have to use the crd.proejctcalico.org/v1 variants of the calico resources. Which is unsupported and dangerous (see e.g. projectcalico/calico#6412). And I can confirm that really really bad things can happen when using the low-level ones (i.e. api group crd.projectcalico.org/v1) ...

@ScheererJ
Copy link
Member

Could you please share what really really bad things happen using the old calico apis? I am aware that the calico team is not happy about them being used, but I wonder how bad it could get.

@cbarbian-sap
Copy link
Author

cbarbian-sap commented Aug 29, 2022

It's about the missing validation/defaulting in the internal group.
For example, I missed to specify protocol: TCP in one port-matching rule of a GlobalNetworkPolicy (naively assuming that TCP would be default); result: calico accepted the resource w/o any complain, and silently just did not create the netfilter rule. Finally I found it out, but I had to study iptables-save output for a while to see it ...
Using projectcalico.org/v3 there was a proper admission error.

@gardener-robot gardener-robot added the lifecycle/stale Nobody worked on this for 6 months (will further age) label May 8, 2023
@gardener-robot gardener-robot added lifecycle/rotten Nobody worked on this for 12 months (final aging stage) and removed lifecycle/stale Nobody worked on this for 6 months (will further age) labels Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/rotten Nobody worked on this for 12 months (final aging stage)
Projects
None yet
Development

No branches or pull requests

3 participants