-
Notifications
You must be signed in to change notification settings - Fork 79
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
optional cephfs #1511
optional cephfs #1511
Conversation
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-9cebaa8-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-cb8b192-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-fd4b0eb-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-de1dcd1-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-0781b0a-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-ae2db07-rook-1.4.9-k8s-docker |
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-25b1886-rook-1.4.9-k8s-docker |
aa98228
to
5cd8354
Compare
Testgrid Run(s) Executing @ https://testgrid.kurl.sh/run/pr-1511-703a469-rook-1.4.9-k8s-docker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need to make kurlkinds
addons/rook/template/base/install.sh
Outdated
insert_resources "$dst/kustomization.yaml" filesystem.yaml | ||
insert_patches_strategic_merge "$dst/kustomization.yaml" patches/cephfs-storageclass.yaml | ||
insert_patches_strategic_merge "$dst/kustomization.yaml" patches/filesystem.yaml | ||
cat << EOF >> "$dst/kustomization.yaml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems brittle? is it possible to use function insert_patches_(Json6902)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do not yet have such a function, though I can add one
to clarify, I will remove the duplicate |
044d6f9
to
c062255
Compare
No description provided.