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 antctl CLI command to run pre-installation sanity checks #6153

Closed
antoninbas opened this issue Mar 27, 2024 · 2 comments
Closed

Add antctl CLI command to run pre-installation sanity checks #6153

antoninbas opened this issue Mar 27, 2024 · 2 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@antoninbas
Copy link
Contributor

antoninbas commented Mar 27, 2024

This is a subtask for #5896

Regardless of the installation method for Antrea, it would be useful to have an antctl CLI command to quickly validate that a K8s cluster is ready for Antrea installation.

Some ideas we had for pre-installation sanity checks:

@tnqn @luolanzone feel free to add some ideas as well

Ideally, this CLI command would belong to the same top-level command group as the "post-installation" command (#6061).

@antoninbas antoninbas added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 27, 2024
@antoninbas
Copy link
Contributor Author

cc @kanha-gupta

@tnqn
Copy link
Member

tnqn commented Mar 27, 2024

check if another CNI is present; we can look for them by name (Cilium, Calico, Weave, Flannel)

Perhaps we can just check this by ensuring no conf in /etc/cni/net.d/ will take priority over 10-antrea.conflist, which could generally work for all CNIs. Also, even if there is no DaemonSet running for other CNIs, container runtime will not invoke antrea unless the config file is the 1st in alphabetical order. I handled several cases that users said pod network is not working after installing antrea then it turned out that there was a stale CNI config in /etc/cni/net.d/ even though there was no running DaemonSet for that CNI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants