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.

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

Jump to bottom

#0: fix blachole scheduled post-commit #16995

Merged
merged 1 commit into from
Jan 22, 2025
Merged

#0: fix blachole scheduled post-commit #16995

merged 1 commit into from
Jan 22, 2025

Conversation

ttmchiou
Copy link
Contributor

@ttmchiou ttmchiou commented Jan 22, 2025 •
edited
Loading

Ticket

Link to Github Issue
Fix broken blackhole post-commit scheduled call.
Previous PR did not include a default value for scheduled blackhole post-commit runs which use workflow_call, only workflow_dispatch

Checklist

  • Post commit CI passes
  • Blackhole Post commit (if applicable)
  • Model regression CI testing passes (if applicable)
  • Device performance regression CI testing passes (if applicable)
  • (For models and ops writers) Full new models tests passes
  • New/Existing tests provide coverage for changes

@ttmchiou ttmchiou requested a review from a team as a code owner January 22, 2025 22:39
@ttmchiou ttmchiou merged commit 03260e0 into main Jan 22, 2025
19 checks passed
@ttmchiou ttmchiou deleted the mchiou/0-blackhole-fix branch January 22, 2025 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tt-rkim tt-rkim tt-rkim approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ttmchiou @tt-rkim