Navigation Menu

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.

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

GitHub Actions stalled in the "Waiting to run" state #18286

Closed
jugglinmike opened this issue Aug 5, 2019 · 3 comments
Closed

GitHub Actions stalled in the "Waiting to run" state #18286

jugglinmike opened this issue Aug 5, 2019 · 3 comments

Comments

@jugglinmike
Copy link
Contributor

We landed gh-17680 under the assumption (and limited observation) that GitHub Actions run consistently for all of GitHub's pull_request events over the lifetime of a pull request. Reviewing recent performance of that Action shows that this is not the case. That's a problem because consistency is important for the application we have in mind (submission preview).

I've created a ticket with GitHub support and included the text below. I'll update this issue as I learn more.

/cc @jgraham @sideshowbarker

Hello,

My project [1] recently added a GitHub Workflow for the pull_request event. The Workflow's only Action frequently stalls in the "Waiting to run" state. This is not consistent--the Action does occasionally run as expected for pull request events.

I don't know a way to analyze this at scale, but my limited research suggests that this may be related to the state of the branch. Specifically, this seems to occur when a commit is force-pushed and when the branch is deleted.

Is this a fundamental limitation of the GitHub Actions feature? If so, can the status be updated to indicate the finality of this state? If not, can you speak to the likelihood of the problem being solved?

Thanks for your help!

[1] https://github.com/web-platform-tests/wpt

@jugglinmike jugglinmike self-assigned this Aug 5, 2019
@jugglinmike
Copy link
Contributor Author

Still no response from GitHub, but I have been able to learn a bit more.

This week, GitHub announced a new Continuous Integration service built on the Actions feature. I've been experimenting with that a bit, and the improved UI gives more insight as to what's going wrong (or at least, what will go wrong when we migrate to the new system).

It looks like the built-in "checkout" action is attempting to fetch the GitHub-created git ref at refs/pull/XX/merge, even when configured to fetch master. That seems like a mistake to me, but this functionality is all so new that it's tough to know what's intended. I've filed an issue with the maintainers to find out.

jugglinmike added a commit to bocoup/wpt that referenced this issue Aug 12, 2019
Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests#18286
[2] web-platform-tests#18359
@jugglinmike
Copy link
Contributor Author

The maintainers of that Action got back to me very quickly, and it looks like this was just a misunderstanding about the documentation. Following some experimentation on the demo repository, it's looking like this is just a case of incorrect configuration. gh-18393 should resolve the problem.

@jugglinmike
Copy link
Contributor Author

As I filed the pull request above, GitHub's support staff replied to my request
for help:

GitHub Support (GitHub Developer Support)

Aug 12, 6:44 PM UTC

Hi Mike,

Thanks for reaching out! I see that you recently updated your repository's workflows to the new YAML syntax:

https://github.com/web-platform-tests/wpt/tree/master/.github/workflows

If you're still running into this "Waiting to run" state for these workflow executions, could you please share a link with us so we can take a look and provide input? Thanks

Here's my response:

Thanks for getting back to me!

The new UI correctly communicates the final (failed) state of the Workflow, so you can consider this issue resolved.

In case you're curious, the associated log also helped us verify that it was indeed a problem with delete branches. Better still, the introduction of a dedicated "checkout" Action means that we can configure the system to work as we intended:

#18393

Thanks again,

Mike

@foolip
Copy link
Member

foolip commented Aug 19, 2019

@jugglinmike closing per your "you can consider this issue resolved" in that reply.

@foolip foolip closed this as completed Aug 19, 2019
jugglinmike added a commit that referenced this issue Aug 20, 2019
* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] #18286
[2] #18359

* fixup! [ci] Check out master when managing tags & labels
natechapin pushed a commit to natechapin/wpt that referenced this issue Aug 23, 2019
…#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests#18286
[2] web-platform-tests#18359

* fixup! [ci] Check out master when managing tags & labels
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Aug 27, 2019
… & labels, a=testonly

Automatic update from web-platform-tests
[ci] Check out master when managing tags & labels (#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests/wpt#18286
[2] web-platform-tests/wpt#18359

* fixup! [ci] Check out master when managing tags & labels

--

wpt-commits: 6e9bf9411482aa2d931c005ad66750414b6a5fee
wpt-pr: 18393
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Aug 27, 2019
… & labels, a=testonly

Automatic update from web-platform-tests
[ci] Check out master when managing tags & labels (#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests/wpt#18286
[2] web-platform-tests/wpt#18359

* fixup! [ci] Check out master when managing tags & labels

--

wpt-commits: 6e9bf9411482aa2d931c005ad66750414b6a5fee
wpt-pr: 18393
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 4, 2019
… & labels, a=testonly

Automatic update from web-platform-tests
[ci] Check out master when managing tags & labels (#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests/wpt#18286
[2] web-platform-tests/wpt#18359

* fixup! [ci] Check out master when managing tags & labels

--

wpt-commits: 6e9bf9411482aa2d931c005ad66750414b6a5fee
wpt-pr: 18393

UltraBlame original commit: c883a9dca1672326ff0b5e8cdc47ae41b86fe535
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 4, 2019
… & labels, a=testonly

Automatic update from web-platform-tests
[ci] Check out master when managing tags & labels (#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests/wpt#18286
[2] web-platform-tests/wpt#18359

* fixup! [ci] Check out master when managing tags & labels

--

wpt-commits: 6e9bf9411482aa2d931c005ad66750414b6a5fee
wpt-pr: 18393

UltraBlame original commit: c883a9dca1672326ff0b5e8cdc47ae41b86fe535
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 4, 2019
… & labels, a=testonly

Automatic update from web-platform-tests
[ci] Check out master when managing tags & labels (#18393)

* [ci] Check out master when managing tags & labels

Previously, we identified a problem when attempting to run a GitHub
Action for the "pull_request" event when the underlying branch had been
deleted [1]. By migrating to the latest GitHub Action beta release [2],
we've since gained more insight into the process and more control over
its behavior.

Alter the configuration for the new "checkout" Step to consistently
check out the `master` branch. Update the Action specifier to reference
a stable release of the Action.

[1] web-platform-tests/wpt#18286
[2] web-platform-tests/wpt#18359

* fixup! [ci] Check out master when managing tags & labels

--

wpt-commits: 6e9bf9411482aa2d931c005ad66750414b6a5fee
wpt-pr: 18393

UltraBlame original commit: c883a9dca1672326ff0b5e8cdc47ae41b86fe535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants