bug: failed to parse CNI plugin versions in ubuntu-latest in github-actions #23229
Assignees
Labels
Comments
As I understand https://github.com/containernetworking/plugins/releases/tag/v0.8.0 , the CNI spec v0.4.0 was added in v0.8.0 of containernetowrking-plugins. So I think protocol 0.4.0 should be detected. And --help and --version of each executable in the directory: |
|
The XY issue is, Nomad <1.8 did not auto add |
|
@Kamilcuk these are very old versions of the CNI plugins. You should be using more current versions that will report their fingerprint correctly (and fix a ton of bugs!) |
tgross
added
theme/docs
|
I'm going to update our docs to establish a minimum version of the plugins and add a deprecation warning on the 1.8.x release notes. |
|
Hi. Is it be possible to remove the check from job specification or force Nomad to know that CNI plugins with specific version exists? It would be nice to have a |
|
@Kamilcuk I'm not sure why we'd want to do that; the reason we added the constraint is because users were running into problems where they were deployed on old versions of CNI plugins (or no CNI plugins at all!). Wouldn't this just open up users to having incorrect behavior? And in any case, CNI plugins aren't associated with the |
|
Docs are being updated in #23707 |
github-project-automation
bot
moved this from Triaging
to Done
in Nomad - Community Issues Triage
github-project-automation
bot
moved this from Triaging
to Done
in Nomad - Community Issues Triage
tgross
pushed a commit
that referenced
this issue
Aug 6, 2024
- Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583
tgross
pushed a commit
that referenced
this issue
Aug 6, 2024
- Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583
tgross
pushed a commit
that referenced
this issue
Aug 6, 2024
- Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583
tgross
added a commit
that referenced
this issue
Aug 6, 2024
) - Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583 Co-authored-by: Aimee Ukasick <[email protected]>
tgross
added a commit
that referenced
this issue
Aug 6, 2024
) - Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583 Co-authored-by: Aimee Ukasick <[email protected]>
tgross
pushed a commit
that referenced
this issue
Aug 6, 2024
- Pulled common content from multiple pages into new partials - Refactored install/index to be OS-based so I could add linux-distro-based instructions to install-consul-cni-plugins.mdx partial. The tab groups on the install/index page do match and change focus as expected. - Moved CNI overview-type content to networking/index - Refactored networking/cni to include install CNI plugins and configuration content (from install/index). - Moved CNI plugins explanation in bridge mode configuration section into bullet points. They had been #### headings, which aren't rendered in the R page TOC. I tried to simplify and format the bullet point content to be easier to scan. Ref: https://hashicorp.atlassian.net/browse/CE-661 Fixes: #23229 Fixes: #23583 Co-authored-by: Aimee Ukasick <[email protected]>
|
For those looking for a working solution for this problem, simply download the latest zip here and unzip it into |
|
I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Nomad version
Nomad v1.8.0
BuildDate 2024-05-28T17:38:17Z
Revision 28b82e4
Operating system and Environment details
github-actions ubuntu-latest
Issue
nomad does not detect CNI pluigns, but they are installed in github-actions.
Reproduction steps
Run the following in github actions:
Expected Result
Nomad should have access to CNI plugins, or report the version of them.
Actual Result
Nomad has no access to CNI plugins, but they are available at /opt/cni/bin
Is this expected? Is the CNI plugins in github-actions ubuntu-latest too old or is this a bug in version detection?
Thanks.
Nomad Server logs (if appropriate)
https://github.com/Kamilcuk/nomad-tools/actions/runs/9397590511/job/25881118293
The text was updated successfully, but these errors were encountered: