Skip to content

Conversation

pipiland2612
Copy link
Owner

No description provided.

Signed-off-by: pipiland2612 <[email protected]>
Signed-off-by: pipiland2612 <[email protected]>
Signed-off-by: pipiland2612 <[email protected]>
Copy link

Metrics Comparison Summary

summary_metrics_snapshot_elasticsearch

📊 Metrics Diff Summary

Total Changes: 3

  • 🆕 Added: 0 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 3 metrics

🔄 Modified Metrics

  • jaeger_storage_latency_seconds
    • Added variants: 90
    • Removed variants: 72
  • jaeger_storage_responses_total
    • Added variants: 5
    • Removed variants: 4
  • jaeger_storage_requests_total
    • Added variants: 5
    • Removed variants: 4

summary_metrics_snapshot_elasticsearch

📊 Metrics Diff Summary

Total Changes: 3

  • 🆕 Added: 0 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 3 metrics

🔄 Modified Metrics

  • jaeger_storage_latency_seconds
    • Added variants: 90
    • Removed variants: 72
  • jaeger_storage_responses_total
    • Added variants: 5
    • Removed variants: 4
  • jaeger_storage_requests_total
    • Added variants: 5
    • Removed variants: 4

➡️ View full metrics file

yurishkuro pushed a commit to jaegertracing/jaeger that referenced this pull request Aug 15, 2025
## Which problem is this PR solving?
- Resolves part of #6278
- Supports #7414

## Description of the changes
- Add upload pr_number artifacts action
- The ci-comment.yml workflow can now detect pr number by downloading
the artifact

## How was this change tested?
- from my repostory pull request:
pipiland2612#2
- log from the action
https://github.com/pipiland2612/jaeger/actions/runs/16997159461/job/48190513998:

```
Successfully extracted metrics_snapshot_elasticsearch_8.x_v2
Processing artifact: diff_metrics_snapshot_elasticsearch_8.x_v2 (ID: 3776313824)
Extracting to /home/runner/work/jaeger/jaeger/.metrics/diff_metrics_snapshot_elasticsearch_8.x_v2
Successfully extracted diff_metrics_snapshot_elasticsearch_8.x_v2

Starting PR number detection...
Workflow run details:
- Head branch: error_branch
- Head repository: pipiland2612/jaeger
- Commit SHA: 49af2ee

Attempt 1: Searching PRs for branch pipiland2612/jaeger:error_branch
Found PR Number from artifact: 2

Final PR Number: 2
```

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [ ] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

Signed-off-by: pipiland2612 <[email protected]>
@pipiland2612 pipiland2612 force-pushed the main branch 5 times, most recently from 935f19c to faef1a9 Compare August 17, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant