Skip to content

Enhancement: Update configuration and improve error handling in netwo…#1180

Merged
Teingi merged 8 commits intooceanbase:masterfrom
wayyoungboy:410-fix-check
Feb 5, 2026
Merged

Enhancement: Update configuration and improve error handling in netwo…#1180
Teingi merged 8 commits intooceanbase:masterfrom
wayyoungboy:410-fix-check

Conversation

@wayyoungboy
Copy link
Member

…rk tasks

  • Added 'max_workers' configuration to inner_config.yml and config.py to optimize task execution.
  • Improved error handling in TCP retransmission and instruction set tasks by replacing error messages with appropriate fail and warning reports.
  • Enhanced SSH connection management in check handler and task base to ensure thread safety and prevent connection contention.
  • Updated report generation logic to conditionally include observer and obproxy version information based on the 'build_before' case.

…rk tasks

- Added 'max_workers' configuration to inner_config.yml and config.py to optimize task execution.
- Improved error handling in TCP retransmission and instruction set tasks by replacing error messages with appropriate fail and warning reports.
- Enhanced SSH connection management in check handler and task base to ensure thread safety and prevent connection contention.
- Updated report generation logic to conditionally include observer and obproxy version information based on the 'build_before' case.
- Changed error message logging from 'add_fail' to 'error' for better clarity.
- Added a fail report for parsing errors in TCP retransmission values to enhance reporting accuracy.
-Remove excess blank lines and spaces
-Unified code indentation format
-Optimize long string wrapping
Copy link
Member

@Teingi Teingi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Teingi Teingi merged commit 480bbdf into oceanbase:master Feb 5, 2026
15 checks passed
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.

2 participants