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

tmpfs options are not parsed #290

Open
LordAro opened this issue Oct 3, 2023 · 2 comments
Open

tmpfs options are not parsed #290

LordAro opened this issue Oct 3, 2023 · 2 comments

Comments

@LordAro
Copy link

LordAro commented Oct 3, 2023

According to the docs, I should be able to specify some options when creating a tmpfs mount - https://docs.podman.io/en/latest/markdown/podman-run.1.html#tmpfs-fs

I've specified these in the nomad job file as:

        tmpfs = [
          "/tmp/csi/keys:rw,size=1000k"
        ]

But it seems not to be working, when I podman container inspect directly I get:

...
               "Tmpfs": {
                    "/tmp/csi/keys:rw,size=1000k": "rw,rprivate,nosuid,nodev,tmpcopyup"
               },
...

Which rather suggests that it's mounted a folder named /tmp/csi/keys:rw,size=1000k rather than /tmp/csi/keys

  • nomad 1.6.2
  • nomad-driver-podman 0.5.1
  • podman 4.3.1

I'm very new to this, so maybe I'm missing something obvious...

@lgfa29
Copy link
Contributor

lgfa29 commented Nov 25, 2023

Hi @LordAro 👋

Thanks for the report. I think we're missing logic here to split the input string into destination and options.

nomad-driver-podman/driver.go

Lines 1359 to 1365 in 259a211

for _, dst := range driverConfig.Tmpfs {
bind := spec.Mount{
Destination: dst,
Type: "tmpfs",
}
binds = append(binds, bind)
}

I will try to to push a fix next week.

@Cellivar
Copy link

Just tripped over this myself tonight trying to get the Ceph CSI driver running, looks like the parser is still doing the simple operation here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants