thoth-station / storages

Storage and database adapters for project Thoth
https://thoth-station.github.io/
GNU General Public License v3.0
14 stars 16 forks source link

THOTH_CEPH_PREFIX should use the same mechanims as other env vars #2717

Open VannTen opened 2 years ago

VannTen commented 2 years ago

Problem statement

As a Thoth developer, I would like to have a single place defining ceph prefix in CephStore, using and environment variable. (Either in storages code itself, or in other components using storage)

Proposal description

Use same logic that for bucket and other (see thoth/storages/ceph.py). I think this needs some work on adapting storages codes which set the prefix itself for "subprefixing".

Introducing a global prefix / sub prefix might be something to think about

Additional context

Relevant: #2700

Acceptance Criteria

/kind feature /sig stack-guidance

VannTen commented 2 years ago

Same remarks for THOTH_DEPLOYMENT_NAME

VannTen commented 2 years ago

/priority important-longterm