Skip to content

Rerun prod task cleanup, stop using keys, support task: "all".#4328

Merged
auto-submit[bot] merged 4 commits intoflutter:mainfrom
matanlurey:rerun-prod-task-cleanup
Mar 12, 2025
Merged

Rerun prod task cleanup, stop using keys, support task: "all".#4328
auto-submit[bot] merged 4 commits intoflutter:mainfrom
matanlurey:rerun-prod-task-cleanup

Conversation

@matanlurey
Copy link
Contributor

Closes flutter/flutter#164032.
Closes flutter/flutter#165058.

Summary of changes:

  • Rename /api/reset-prod-task to /api/rerun-prod-task for naming clarity (including on the frontend)
  • Deleted duplicate (/api/v2/reset-prod-task)
  • Now uses a composite key of task/commit/repo/branch1
  • The frontend is no longer reliant on a commitKey (easier to migrate from datastore to firestore)
  • Uses the same functionality (with task: all) to support the "run all tasks for the given commit" (for release branches)

Footnotes

  1. Maybe we could get just a get away with task+commit? Wdut @jtmcdole?

@matanlurey matanlurey requested a review from jtmcdole March 12, 2025 16:31
Comment on lines +69 to +74
final {
_paramBranch: String branch,
_paramRepo: String repo,
_paramCommitSha: String commitSha,
_paramTaskName: String taskName,
} = requestData!;
Copy link
Member

Choose a reason for hiding this comment

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

me: takes notes.

@matanlurey
Copy link
Contributor Author

Good luck (to me).

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App. label Mar 12, 2025
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App. label Mar 12, 2025
@auto-submit
Copy link
Contributor

auto-submit bot commented Mar 12, 2025

autosubmit label was removed for flutter/cocoon/4328, because Pull request flutter/cocoon/4328 is not in a mergeable state.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App. label Mar 12, 2025
@auto-submit auto-submit bot merged commit 42f53a3 into flutter:main Mar 12, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change calls to reset-prod-task to no longer use taskKey Update Cocoon to receive an RPC to schedule post-submits on a release branch

2 participants