Skip to content

fix(presign): remove storage baseUri configuration#264

Merged
andrewazores merged 2 commits intocryostatio:mainfrom
andrewazores:external-s3-virtual-host
Oct 9, 2025
Merged

fix(presign): remove storage baseUri configuration#264
andrewazores merged 2 commits intocryostatio:mainfrom
andrewazores:external-s3-virtual-host

Conversation

@andrewazores
Copy link
Member

@andrewazores andrewazores commented Sep 30, 2025

@andrewazores andrewazores marked this pull request as ready for review October 2, 2025 17:46
@andrewazores
Copy link
Member Author

Tested and confirmed that this is working with external storage and presigned transfers for reports/datasource (see linked Cryosat PR and linked dependencies).

helm install --set storage.provider.url=https://s3.us-east-005.backblazeb2.com --set storage.storageSecretName=s3cred --set storage.provider.region=us-east-1 --set storage.provider.usePathStyleAccess=false --set storage.provider.metadata.storageMode=metadata --set core.route.enabled=true --set authentication.openshift.enabled=true --set core.discovery.kubernetes.enabled=true --set core.discovery.kubernetes.namespaces='{apps1}' --set core.image.repository=quay.io/andrewazores/cryostat --set core.image.tag=external-s3-baseuri-1 --set reports.image.repository=quay.io/andrewazores/cryostat-reports --set reports.image.tag=external-s3-1 --set datasource.image.repository=quay.io/andrewazores/jfr-datasource --set datasource.image.tag=external-s3-1 cryostat --set reports.replicas=1 ./charts/cryostat

@andrewazores andrewazores requested a review from a team October 2, 2025 17:48
@andrewazores andrewazores force-pushed the external-s3-virtual-host branch from 0fcd454 to 38caaa0 Compare October 9, 2025 19:19
@andrewazores andrewazores merged commit 36018fb into cryostatio:main Oct 9, 2025
7 checks passed
@andrewazores andrewazores deleted the external-s3-virtual-host branch October 9, 2025 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant