-
Notifications
You must be signed in to change notification settings - Fork 85
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
O_TMPFILE on Linux with non-supporting kernel ("/tmp: Operation not supported") #1289
Comments
I'm disinclined to do anything about this without understanding how/why the option is disabled. If the environment in the linked issue is as reported, there's no reason this should be happening. |
The StackRox project has had success running ClairCore by setting the |
I find it's dangerous as cannot determine size of this space (empty dir can have limit, but not sure about usage), and in cluster it
utilizes space of the node, which is bad, so it makes sense to provide a
volume from special nfs server.
…On Mon, Mar 18, 2024, 19:42 Ross Tannenbaum ***@***.***> wrote:
The StackRox project
<https://github.com/stackrox/stackrox/blob/4.4.0/image/templates/helm/shared/templates/02-scanner-v4-07-indexer-deployment.yaml#L145>
has had success running ClairCore by setting the /tmp volume to emptyDir:
{}. We found this resolved someone else's problem, too. Have you tried
that?
—
Reply to this email directly, view it on GitHub
<#1289 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACO3F7I5URNQJUNKVZHPXDYY4RRTAVCNFSM6AAAAABE3H424OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBUGU2TIOJRGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
NFS is not supported |
@hdonnay Could you please elaborate on that? I mean, I cannot find anything about it at Clair Documentation pages. |
To clarify, in my case host OS is Oracle Linux. As for |
And yes, quite expected, |
We've just begun testing claircore v1.5.25 & clair v4.7.3 and we're also running into this issue:
|
NFS doesn't support O_TMPFILE and is documented to have issues around file locking, making it a bad choice for scratch space like |
There's no way to disable O_TMPFILE support, so either your vendor is doing some very invasive kernel patching and you should contact their support, or there's something odd about your environment. What does |
What does |
There is no output (catting |
@cathyyoung FYI I rolled back the update, so that may be why it is not there. |
I understand the nice functionality for scratch space provided by flag support, but there should be a fallback option. Cleaning up can be done with standard measures (aka |
Output does not contain a mention of
|
The linked PR adds a fallback path and additional documentation. This will be in the next release. |
That's great news @hdonnay, thanks for putting that fallback in as we're currently blocked on this. When do you anticipate the next release will be? |
Happens in some k8s environments for me, as host kernel seems to be used as Docker kernel:
I was struggling with this for last week and found similar issues in other projects, and just few days ago @kamalpreetSec reported it under clair issues
Error opening temporary file in /tmp: Operation not supported
For me it happened with tmpfs and nfsfs in k8s cluster, both rootless and non-rootless container, alpine, debian and original ubi-minimal, default and 777 permissions.
Similar patches: https://github.com/libvips/libvips/pull/1155/files (original thread)
Pull request by @RTann is this repo, great job on introducing OS-specific temp file handling in #1140 . This time it is more corner-case'ish issue.
Proposed solution:
fallback to opening temporary dir/file without using
unix.O_TMPFILE
flag.cc @hdonnay @kamalpreetSec
The text was updated successfully, but these errors were encountered: