Clarify storage driver env vars #729
Conversation
butonic
requested review from
micbar,
individual-it and
C0rby
and removed request for
micbar
|
In https://github.com/owncloud/ocis/blob/master/storage/pkg/flagset/gateway.go I can also still find: https://github.com/owncloud/ocis/blob/master/docs/extensions/storage/configuration.md has lots of |
butonic
force-pushed
the
clarify-storage-driver-env-vars
branch
from
e265078
to
e3309d6
Compare
|
Unfortunately, there is not only a storage registry in reva, but also an appregistry ... the prefix is STORAGE, that is why I left
unchanged ... |
|
@micbar doesn't the configuration.md get generated? |
Yes is does. We could delete it from git. |
|
we need also to adjust CI in phoenix, maybe after owncloud/web#4228 gets merged |
butonic
force-pushed
the
clarify-storage-driver-env-vars
branch
6 times, most recently
from
fb19ff9
to
1c22a41
Compare
butonic
force-pushed
the
clarify-storage-driver-env-vars
branch
2 times, most recently
from
87b2aba
to
69aa1ee
Compare
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
butonic
force-pushed
the
clarify-storage-driver-env-vars
branch
from
69aa1ee
to
8eb0864
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
After renaming ocsi-reva to storage and combining the storage and data providers some env vars were confusingly named
STORAGE_STORAGE_.... We are changing the prefix for driver related env vars toSTORAGE_DRIVER_.... This makes changing the storage driver using eg.:STORAGE_HOME_DRIVER=eosand setting driver options usingSTORAGE_DRIVER_EOS_LAYOUT=...less confusing.