-
Notifications
You must be signed in to change notification settings - Fork 3
feat: Add GITHUB_RUN_ATTEMPT to test runner and workflows for improved run tracking #148
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
Conversation
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.
Pull request overview
This PR adds the GITHUB_RUN_ATTEMPT variable to test runners and workflows to improve tracking of workflow re-runs. The change allows distinguishing between different attempts of the same workflow run, which is useful for debugging and log management.
Changes:
- Added GITHUB_RUN_ATTEMPT environment variable to workflow files
- Updated test runner script to incorporate run attempt in log directory naming
- Modified test execution commands to include run_attempt in build identifiers and paths
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .github/workflows/run-tests-on-linux.yml | Added GITHUB_RUN_ATTEMPT to environment variables and updated test execution paths to include run attempt |
| .github/workflows/new-framework-test.yml | Added GITHUB_RUN_ATTEMPT to prepare_test_env.py environment variables |
| .github/scripts/test_runner.py | Added run_attempt variable extraction, created timestamped log directory names, and updated all test commands to use run_attempt |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…up for enhanced tracking
https://project.feishu.cn/taosdata_td/job/detail/6662885983 增加 mac 的 日志报错
https://project.feishu.cn/taosdata_td/job/detail/6726400782 其余 ci 流程优化