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.

Sign up for GitHub

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

Jump to bottom

create pod instead of process when mount & watch mount pod in node service #100

Merged

davies merged 18 commits into juicedata:master from zwwhdls:feature/csi-operator

Jul 12, 2021

Member

zwwhdls commented Jun 30, 2021

create pod instead of process when mount
recording mount reference of mount path in configMap
watch mount pod in node service

zwwhdls added 3 commits

June 28, 2021 14:59


          create pod in noce service

21a352f


          watch pod in driver & update chart

87c27fc


          remove debug code

3f5f05c

zwwhdls requested review from chnliyong and yujunz

June 30, 2021 03:26

davies reviewed

View reviewed changes

charts/juicefs-csi-driver/values.yaml Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

charts/juicefs-csi-driver/values.yaml Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

charts/juicefs-csi-driver/values.yaml Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/controller/pod_driver.go Show resolved Hide resolved

xiaogaozi linked an issue

that may be closed by this pull request

Support rolling upgrade #80

Closed

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/resource.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/util.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/driver/node.go Outdated Show resolved Hide resolved

yujunz reviewed

View reviewed changes

Contributor

yujunz left a comment

Haven't go over the code yet. Some early comments on RBAC.

charts/juicefs-csi-driver/templates/daemonset.yaml Outdated Show resolved Hide resolved

charts/juicefs-csi-driver/templates/serviceaccount.yaml Outdated Show resolved Hide resolved

charts/juicefs-csi-driver/templates/serviceaccount.yaml Outdated Show resolved Hide resolved

zwwhdls added 5 commits

July 1, 2021 23:37


          fix util

ddd4ca4


          debug

901d895


          debug

6eadbdc


          debug authfs

0abf63d


          finish authFs & mount & fix some bugs

a8f6965

yujunz reviewed

View reviewed changes

Contributor

yujunz left a comment

Overall it looks good to me as long as tested OK.

davies reviewed

View reviewed changes

pkg/util/pod.go Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/util/pod.go Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/resource.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/resource.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/resource.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/client.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/driver/node.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/driver/node.go Outdated Show resolved Hide resolved

zwwhdls added 4 commits

July 6, 2021 15:57


          update authfs in driver & fix some bugs

7bb8b31


          mount pod resource read from env & set default value in chart

ee91e60


          add licence header

7e228f5


          cal refs in pod annotation instead of configMap

11136ad

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved

davies reviewed

View reviewed changes

pkg/juicefs/juicefs.go Outdated Show resolved Hide resolved


          fix pod patch

832ccea

davies approved these changes

View reviewed changes

davies and others added 5 commits

July 9, 2021 23:45


          remove unused lock

da11cd8


          Update juicefs.go

b41cbf1


          add annotation prefix & fix mount pod metrics & add mount pod priorit…

2cc9d2a

…y & add latest chart


          add conflict retry

fcf479c


          fix mount option

davies merged commit 51a4083 into juicedata:master

davies mentioned this pull request

Spawn a new pod for each juicefs mount point #49

Closed

kevincai mentioned this pull request

Nodeserver shutdown makes mount unavailable until restart business pod Alluxio/alluxio#14917

Closed

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

Labels

None yet