-
Notifications
You must be signed in to change notification settings - Fork 73
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
[HOTFIX]: Add manual trigger for cache updater stack #1001
Conversation
Your feature branch infrastructure has been deployed! Your webhook URL is: https://903kan80ec.execute-api.us-east-2.amazonaws.com/prod/webhook/githubEndpoint/trigger/build For more information on how to use this endpoint, follow these instructions. |
forceRun: | ||
description: Force the cache to be rebuilt. | ||
default: 'false' | ||
push: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NOTE: Remove this before merging
@@ -20,7 +29,9 @@ jobs: | |||
aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | |||
aws-region: us-east-2 | |||
- name: Rebuild Cache if New Snooty Parser Version | |||
uses: mongodb/docs-worker-actions/rebuild-parse-cache@main | |||
uses: mongodb/docs-worker-actions/rebuild-parse-cache@cache-update-hotfix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NOTE: revert this after merging this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the context on the problem. Lgtm with one nit
src/cache-updater/index.ts
Outdated
const errorMessage = `ERROR! GITHUB_BOT_USERNAME is not set. ${repoOwner}/${repoName} will not have their cache updated`; | ||
console.error(`ERROR! GITHUB_BOT_USERNAME is not set. ${repoOwner}/${repoName} will not have their cache updated`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Nit) Should we consolidate to console.error(errorMessage)
?
Stories/Links:
Notes
This PR resolves the issue related to
10gen
repositories not being built. For confirmation, I tested the process manually, and the updated cache files for10gen
repos are now present. To verify, view thesnooty-parse-cache
S3 bucket in the AWS console, and confirm that the.snooty-cloud-docs-...
ZIP file is present within the repository, and that it was updated today (February 23rd).README updates