-
Notifications
You must be signed in to change notification settings - Fork 16
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
S3 instead of Ceph #2663
Comments
/area documentation |
/kind cleanup |
/remove kind/feature area/documentation |
/remove kind/feature |
/triage needs-information |
Potential problems:
- CephStore is in `__all__` (__init__.py), so changing it is probably a
breaking change. I think at least thoth-station/slo-reporter use it,
might be other, I just did a grep on the repos I had cloned.
|
Relevant: code external to storages that would need modification). https://github.com/search?q=org%3Athoth-station+-repo%3Athoth-station%2Fstorages+CephStore&type=code |
/sig stack-guidance |
Issues needing reporter input close after 60d, If there is new input, reopen with |
User stories
API
implementation
Acceptance criterias
the thoth.storages public interface)
Since we use Ceph as S3 store only (AFAICT), can we use consistently S3 instead
of Ceph in the code and documentation ?
In the documentation or in the code, using ceph as a name does make it clear
that we're using an object store, and targeting the S3 API (since Ceph has other
storages modes).
The text was updated successfully, but these errors were encountered: