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

[CNB] Add optional suffix for bucket names of secondary blob store #1307

Open
chibenwa opened this issue Nov 14, 2024 · 3 comments · May be fixed by #1309
Open

[CNB] Add optional suffix for bucket names of secondary blob store #1307

chibenwa opened this issue Nov 14, 2024 · 3 comments · May be fixed by #1309
Assignees
Labels
bug Something isn't working customer

Comments

@chibenwa
Copy link
Member

Description

I want to have distinct bucket names for primary blob store and secondary blob store because I need to host them on the same OVH account.

image

Hence, I need a blobStore.properties :

objectstorage.s3.secondary.bucket.suffix=copy

Also include a HELM chart property for this.

@chibenwa chibenwa added bug Something isn't working customer labels Nov 14, 2024
@quantranhong1999
Copy link
Member

How about objectstorage.s3.secondary.bucketPrefix?

Suffix would likely require changes on James's S3BlobStoreConfiguration.

@quantranhong1999
Copy link
Member

code change: #1309

@quantranhong1999
Copy link
Member

helm v5.2: https://ci.linagora.com/linagora/lrs/saas/tools/helm-charts/tmail-backend/-/merge_requests/109

I would take the chance to backport the secondary blob store config to branch 4.0 too as it seems 4.0 still can be used by CNB prod...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working customer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants