Skip to content

Add POST /api/schedule-postsubmits-for-commit#4286

Closed
matanlurey wants to merge 1 commit intoflutter:mainfrom
matanlurey:add-commit-sha-postsubmit
Closed

Add POST /api/schedule-postsubmits-for-commit#4286
matanlurey wants to merge 1 commit intoflutter:mainfrom
matanlurey:add-commit-sha-postsubmit

Conversation

@matanlurey
Copy link
Contributor

@matanlurey matanlurey commented Mar 4, 2025

Closes flutter/flutter#164032.

This needs more work that I want to pair with @jtmcdole on, namely:

  1. Would addPullRequest be where we skip (merged) PRs on a release branch?
    Yes.
  2. If we skip, doesn't that mean flutter-dashboard would be missing the commit, and there would not be a commit to click?
    Yes. We need to add the commit to datastore with no tasks..
  3. Does skipping mean, "add the commit, but schedule no builds.
    Yes.
  4. If (3), then this would fail, as it expects the commit to not already exist. What should we do instead?
    We need to make this function re-entrant, that starts with a 0-task commit.

@matanlurey
Copy link
Contributor Author

Superseded (mostly) by #4328.

Tracking additional work: flutter/flutter#163896.

@matanlurey matanlurey closed this Mar 12, 2025
@matanlurey matanlurey deleted the add-commit-sha-postsubmit branch March 12, 2025 19:46
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.

Update Cocoon to receive an RPC to schedule post-submits on a release branch

1 participant