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

Add command to sync tool to store configs into local dev R2 #3865

Conversation

danyx23
Copy link
Contributor

@danyx23 danyx23 commented Aug 12, 2024

So far, the syncGraphersToR2 tool lets you sync all needed entries from the chart_configs table to an actual R2 bucket. This is useful for production and staging servers but in local dev the R2 bindings currently can't connect to an actual R2 bucket and instead simulate one using a local folder.

This PR adds a tool to store single chart configs by slug from the database into the local R2 bucket by invoking the wrangler cli tool for storing R2 objects.

@owidbot
Copy link
Contributor

owidbot commented Aug 12, 2024

Quick links (staging server):

Site Admin Wizard

Login: ssh owid@staging-site-danyx23-add-subcommand-to-sync-graphers-to-r2

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 (a618c2) ✅

Edited: 2024-08-15 15:09:34 UTC
Execution time: 1.34 seconds

Copy link
Contributor Author

danyx23 commented Aug 13, 2024

@danyx23 danyx23 changed the base branch from use-r2-grapher-configs-in-cf-functions to use-r2-in-cf-functions August 13, 2024 13:51
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from ed7d910 to a7ad2fb Compare August 13, 2024 14:49
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from a7ad2fb to 03be17a Compare August 13, 2024 16:36
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from 03be17a to 57be3d9 Compare August 13, 2024 16:55
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from 57be3d9 to cc54317 Compare August 13, 2024 20:21
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from cc54317 to 22eaef8 Compare August 13, 2024 20:50
@danyx23 danyx23 changed the title Create a PR that changes the script in ./devTools/syncGraphersToR2 so that it supports multiple commands Add command to sync tool to store configs into local dev R2 Aug 13, 2024
@danyx23 danyx23 force-pushed the danyx23/add-subcommand-to-sync-graphers-to-r2 branch from 22eaef8 to 958e4fa Compare August 14, 2024 09:46
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