adoptium / aqa-test-tools

Home of Test Results Summary Service (TRSS) and PerfNext. These tools are designed to improve our ability to monitor and triage tests at the Adoptium project. The code is generic enough that it is extensible for use by any project that needs to monitor multiple CI servers and aggregate their results.
Apache License 2.0
28 stars 79 forks source link

Return original _id in getTotals request #851

Closed jiekang closed 6 months ago

jiekang commented 6 months ago

Fixes https://github.com/adoptium/aqa-test-tools/issues/850

netlify[bot] commented 6 months ago

Deploy Preview for eclipsefdn-adoptium-trss ready!

Name Link
Latest commit 53fcc787e0478ed43dbaf6fde196a677a0b31860
Latest deploy log https://app.netlify.com/sites/eclipsefdn-adoptium-trss/deploys/65f9d7a24ee8430008c6c633
Deploy Preview https://deploy-preview-851--eclipsefdn-adoptium-trss.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

llxia commented 6 months ago
smlambert commented 6 months ago

@llxia - feel free to revert - we can discuss best plan for metrics gathering in the issue and then re-approach

llxia commented 6 months ago

Thanks @smlambert . I will create a separate PR to remove line 162, preventing any potential confusion down the road. Sure, let's discuss metrics gathering approaches.