Skip to content
This repository was archived by the owner on Feb 8, 2021. It is now read-only.

Conversation

@bergwolf
Copy link
Member

The PR imports runv nfs support (hyperhq/runv#417) and adds nfs volumes support to hyperd, with a podfile example listed.

podfile example:

{
  "resource": {
    "vcpu": 1,
    "memory": 1024
  },
  "containers": [{
      "image": "bergwolf/ubuntu-fio",
      "volumes": [{
        "volume": "sharevolume",
        "path": "/export",
        "readOnly": false
      }]
  }],
  "volumes": [{
    "name": "sharevolume",
    "source": "192.168.80.72:/home/bergwolf",
    "format": "nas",
    "fstype": "nfs"
  }]
}
[hypervsock@~]$sudo hyperctl run -d -p ubuntu.pod
POD id is pod-VVnZwkkoZp
Time to run a POD is 7733 ms
[hypervsock@~]$sudo hyperctl exec -t pod-VVnZwkkoZp bash
root@pod-VVnZwkkoZp:/# df
Filesystem                   1K-blocks      Used Available Use% Mounted on
/dev/sda                     104805376    298180 104507196   1% /
devtmpfs                         23808         0     23808   0% /dev
tmpfs                            26940         0     26940   0% /dev/shm
rootfs                           23808     16044      7764  68% /lib/modules/4.4.28-hyper
192.168.80.72:/home/bergwolf 410056704 108126208 301930496  27% /export

Signed-off-by: Peng Tao <bergwolf@gmail.com>
Signed-off-by: Peng Tao <bergwolf@gmail.com>
{
"ImportPath": "gopkg.in/yaml.v2",
"Rev": "7ad95dd0798a40da1ccdff6dff35fd177b5edf40"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Are the packages below necessary?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, I don't think so. They were added by godeps but it looks to be a mistake. I removed these files but forgot to update Godeps.json. I'll drop them.

To include nfs volume support.

Signed-off-by: Peng Tao <bergwolf@gmail.com>
@bergwolf bergwolf force-pushed the nfs-volume branch 13 times, most recently from 312ba1b to cfef7db Compare February 8, 2017 14:11
@bergwolf
Copy link
Member Author

bergwolf commented Feb 9, 2017

retest this please @hykins

1 similar comment
@bergwolf
Copy link
Member Author

bergwolf commented Feb 9, 2017

retest this please @hykins

@bergwolf bergwolf force-pushed the nfs-volume branch 4 times, most recently from d89019f to efbd230 Compare February 10, 2017 08:10
aufs depends on 9pfs that does not have kernel export ops and thus
cannot be exported by nfs-ganesha. A new testing nfs server image
was introduced to work around it.

Signed-off-by: Peng Tao <bergwolf@gmail.com>
@bergwolf
Copy link
Member Author

@gao-feng updated and added ci case.

@@ -0,0 +1,20 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

seems like this pod file is useless?

Copy link
Member Author

Choose a reason for hiding this comment

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

It was provided as an example of how to add an nfs volume to a pod.

@gao-feng
Copy link
Contributor

LGTM

@gao-feng gao-feng merged commit 0d6f6b3 into hyperhq:master Feb 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants