-
Notifications
You must be signed in to change notification settings - Fork 283
Add Configuration for analyzing dump and format console log for RF #4931
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
base: main
Are you sure you want to change the base?
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 refactors reliability framework commands to improve organization and adds enhanced console logging and dump analysis functionality. The refactoring consolidates similar functionality while adding better user feedback and error handling.
- Renamed and reorganized ReliabilityFramework commands to use "RF" prefix for consistency
- Enhanced console output with formatted messages and better error reporting for dump analysis
- Added configuration files and improved file handling with proper error checking
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
Program.cs | Updated command registrations to use new RF-prefixed command names |
RFCreateSuitesCommand.cs | New consolidated command replacing the old create test suite functionality |
RFAnalyzeCommand.cs | Refactored analyze command with enhanced console output and error handling |
RFAggregateCommand.cs | Refactored aggregate command with improved console formatting |
Configuration files | New configuration classes and YAML files for RF commands |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...c/GC.Infrastructure/GC.Infrastructure/Commands/ReliabilityFramework/RFCreateSuitesCommand.cs
Show resolved
Hide resolved
...s/gc/GC.Infrastructure/GC.Infrastructure.Core/Configurations/RFCreateSuites.Configuration.cs
Outdated
Show resolved
Hide resolved
...rks/gc/GC.Infrastructure/GC.Infrastructure/Commands/ReliabilityFramework/RFAnalyzeCommand.cs
Show resolved
Hide resolved
…figurations/RFCreateSuites.Configuration.cs correct the typo Co-authored-by: Copilot <[email protected]>
Create Test Suites for RF Console Log:
────────────────── Create Test Suites For ReliabilityFramework Test ────────────────
Suite Created: loh
├── Datas at D:\ReliabilityFramework\TestSuites\win-x64\loh\Datas
├── Server at D:\ReliabilityFramework\TestSuites\win-x64\loh\Server
└── Workstation at D:\ReliabilityFramework\TestSuites\win-x64\loh\Workstation
Suite Created: poh
├── Datas at D:\ReliabilityFramework\TestSuites\win-x64\poh\Datas
├── Server at D:\ReliabilityFramework\TestSuites\win-x64\poh\Server
└── Workstation at D:\ReliabilityFramework\TestSuites\win-x64\poh\Workstation
Suite Created: non_induced
├── Datas at D:\ReliabilityFramework\TestSuites\win-x64\non_induced\Datas
├── Server at D:\ReliabilityFramework\TestSuites\win-x64\non_induced\Server
└── Workstation at D:\ReliabilityFramework\TestSuites\win-x64\non_induced\Workstation
Suite Created: finalization
├── Datas at D:\ReliabilityFramework\TestSuites\win-x64\finalization\Datas
├── Server at D:\ReliabilityFramework\TestSuites\win-x64\finalization\Server
└── Workstation at D:\ReliabilityFramework\TestSuites\win-x64\finalization\Workstation
Display Analyze Dump Result:
────────────────── Analyze Dumps Collected In Reliability Framework Test ──────────────────
Successfully analyzed dump D:\RF_output\Dumps\stress_0.dmp
Callstack output: D:\RF_output\AnalyzeDump\stress_0_callstack.txt
Callstack for all threads output: D:\RF_output\AnalyzeDump\stress_0_callstack_allthreads.txt
fix some coding format addressed in PR#4803 & 4848