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

Support for: tmpfs #436

Closed
cdrage opened this issue Feb 22, 2017 · 4 comments · Fixed by #484
Closed

Support for: tmpfs #436

cdrage opened this issue Feb 22, 2017 · 4 comments · Fixed by #484

Comments

@cdrage
Copy link
Member

cdrage commented Feb 22, 2017

No error / warning when using this key.

@cdrage cdrage changed the title Support for tmpfs: /run Support for: tmpfs Feb 22, 2017
@kadel
Copy link
Member

kadel commented Mar 1, 2017

There is actually equivalent in Kubernetes.
Its EmptyDir volume that has medium: "memory"

https://kubernetes.io/docs/user-guide/volumes/#emptydir

@kadel
Copy link
Member

kadel commented Mar 3, 2017

My proposal is to convert tmpfs to empty dir volume.

for docker-compose.yml like this:

version: "2"

services:
    foo:
        image: busybox
        command: ["sh","-c", "mount | grep /foo; sleep 3200"]
        tmpfs: 
           - /foo

kompose should generate objects that have PodSpec like this:

      metadata:
        labels:
          service: foo
      spec:
        containers:
        - args:
          - sh
          - -c
          - mount | grep /foo; sleep 3200
          image: busybox
          name: foo
          volumeMounts:
          - mountPath: /foo
            name: foo-claim0
        restartPolicy: Always
        volumes:
        - name: foo-claim0
          emptyDir:
              - medium: "Memory"

volumeMounts and volumes are important parts.

@cdrage
Copy link
Member Author

cdrage commented Mar 3, 2017

@kadel That looks great. I think that'd be the best way to convert 'tmpfs" to Kubernetes. Even if it's not used very much in docker-compose I think it's an awesome way to convert to a kubernetes-equivilant.

@surajnarwade
Copy link
Contributor

I would like to work on it :)

surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 10, 2017
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 10, 2017
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 10, 2017
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 13, 2017
fixes kubernetes#436
This commit will add support for tmpfs, configEmptyVolumeSource
function is being modified as it have to work in two ways now.
(For emptyvols and tmpfs)
Added unit test for tmpfs too.
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 15, 2017
fixes kubernetes#436
This commit will add support for tmpfs, configEmptyVolumeSource
function is being modified as it have to work in two ways now.
(For emptyvols and tmpfs)
Added unit test for tmpfs too.
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 16, 2017
fixes kubernetes#436
This commit will add support for tmpfs, configEmptyVolumeSource
function is being modified as it have to work in two ways now.
(For emptyvols and tmpfs)
Added unit test for tmpfs too.
surajnarwade added a commit to surajnarwade/kompose that referenced this issue Mar 16, 2017
fixes kubernetes#436
This commit will add support for tmpfs, configEmptyVolumeSource
function is being modified as it have to work in two ways now.
(For emptyvols and tmpfs)
Added unit test for tmpfs too.
procrypt pushed a commit to procrypt/kompose that referenced this issue Mar 20, 2017
fixes kubernetes#436
This commit will add support for tmpfs, configEmptyVolumeSource
function is being modified as it have to work in two ways now.
(For emptyvols and tmpfs)
Added unit test for tmpfs too.
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 a pull request may close this issue.

3 participants