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

Fix "config is \"1.0.0\", plugin supports [\"0.4.0\"]" #652

Closed

Conversation

AkihiroSuda
Copy link
Contributor

Some plugins could not be executed with cniVersion: "1.0.0" JSON because version.Current() was not included in version.PluginSupports() checks.

@AkihiroSuda
Copy link
Contributor Author

Any chance to release v1.0.1 with this?

@gaby
Copy link

gaby commented Aug 27, 2021

@squeed Can you take a look at this?

@squeed
Copy link
Member

squeed commented Sep 1, 2021

@AkihiroSuda sorry for missing this - was on vacation. Two small changes - can you make them?

Some plugins could not be executed with `cniVersion: "1.0.0"` JSON
because `version.Current()` was not included in `version.PluginSupports()` checks.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@squeed
Copy link
Member

squeed commented Sep 7, 2021

Hey, thanks for discovering this! It turns out there were a few more that needed fixing, so I filed a separate PR: #661

@AkihiroSuda AkihiroSuda closed this Sep 7, 2021
@squeed
Copy link
Member

squeed commented Sep 7, 2021

And v1.0.1 is released 🤦

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 this pull request may close these issues.

4 participants