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

feat: utilize Scarf Gateway endpoint for RisingWave image and operator helm charts #91

Closed
wants to merge 1 commit into from

Conversation

alicelyy
Copy link

This PR updates the RisingWave configuration for helm charts to fetch the RisingWave main container image as well as the RisingWave operator container image via a Scarf endpoint. This allows RisingWave maintainers to collect basic de-identified download and adoption metrics. It does not affect where the containers are being hosted, as Scarf is only redirecting traffic back to GHCR.

image

@arkbriar
Copy link
Collaborator

arkbriar commented Jun 2, 2024

It doesn't work now.

docker pull docker.risingwave.com/risingwavelabs/risingwave:v1.8.2
Error response from daemon: failed to resolve reference "docker.risingwave.com/risingwavelabs/risingwave:v1.8.2": failed to do request: Head "https://docker.risingwave.com/v2/risingwavelabs/risingwave/manifests/v1.8.2": tls: failed to verify certificate: x509: certificate is valid for ingress.local, not docker.risingwave.com

@arkbriar
Copy link
Collaborator

Close since #98 was merged

@arkbriar arkbriar closed this Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants