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

Update kind cluster example with working local storage for all nodes #116

Merged
merged 4 commits into from
Mar 2, 2019

Conversation

rimusz
Copy link
Contributor

@rimusz rimusz commented Feb 21, 2019

Signed-off-by: rimas [email protected]

@rimusz
Copy link
Contributor Author

rimusz commented Feb 21, 2019

cc @unguiculus
screenshot 2019-02-21 12 45 56

@rimusz
Copy link
Contributor Author

rimusz commented Feb 22, 2019

Do not merge as rancher/local-path-provisioner generally seems to support multi-node clusters, but volumes always seem to be mounted using root as group irrespective of a specified security context, so some charts testing will fail.

See rancher/local-path-provisioner#7

@rimusz rimusz changed the title Update kind cluster example with working local storage for all nodes Update kind cluster example with working local storage for all nodes [do not merge] Feb 22, 2019
@unguiculus
Copy link
Member

Closing issue for now.

@unguiculus unguiculus closed this Feb 25, 2019
@unguiculus
Copy link
Member

Let's give it another try. The issue in local-path-provisioner was fixed.

@unguiculus unguiculus reopened this Feb 27, 2019
@helm-bot helm-bot added size/L and removed size/L labels Feb 27, 2019
Signed-off-by: Reinhard Nägele <[email protected]>
@helm-bot helm-bot added size/L and removed size/L labels Feb 27, 2019
@unguiculus
Copy link
Member

Still not quite there yet.

Fix coming: rancher/local-path-provisioner#9

@cpanato
Copy link
Member

cpanato commented Feb 28, 2019

cool! i'm using this in our tests as well

@helm-bot helm-bot added size/L and removed size/L labels Mar 2, 2019
Signed-off-by: Reinhard Nägele <[email protected]>
@helm-bot helm-bot added size/L and removed size/L labels Mar 2, 2019
@unguiculus unguiculus changed the title Update kind cluster example with working local storage for all nodes [do not merge] Update kind cluster example with working local storage for all nodes Mar 2, 2019
@unguiculus unguiculus merged commit 8e67de4 into helm:master Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants