ANALYTICS: different query names for brach queries #28422
Conversation
|
🟢 |
|
⚪
🟢 |
|
⚪
🟢 |
There was a problem hiding this comment.
Pull Request Overview
This PR enhances query tracking by incorporating branch names into query identifiers across multiple analytics and testing scripts. This improvement enables better differentiation and monitoring of queries executed for different branches in the YDB statistics system.
- Query names are updated to include branch identifiers for improved tracking
- Changes ensure queries for different branches (main, stable-*, etc.) are distinguishable in statistics
- Proper handling of None branch values in conditional query naming
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/scripts/tests/get_muted_tests.py | Updated query names to include branch suffix for test retrieval queries, with fallback handling for None branch |
| .github/scripts/tests/create_new_muted_ya.py | Added branch suffix to monitor data query name for better query tracking |
| .github/scripts/analytics/tests_monitor.py | Updated four query names (max monitor date, branch creation, monitor data, history) to include branch identifiers |
| .github/scripts/analytics/flaky_tests_history.py | Added branch suffix to query names for last date and flaky test history queries |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 participants
Changelog entry
...
Changelog category
Description for reviewers
...