Skip to content
This repository has been archived by the owner on Jan 19, 2018. It is now read-only.

ReadWrite no longer a parameter for Persistent Storage. #587

Closed
cdrage opened this issue Mar 2, 2016 · 7 comments
Closed

ReadWrite no longer a parameter for Persistent Storage. #587

cdrage opened this issue Mar 2, 2016 · 7 comments
Labels

Comments

@cdrage
Copy link
Member

cdrage commented Mar 2, 2016

http://www.projectatomic.io/blog/2016/01/atomic-app-0-3-1/ and docs need to be updated.

This will be fixed by #576 for the README.md, incoming PR for atomic-site.

Basically, we implemented persistent storage before kubernetes hit 1.0.0 and thus the API changed before release.

ReadWriteOnce, ReadOnlyMany, ReadWriteMany are the new options.

@cdrage cdrage added the need_doc label Mar 2, 2016
@jberkus
Copy link

jberkus commented Mar 3, 2016

Do those names also work with Marathon? How do we interpret them for Docker?

@cdrage
Copy link
Member Author

cdrage commented Mar 3, 2016

@jberkus

Unfortunately just Kubernetes is supported at the moment :( the docs will be updated soon with the information about it.

@jberkus
Copy link

jberkus commented Mar 3, 2016

My point is, we shouldn't be using provider-specific names in Nulecule files.

@goern
Copy link
Contributor

goern commented Mar 3, 2016

ReadWriteOnce, ReadOnlyMany, ReadWriteMany these words seem to nail the desired behavior of the persistent volume pretty well, I would consider them as 'common sense and meaningful' rather than 'provider specific names'.

@jberkus
Copy link

jberkus commented Mar 3, 2016

Ok. In that case, though, we'll want to have plans for how those will map to other providers.

@dustymabe
Copy link
Contributor

@cdrage - there are also several tests that mention 'ReadWrite':

./tests/units/persistent_storage/test_ps.py
./tests/units/persistent_storage/test_examples/ps-helloapache/Nulecule

@cdrage
Copy link
Member Author

cdrage commented Mar 15, 2016

Closing as #610 #611 has fixed this. Plus the update to the docs

@cdrage cdrage closed this as completed Mar 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants