Azure Pipeline run queued and not restarted about wpt

[TalbotG]

It seems that the failing commit b119144 (with 10 canceled checks) in #42345 was successfully squashed and merged (or registered or embedded) .

So, I am closing this issue now.

Thank you @foolip and @whimboo for your assistance. This is appreciated.

[whimboo]

@TalbotG I don't know what the problem is. But what we usually do is to close and reopen a PR to re-trigger all the checks. That's the first thing that I would suggest to do. If that doesn't help maybe someone else has more ideas.

[foolip]

I see that the checks are passing on the PR now.

In https://github.com/web-platform-tests/wpt/runs/25140134004 (for commit b119144) I think the problem was actually that the Taskcluster tasks were cancelled for some reason.

Closing this as the problem did not persist.

[TalbotG]

I see that the checks are passing on the PR now.

@foolip Philip,

Where exactly do you see that the checks are passing or have passed?

When I visit b119144 I see:

"
Some checks were not successful
10 cancelled, 1 queued, and 3 successful checks
"

and when I visit

https://github.com/web-platform-tests/wpt/runs/25140134004

I see a red circle with some green at top-left corner

The commit is preceded by a red "x" in the #42345 page

[Addendum]
I see Azure Pipelines / Azure Pipelines
started 2d 15h m and s ago
!

It does not perform or execute.
[/Addendum]

[TalbotG]

what we usually do is to close and reopen a PR to re-trigger all the checks

@whimboo Henrik,
Please visit #45943 and support item 2 because what you suggest is not in documentation and, in my opinion, it should.

[foolip]

@TalbotG if I scroll to the end of #42345 I see "All checks have passed". That's because the tip of the PR branch is now commit 48b9f4a, any failures previously don't block the PR.

[TalbotG]

if I scroll to the end of #42345 I see "All checks have passed".

I do see "All checks have passed" but that is for the last commit, which is commit 48b9f4a .

When I visit again b119144 I still see:

"
Some checks were not successful
10 cancelled, 1 queued, and 3 successful checks
"

When I visit again

https://github.com/web-platform-tests/wpt/runs/25140134004

I still see a red circle with some green at top-left corner.

any failures previously don't block the PR.

But what about commit b119144 ? It will not register. It will not modify the tests. Submiting the PR will not register, will not embed commit b119144

There are 18 commits in PR42345. Only 17 commits will modify the tests. The commit b119144 will not modify the tests because of what I quoted up here.

[TalbotG]

The "Re-run all checks" button does not work for a particular, specific commit. It only works for the last commit.

[TalbotG]

What does "This run will be cleaned up after 10 days based on your project settings." mean?
I see this in
https://dev.azure.com/web-platform-tests/wpt/_build/results?buildId=123694&view=results

[TalbotG]

At the bottom of the page
b119144?diff=split&w=0
I see 10 comments of failures: "Uh oh! Looks like an error! Details Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes: (...)" which must be related to my attemps to re-run the checks for commit b119144

[whimboo]

taskcluster/github does not have sufficient scopes and is missing the following scopes: (...)" which must be related to my attemps to re-run the checks for commit b119144

Yes, that is correct. As I stated earlier you have to close and reopen the PR to trigger the checks again.

[TalbotG]

Henrik,

I closed and reopened Pull Request 42345 a few min ago. The 19 Continuous Integration checks started, ran (but I did not see which commit it was running on) and when I visit again b119144 , I still see:
"
Some checks were not successful
10 cancelled, 1 queued, and 3 successful checks
"
and still see a red circle with some green at top-left corner in
https://github.com/web-platform-tests/wpt/runs/25140134004

[foolip]

@TalbotG the PR is once again passing the checks. If you get it reviewed and still can't merge it, let me know and I can check why CI is failing.

[TalbotG]

Philip,

I am mostly concerned about the changes, the modifications in commit b119144 if I review and approve the PR. I want the changes and modifications (removing the margin-top as the label given to the commit states) to be registered, to be carried over (I can not find the correct verb here) when I approve and merge the #42345.

Anyway... I will try to approve and to merge the #42345 tomorrow.

[foolip]

@TalbotG if the checks are passing on the pull request and you squash all the commits when you merge, then none of the original commits will really matter. The CI failures that were on the branch also don't get in the way of merging as long as they're fixed in subsequent commits.

[TalbotG]

as long as they're fixed in subsequent commits

I do not know for sure if they were fixed in subsequent commits. In my opinion, they were not.

I just approve the whole #42345 a few min. ago. [fingers crossed]

I will close this Issue as soon as I know for sure that the failing commit b119144 (with 10 canceled checks) in #42345 was successfully squashed and merged (or registered or embedded) .

[whimboo]

Sounds good. Thanks for letting us know!