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

Update development documentation for CNI installation #8

Closed
hama-25179 opened this issue Jan 19, 2023 · 1 comment · Fixed by #14
Closed

Update development documentation for CNI installation #8

hama-25179 opened this issue Jan 19, 2023 · 1 comment · Fixed by #14

Comments

@hama-25179
Copy link
Contributor

When set up according to the development docs, CNI is not installed on the workload cluster.
https://github.com/kubernetes-sigs/cluster-api-addon-provider-helm/blob/main/docs/development.md

If CNI is not installed, the sample HelmChartProxy of nginx-ingress will not work, so I want to install CNI automatically.
Registering HelmChartProxy of calico as a Tilt resource, developers will no longer have to worry about CNI.

May I create a pull request for the above content?
I already created a HelmChartProxy manifest of calico and a shell script to register as a Tilt resource, and improved the document.

Note that I have worked together with @hiromu-a5a on this issue.

@hama-25179 hama-25179 changed the title Update development documentation for CNI instllation Update development documentation for CNI installation Jan 19, 2023
@Jont828
Copy link
Contributor

Jont828 commented Jan 19, 2023

Thanks for opening this issue, I'll try to repro when I have some time. And absolutely, contributions are welcome!

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 a pull request may close this issue.

2 participants