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

[CI-1296] Don't use subPath on windows: https://github.com/moby/moby/issues/30555 #2740

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

rene-dekker
Copy link
Member

@rene-dekker rene-dekker commented Jul 14, 2023

When running fluentd on windows nodes, the containers get stuck during creation:

 Warning  Failed          9s (x12 over 3m27s)   kubelet            Error: Error response from daemon: invalid volume specification: 'c:\var\lib\kubelet\pods\cb042ed8-f755-4b7b-9bf2-780f6377a29d\volumes\kubernetes.io~configmap\tigera-ca-bundle\ca-bundle.crt:c:/etc/pki/tls/cert.pem:ro': invalid mount config for type "bind": source path must be a directory

This is because we are mounting a file under a different name, using the subPath feature of volume mounts. SubPath does not work with runtime = docker on all platforms. See: moby/moby#30555

This mount is not used on windows, because on windows we don't support uploading data to external sources, so we can simply fix it by not mounting the file in question.

Copy link
Member

@coutinhop coutinhop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rene-dekker rene-dekker changed the title Don't use subPath on windows: https://github.com/moby/moby/issues/30555 [CI-1296] Don't use subPath on windows: https://github.com/moby/moby/issues/30555 Jul 19, 2023
@rene-dekker rene-dekker merged commit 43d98a1 into tigera:master Jul 19, 2023
5 checks passed
@rene-dekker rene-dekker deleted the CI-1296 branch August 10, 2023 20:01
Tamas-Biro1 pushed a commit to Tamas-Biro1/operator that referenced this pull request Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants