Skip to content

chore: Update references from GCS buckets to new storage paths #537

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

Merged
merged 2 commits into from
May 6, 2025

Conversation

jstlaurent
Copy link
Contributor

@jstlaurent jstlaurent commented Apr 30, 2025

Changelogs

  • Updated existing HTTP paths from https://storage.googleapis.com/graphium-public/ to https://storage.valencelabs.com/graphium/
  • Updated existing HTTP paths from https://storage.googleapis.com/datasets-public-research/ to https://storage.valencelabs.com/datasets-public-research/
  • Updated gs and s3 protocol URL for both buckets to use http protocol instead

Checklist:

  • Was this PR discussed in an issue? It is recommended to first discuss a new feature into a GitHub issue before opening a PR.
  • Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
  • Update the API documentation is a new function is added, or an existing one is deleted.
  • Write concise and explanatory changelogs above.
  • If possible, assign one of the following labels to the PR: feature, fix or test (or ask a maintainer to do it for you).

Both public GCS buckets (graphium-public and datasets-public-research) are being retired on May 30th 2025, and the existing links will cease to function. All the data was replicated as-is to a new public storage location, accessible using HTTP URLs with the prefix https://storage.valencelabs.com/.

@jstlaurent jstlaurent requested a review from DomInvivo as a code owner April 30, 2025 17:13
@jstlaurent jstlaurent self-assigned this Apr 30, 2025
@jstlaurent jstlaurent force-pushed the chore/update-bucket-references branch from 8bc4f78 to 4d3bee0 Compare April 30, 2025 17:27
@jstlaurent jstlaurent force-pushed the chore/update-bucket-references branch from 4d3bee0 to 4621a5d Compare April 30, 2025 17:37
@jstlaurent
Copy link
Contributor Author

Test failures appear to be unrelated to my changes.

Copy link
Collaborator

@DomInvivo DomInvivo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved!

@jstlaurent jstlaurent merged commit f1c1809 into main May 6, 2025
1 of 4 checks passed
@jstlaurent jstlaurent deleted the chore/update-bucket-references branch May 6, 2025 15:03
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.

2 participants