-
Notifications
You must be signed in to change notification settings - Fork 4
Cluster Caching #226
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
Closed
Closed
Cluster Caching #226
Changes from 139 commits
Commits
Show all changes
150 commits
Select commit
Hold shift + click to select a range
c6be0eb
Bring in search logic
thehabes a11c7e6
It searches!
thehabes 4cefb0b
idNegotiation on search results
thehabes 5ac7e49
All the search logic
thehabes e0f5b00
Lint, and add support for passing search options into the endpoint
thehabes c148362
polish
thehabes e3da99e
Update API documentation
thehabes 9bb1234
polish
thehabes 8d38409
polish
thehabes c376bd4
polish
thehabes e3f0553
polish
thehabes 77d5430
polish
thehabes ab2505e
polish
thehabes d3386eb
polish
thehabes cadc2f1
polish
thehabes efb7dbf
exists test for new routes
thehabes e5f2486
Update public/API.html
thehabes 052be8c
Update public/API.html
thehabes 618a3f3
Update public/API.html
thehabes 4a0093d
Update controllers/search.js
thehabes 477abfa
Update controllers/search.js
thehabes f1b79f7
get rid of utils. prefix from createExpressError
thehabes 6d063b2
Update public/API.html
thehabes 065e6bb
Update public/API.html
thehabes 4108379
slop formatting
thehabes b45e2fc
Touch ups to API.html as discussed at standup.
thehabes 18afbd4
bump version because of new search feature
thehabes b28d7bc
initia idea
thehabes a6e60c3
tests for cache
thehabes a8d368c
gog routes too
thehabes 0e18316
cleanup
thehabes 970eaed
fix cachiung
thehabes 793fd62
oh baby a lot going on here
thehabes 1e52989
merge in main
thehabes 9016fd8
structure
thehabes 84158db
Update cache/__tests__/cache.test.js
thehabes 24cf701
Changes from testing
thehabes c05d4d5
Changes from testing
thehabes 15370ec
Changes from testing
thehabes f0d31ba
changes from testing
thehabes ec744af
Update docs for limit control
thehabes 0deea37
update tests
thehabes 1f3fc8c
changes from testing
thehabes 856cd1c
changes from testing
thehabes ebd9b04
Move test files
thehabes 6cf9e21
documentation
thehabes 05bf04c
cleanup
thehabes 4f0ba84
add status
thehabes dd90275
changes from testing
thehabes c8e7a45
changes from testing
thehabes 2e39802
remove this from details
thehabes 1c81ebf
reduce logs
thehabes c4cdcd5
amendments
thehabes 5558b46
updated integration test
thehabes bcd7829
closer to real stress tests
thehabes 46943e6
Metrics
thehabes 777f9aa
Catch those hits
thehabes f75d04e
changes from testing scripts in local environment
thehabes 030366a
changes from testing scripts in local environment
thehabes 2973d61
changes from testing scripts in local environment
thehabes 2ba15f8
Changes from testing in local environment
thehabes ca97954
Changes from testing in local environment
thehabes 4a793be
Changes from testing in local environment
thehabes b8a70b0
Changes from testing in local environment
thehabes 11d815c
requirements for running the .sh files in localhost environments
thehabes e9666c3
requirements for running the .sh files in localhost environments
thehabes aa934da
requirements for running the .sh files in localhost environments
thehabes 1fca678
requirements for running the .sh files in localhost environments
thehabes 20da77d
updates from testing
thehabes f14072d
updates from testing
thehabes 128c3e7
Changes from running between environments
thehabes 14d25f9
Changes from running between environments
thehabes d2f6358
Changes from testing across environments
thehabes 1904584
changes from testing across environments
thehabes ebcc2da
changes from testing across environments
thehabes 7cfed96
Changes from testing across environments
thehabes 02e1a01
Changes from testing across environments
thehabes 0dfedd8
Changes from testing across environments
thehabes c4373b8
log touchups
thehabes b8f6b13
This should just be a warning not a failure
thehabes 82a46d2
touchup
thehabes 86760d4
Deeper check for queries, more consideration around __rerum and _id p…
thehabes fa6e2cf
CACHING switch
thehabes 3f1f399
Clean out /cache/clear route and logic
thehabes 26bba5e
documentation
thehabes 750f518
no more cacheClear
thehabes 4e17463
Dang need it for tests
thehabes cdf121b
Don't test these
thehabes 79040af
fix tests
thehabes ed823c9
Merge remote-tracking branch 'refs/remotes/origin/224-caching' into 2…
thehabes 18896ad
Fix tests
thehabes 6409fd1
cache action checks
thehabes 760a53f
Point to devstore
thehabes ec2f952
Fixes from testing against devstore
thehabes bd23fed
try again
thehabes b8716a8
Add debug logs for dev
thehabes 5380d9b
debugging
thehabes 468810d
Fix uninitialized variable error in cache-metrics.sh
thehabes 39a7ea7
Add PM2 cluster synchronization for cache operations
thehabes 975b177
Fix PM2 cluster sync - use cache.cache.size instead of cache.length()
thehabes 22b0ed1
Add debug logging to PM2 cache sync + optimize test pagination
thehabes 96e514c
Remove non-functional PM2 sync code, document cluster behavior
thehabes a839f2a
debugging
thehabes 3c31de9
Implement PM2 cluster cache synchronization
thehabes e81b3e6
Fix async cache tests and add local fallback for PM2 cluster cache
thehabes f6e82f7
update test
thehabes 972806d
remove old files
thehabes 0c7eba9
No devstore for now
thehabes ef67fde
Redo for the clustering
thehabes 2c34ba5
LRU behavior
thehabes ce86f16
reset
thehabes c41cf82
geez
thehabes 5b84dac
Changes from testing
thehabes 81cf0d8
not sure
thehabes 28510de
close
thehabes 82d00cc
close
thehabes 59fafc0
clauded around
thehabes 10498d9
Fix cache-metrics.sh to not count clock skew as operation failures
claude 3b63ff4
cleaning up and preparing for dev
claude 3834d60
big clean
thehabes 91f967b
little cleanup
claude d509c14
Support for non-cluster environment scenarios.
claude 6753267
Getting ready for dev
claude f89a434
changes for security
claude 466e4c4
extra lines
claude 4c6f5de
force deploy
claude 09efb40
This is working when mounted against the GoG app!
claude 7e5d5b0
From testing and tests
claude b7b8007
cmon GitHub
claude 8d52fb9
cmon GitHub
claude 4e27427
consolodate tests and for the love of GH don't break GitHub cmon GitHub
claude c2cf181
looking good
claude 9fc3c93
looking good
claude f8348b5
changes from testing, and new reports
claude 2508c00
wsl clock check
claude 277e488
it's working! Need to get things ready for an official review still
claude 6379d15
it's working! Need to get things ready for an official review still
claude 22ead64
Bring in claude with a yaml file. It can be called like copilot can …
claude 18d0a05
update claude yaml
claude 293e589
Clean up cache implementation: remove unused code and deprecated para…
thehabes ae2d163
ah yea token got it
claude 00e9156
Merge branch '224-cluster-caching' of https://github.com/CenterForDig…
claude 10ba3c6
key
claude 8f1f088
listen to me
claude 43393b1
Merge branch 'main' into 224-cluster-caching
claude 5bcc193
Merge branch 'main' into 224-cluster-caching
claude c48a516
changes from close review and testing
claude c0d0631
Stress tested improvements
claude 3542184
changes from testing
claude 4a6acaf
hmm
claude File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| name: Claude Code | ||
| on: | ||
| issues: | ||
| types: [opened] | ||
| issue_comment: | ||
| types: [created] | ||
| pull_request_review: | ||
| types: [submitted] | ||
| pull_request_review_comment: | ||
| types: [created] | ||
|
|
||
| jobs: | ||
| claude: | ||
| if: | | ||
| (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) || | ||
| (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) || | ||
| (github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude'))) | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
| issues: write | ||
| id-token: write | ||
| actions: read | ||
| steps: | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v5 | ||
| with: | ||
| fetch-depth: 1 | ||
|
|
||
| - name: Run Claude Code | ||
| id: claude | ||
| uses: anthropics/claude-code-action@v1 | ||
|
||
| with: | ||
| anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} | ||
| #trigger_phrase: "claude do the needful" | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -108,3 +108,6 @@ dist | |
| *.env | ||
| /nbproject/private/ | ||
| .hintrc | ||
|
|
||
| # Claude Code settings | ||
| .claude/ | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.