Skip to content

Adding Additional Failure Patterns for Alerts #3663

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.

Sign up for GitHub

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

Open
amd-anclark wants to merge 5 commits into
base: develop
Choose a base branch
from
+5 −3
Open

Adding Additional Failure Patterns for Alerts #3663

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
4b6aa60
Added two new failure patterns to detect. Including test function to …
amd-anclark Jan 27, 2026
9f5a38b
Modifying pattern match to detect docker login failure. Removed passi…
amd-anclark Jan 27, 2026
f82f516
Removing passing tests. Modifying docker pattern to detect failure
amd-anclark Jan 27, 2026
d251ea0
Removed passing tests
amd-anclark Jan 27, 2026
8f79178
Removing test logging function
amd-anclark Jan 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions Jenkinsfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,13 @@ def sendFailureNotifications() {
// Error patterns to scan build logs for specific failure types and send detailed notifications.
def failurePatterns = [
[pattern: /login attempt to .* failed with status: 401 Unauthorized/, description: "Docker registry authentication failed"],
[pattern: /(.*)docker login failed(.*)/, description: "Docker login failed"],
[pattern: /.*docker login failed.*/, description: "Docker login failed"],
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

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

This pattern is too generic and will match the more specific Docker authentication patterns on lines 41 and 47. Consider either removing this pattern since it's now redundant, or making it more specific to catch different Docker login failure scenarios not covered by the other patterns.

Suggested change
[pattern: /.*docker login failed.*/, description: "Docker login failed"],
[pattern: /^.*docker login failed(?!.*(status: 401 Unauthorized|unauthorized: your account must log in with a Personal Access Token)).*$/, description: "Docker login failed"],

Copilot uses AI. Check for mistakes.
[pattern: /HTTP request sent .* 404 Not Found/, description: "HTTP request failed with 404"],
[pattern: /cat: .* No such file or directory/, description: "GPU not found"],
[pattern: /(.*)GPU not found(.*)/, description: "GPU not found"],
[pattern: /Could not connect to Redis at .* Connection timed out/, description: "Redis connection timed out"]
[pattern: /.*GPU not found.*/, description: "GPU not found"],
[pattern: /Could not connect to Redis at .* Connection timed out/, description: "Redis connection timed out"],
[pattern: /.*unauthorized: your account must log in with a Personal Access Token.*/, description: "Docker login failed"],
[pattern: /.*sccache: error: Server startup failed: Address in use.*/, description: "Sccache Error"]
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

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

The description 'Sccache Error' is inconsistent with the capitalization and naming style of other error descriptions in this list. Consider renaming to 'sccache server startup failed' to match the pattern of descriptive, lowercase error messages used elsewhere (e.g., 'Docker registry authentication failed', 'Redis connection timed out').

Suggested change
[pattern: /.*sccache: error: Server startup failed: Address in use.*/, description: "Sccache Error"]
[pattern: /.*sccache: error: Server startup failed: Address in use.*/, description: "sccache server startup failed"]

Copilot uses AI. Check for mistakes.
]

// Get the build log.
Expand Down