Skip to content

[ci] Disable status caching for manager-hosted jobs#922

Merged
davidbiancolin merged 2 commits intodevfrom
no-status-caching
Feb 14, 2022
Merged

[ci] Disable status caching for manager-hosted jobs#922
davidbiancolin merged 2 commits intodevfrom
no-status-caching

Conversation

@davidbiancolin
Copy link
Contributor

@davidbiancolin davidbiancolin commented Feb 7, 2022

This allows a fresh manager to be spun up when rerunning the test workflow. Previously you'd have to create new commit (which would invalidate the cache).

Related PRs / Issues

UI / API Impact

Verilog / AGFI Compatibility

Contributor Checklist

  • Did you set dev as the base branch?
  • Did you add Scaladoc to every public function/method?
  • Did you add at least one test demonstrating the PR?
  • Did you delete any extraneous prints/debugging code?
  • Did you state the UI / API impact?
  • Did you specify the Verilog / AGFI compatibility impact?
  • (If applicable) Did you regenerate and publicly share default AGFIs?
  • (If applicable) Did you mark the PR as "Please Backport"?
  • (On merge) Did you update release notes in the dev-to-master PR ?

Reviewer Checklist (only modified by reviewer)

  • Did you mark the proper release milestone?
  • Did you check whether all relevant Contributor checkboxes have been checked?

Copy link
Contributor

@abejgonzalez abejgonzalez left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Abraham Gonzalez <abe.j.gonza@gmail.com>
@davidbiancolin davidbiancolin merged commit 1de637c into dev Feb 14, 2022
@davidbiancolin davidbiancolin deleted the no-status-caching branch February 14, 2022 23:31
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