Skip to content

Show actual step name in Error Logs Preview instead of generic category#92

Closed
mohit-sheth wants to merge 0 commit intoredhat-performance:mainfrom
mohit-sheth:main
Closed

Show actual step name in Error Logs Preview instead of generic category#92
mohit-sheth wants to merge 0 commit intoredhat-performance:mainfrom
mohit-sheth:main

Conversation

@mohit-sheth
Copy link
Collaborator

@mohit-sheth mohit-sheth commented Feb 21, 2026

Use the real step name (e.g. orion-udn-l3) in the preview header instead of a generic description (e.g. orion changepoint detection failure), since many different steps share the same keyword.

Summary

  • For orion/changepoint failures, display a clickable link to the changepoint visualization HTML in GCS instead of the generic "Error
    Logs Preview"
  • The visualization URL is constructed by probing the job's GCS artifacts directory for the matching step subfolder, handling junit
    step-name prefixes and linking directly to the HTML file when present
  • Threads step_name through analyze_prow_artifacts return tuple so callers can resolve the correct artifact path
  • Extracts duplicated failure_desc logic into a shared _get_failure_desc() helper in SlackMessageFetcher

@vishnuchalla
Copy link
Collaborator

Pre-requisite to consider before merging this: #93

@vishnuchalla
Copy link
Collaborator

Somehow I cannot see the file diff on this PR. But yeah, for change points we never had a Error Logs Preview. It was a bug that is fixed in the above PR.

@mohit-sheth
Copy link
Collaborator Author

closing as this was main branch 😓

@mohit-sheth
Copy link
Collaborator Author

closing in favour of #94

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