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

DM-48924: CLI command for migrating the HTML cache keys to the new format #95

Merged
merged 6 commits into from
Feb 28, 2025

Conversation

jonathansick
Copy link
Member

This CLI command can migrate the NbHtmlCache keys from the old format (based on a base64 encoding of a JSON object with the parameter values and display settings) to the new format computed more reliably from the URL query string.

This command can run in a dry-run mode and can also work on either a specific page, or with all pages.

We'lll run this command via a Kubernetes job.

@jonathansick jonathansick force-pushed the tickets/DM-48924-cache-migration branch 3 times, most recently from 07d452c to 548d276 Compare February 28, 2025 21:43
Base automatically changed from tickets/DM-48924 to main February 28, 2025 22:41
This CLI command can migrate the NbHtmlCache keys from the old format
(based on a b64 encoding of a JSON object with the parameter values and
display settings) to the new format computed more reliably from the URL
query string.

This command can run in a dry-run mode and can also work on either a
specific page, or with all pages.
This was tricky to set up because I wanted to test it at the service
level, but we don't have a useful factory for creating services outside
the app context yet.
Some of the early pages in Times Square on data dev have invalid
parameters --- skip over those.
@jonathansick jonathansick force-pushed the tickets/DM-48924-cache-migration branch from 904a632 to f5f40ac Compare February 28, 2025 22:43
@jonathansick jonathansick marked this pull request as ready for review February 28, 2025 22:45
@jonathansick jonathansick merged commit 69dd27b into main Feb 28, 2025
4 checks passed
@jonathansick jonathansick deleted the tickets/DM-48924-cache-migration branch February 28, 2025 22:48
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.

1 participant