Post JSON summary to report processor #1719
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds automated reporting of workflow execution summaries by POSTing the machine-readable JSON summary file to a workflow analysis processor endpoint. This enables AI/LLM-based analysis to automatically identify root cause failures in WARP test workflows.
Key Changes:
- Added a new step to POST workflow summary JSON to an external API endpoint
- Configured the API call with authentication using GitHub secrets
- Updated step numbering in comments to accommodate the new step
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
fboulnois
force-pushed
the
fb-post-workflow-json
branch
from
40faadf to
5433677
Compare
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Addresses
This is a follow up to #1703 . This POSTs the JSON version of the workflow summary file if it exists to the report processor.
Motivation
@kilonzi and the team have been working on using AI / LLMs to automatically generate reports that identify root cause failures of Warp test workflows. As part of this work, we need to send the JSON version of the markdown summary file that is generated at the end of a workflow run to the workflow analysis processor to generate the final report.
Checklist
If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP team by tagging @broadinstitute/warp-admins in a comment on this PR.