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.

Sign up for GitHub

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

Jump to bottom

Convert scripts/install-cni.sh to a golang #33

Open
jingax10 opened this issue Sep 12, 2018 · 2 comments · May be fixed by #97
Open

Convert scripts/install-cni.sh to a golang #33

jingax10 opened this issue Sep 12, 2018 · 2 comments · May be fixed by #97
Assignees
@rlenglet @varunmar
Labels
enhancement New feature or request

Comments

@jingax10
Copy link
Contributor

It is hard to unit test CNI generator by scripts/install-cni.sh. We want to convert to golang to make it unit testable.
@jingax10 jingax10 added the enhancement New feature or request label Sep 13, 2018
@varunmar varunmar mentioned this issue Feb 5, 2019
Merged
@rlenglet
Copy link

@varunmar Can you please reassign this issue to @bcheung? He who will start working on it soon.

@varunmar varunmar removed their assignment Jun 30, 2020
@varunmar
Copy link
Contributor

Hmm. I think I can't assign it to him directly. I wonder if it restricts it to the GoogleCloudPlatform org or something

@bcheung bcheung linked a pull request Aug 21, 2020 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rlenglet rlenglet

@varunmar varunmar

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Install CNI Refactor bcheung/netd
3 participants
@rlenglet @jingax10 @varunmar