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

[Do not merge] Run try on multiple past commits without merging them to current master #25351

Closed
wants to merge 2 commits into from

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Dec 20, 2019

No description provided.

@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

Trying commit 8c921e5 with merge 2e1d224...

bors-servo added a commit that referenced this pull request Dec 20, 2019
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the try-without-merge branch from 8c921e5 to 603d8f3 Dec 20, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

Trying commit 603d8f3 with merge f222bc4...

bors-servo added a commit that referenced this pull request Dec 20, 2019
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the try-without-merge branch from 603d8f3 to 76d4a3a Dec 20, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

Trying commit 76d4a3a with merge 507a4d3...

bors-servo added a commit that referenced this pull request Dec 20, 2019
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the try-without-merge branch from 76d4a3a to d2fb9c1 Dec 20, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

Trying commit d2fb9c1 with merge b9b5080...

bors-servo added a commit that referenced this pull request Dec 20, 2019
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the try-without-merge branch from d2fb9c1 to 794528f Dec 20, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

Trying commit 794528f with merge 326a188...

bors-servo added a commit that referenced this pull request Dec 20, 2019
[Do not merge] Run try on multiple past commits without merging them to current master
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 20, 2019

Alright! It looks like https://community-tc.services.mozilla.com/tasks/groups/CvCvx9NHRESFPokJbQkQag has started as intended.

In this PR the decision task schedules testing not for that same PR but for two older commits (that I picked at semi-random) whose SHAs are near the top of etc/taskcluster/decision_task.py, without merging them with master. This can hopefully help bisect the regression that causes #24726.

Builds are indexed/cached as usual (for a week), but indexing of WPT tasks is disabled.

@jdm you should be able to take this branch, change the set of commits to test, then either:

Another thing you can tweak is total_chunks in the linux_wpt function: more chunks gives results sooner but more log files to look at.

@bors-servo
Copy link
Contributor

bors-servo commented Dec 20, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Dec 26, 2019

The following script has made my life significantly better when I want to retrigger a set of tasks a large number of times:

export TASKCLUSTER_ROOT_URL=https://community-tc.services.mozilla.com/
eval $(taskcluster signin)
times=$1
shift
for task in $@
do
    for (( i = 1 ; i < $times; i++ ))
    do
        taskcluster task retrigger $task
    done
done
@jdm
Copy link
Member

jdm commented Jan 14, 2020

@SimonSapin Let's say I have pushed branches testA and testB to the servo/servo remote. Will using commits from those branches in the commits list of the decision task work? If not, what else needs modifying?

@SimonSapin
Copy link
Member Author

SimonSapin commented Jan 14, 2020

I’ve added a commit that makes sub-tasks fetch all branches from github.com/servo/servo, which should allow that.

@bors-servo try=wpt

bors-servo added a commit that referenced this pull request Jan 14, 2020
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2020

Trying commit 06e0895 with merge a633cc6...

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2020

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the try-without-merge branch from 06e0895 to 7bf1a5d Jan 14, 2020
@SimonSapin
Copy link
Member Author

SimonSapin commented Jan 14, 2020

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2020

Trying commit 7bf1a5d with merge a59a85c...

bors-servo added a commit that referenced this pull request Jan 14, 2020
[Do not merge] Run try on multiple past commits without merging them to current master
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2020

💔 Test failed - status-taskcluster

@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2020

The latest upstream changes (presumably #25992) made this pull request unmergeable. Please resolve the merge conflicts.

@SimonSapin
Copy link
Member Author

SimonSapin commented Apr 8, 2020

I believe this has served its purpose.

@SimonSapin SimonSapin closed this Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.