Skip to content

KERN-7128 Feature/metrics #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.

Sign up for GitHub

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

Jump to bottom
Merged
ivan-velasco merged 4 commits into from
Dec 31, 2025
Merged

KERN-7128 Feature/metrics #73

ivan-velasco merged 4 commits into from
Dec 31, 2025

Conversation

@ivan-velasco
Copy link
Collaborator

@ivan-velasco ivan-velasco commented Dec 31, 2025 •
edited
Loading

  • Introduced a new command-line flag for generating a text summary output file.
⚡ Performance Metrics (Venom)
• Total HTTP Requests: 34764
• Avg Response Time: 1414 ms (P95: 3445 ms, P99: 19437 ms)
• Min/Max: 12 ms / 55983 ms
• HTTP Failures: 2127 (6.12% failure rate)
• Test Duration: 1051761.9 min

Top 5 Slowest Endpoints:
  1. POST /api/orders: 4500 ms (P95)
  2. GET /api/users: 1500 ms (P95)
  3. GET /api/products: 900 ms (P95)

ivan-velasco and others added 3 commits September 15, 2025 13:15
@ivan-velasco
Merge pull request #67 from socotra/feature/metrics-fixes
eb0f064 
Refactor metrics reporting to include generation time in UTC
@ivan-velasco
Enhance endpoint normalization in metrics aggregation to handle uniqu…
11820be 
…e endpoints and collisions. Introduced logic to apply cardinality limits only to new unique endpoints, improving metrics reporting accuracy.
@ivan-velasco
Add text summary output option and enhance threshold checking in metr…
17274bc 
…ics report

- Introduced a new command-line flag for generating a text summary output file.
- Enhanced the threshold checking logic to return detailed status and breach information.
- Implemented a function to generate a comprehensive text summary of performance metrics, including HTTP request counts, response times, and failure rates.
- Updated the metrics reporting process to include the generation of the text summary after threshold checks.
@ivan-velasco ivan-velasco self-assigned this Dec 31, 2025
@ivan-velasco ivan-velasco merged commit e7f413a into internal/develop Dec 31, 2025
1 check passed
@ivan-velasco ivan-velasco deleted the feature/metrics branch December 31, 2025 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aditi-malik aditi-malik Awaiting requested review from aditi-malik

Assignees

@ivan-velasco ivan-velasco

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ivan-velasco