-
Notifications
You must be signed in to change notification settings - Fork 4
Release v1.5.2 with metrics #253
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
Changes from 22 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
3d37f2e
Renamed LOGGER to logger.
gaurav 4d271de
Added logging to /synonyms API endpoint.
gaurav 096f79c
Added logging to Lookup.
gaurav 923e079
Added logging for bulk_lookup().
gaurav c8ae693
Added NameRes version to /status.
gaurav bc02650
Added BIOLINK_MODEL_TAG.
gaurav 9c255ec
Added basic tests for /status.
gaurav 27f9f4b
Improved tests.
gaurav d4d23b1
Attempted to fix issues.
gaurav b19316c
Fixed key name.
gaurav 7f02500
Update api/server.py
gaurav 3a2bbc9
Update api/server.py
gaurav f7267dc
Added a "recent times" that allows us to track query times.
gaurav 9c89abf
Improved name.
gaurav a0707a6
Apply suggestion from @gaurav
gaurav a3e00f8
Add Solr performance diagnostics to /status and logging
gaurav fc168b8
Added on:pull_request trigger for testing.
gaurav 3979968
Add query rate estimation to /status
gaurav 688737e
Incremented version to v1.5.2.
gaurav a05e4b8
Group Solr metrics under a single 'solr' key in /status
gaurav c9367b6
Merge query_log and expand rate statistics in /status
gaurav a211880
Remove recent_times_ms from /status response
gaurav 66b5654
Fix negative inter_arrival_ms from out-of-order log entries
gaurav 03672b0
Fix three bugs identified in PR review
gaurav 9fcb4e8
Add physical_memory_used_pct to /status Solr OS metrics
gaurav c9fc44c
Extract Solr logic from server.py into api/solr.py with OO design
gaurav 3624ca6
Added on:pull_request trigger for testing.
gaurav f1d3b2e
Add latency bucket proportions to recent_queries in /status
gaurav 82e9e1a
Add ?full=true parameter to /status to reduce Solr load
gaurav 36c7f11
Document ?full query parameter in FastAPI schema for /status
gaurav ddcd29e
Simplify latency bucket counting: single pass + named constants
gaurav 85cb011
Fixed bug: changed default infores to NameRes, not NodeNorm.
gaurav 05e0a0a
Fix doc inaccuracies found during sync-docs audit
gaurav da3f360
Replaced time_ns() with perf_counter_ns().
gaurav 9a805d8
Removed on:pull_request trigger after testing.
gaurav ba2a9b9
Tweaked tests to get them to pass.
gaurav 0e9b80e
Oops, messed with the wrong test.
gaurav 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
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
Oops, something went wrong.
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.
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.
The release workflow now triggers on
pull_request, but the job always logs into GHCR and runsdocker/build-push-actionwithpush: true. On PRs this can publish images unintentionally and also referencesgithub.event.release.target_commitish, which won't exist for PR events. Consider removing thepull_requesttrigger or gating the push/build-args to release events only (e.g.,if: github.event_name == 'release').