Skip to content
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

fix: the build process may timeout when the message is too long #1643

Merged
merged 3 commits into from
Oct 11, 2024

Conversation

piglei
Copy link
Collaborator

@piglei piglei commented Oct 10, 2024

fix: the build process may timeout when the message is too long, changes include:

  1. truncate the log message if it's too long (>4k) and only match the first 1k characters when doing step pattern matching
  2. improve the step pattern by replacing "\s+" with "\s", the latter is much faster when searching a string with many blank characters
  3. reduce the number of patterns by introducing re.IGNORECASE.

@piglei piglei force-pushed the fix-build-timeout-huge-msg branch from 680cf9c to 861a36e Compare October 11, 2024 07:26
@piglei piglei merged commit 03f6d82 into TencentBlueKing:main Oct 11, 2024
7 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