-
Notifications
You must be signed in to change notification settings - Fork 6k
[CI] Some improvements to Nightly reports summaries #11166
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good initiative. Can we see an example message on Slack?
Also, the benefit of the previous action was it pointed to the specific action run of which the failing tests are a part of. Are we doing that in this PR? If not, I emphasize on including that part.
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
@sayakpaul Latest CI Status message in Slack Think this PR should be good to merge now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better. Thanks a lot for iterating!
What does this PR do?
We currently create summary reports for each test module, but the number of pipelines we test in the nightlys has grown considerably. Scrolling through them all is starting to get challenging. This PR introduces an additional step that consolidates the individual reports into a single report with some useful summary information.
A shorter summary report is also sent to the diffusers-ci Slack Channel with a link to the full report in Github Actions.
Example report below
Fixes # (issue)
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.