understanding Auto-Test

Seb seb at wilzba.ch
Thu Mar 22 04:25:00 UTC 2018


On Thursday, 22 March 2018 at 04:12:00 UTC, John Belmonte wrote:
> On Thursday, 22 March 2018 at 01:51:49 UTC, Mike Franklin wrote:
>> I believe what happened is a different PR was merged.  When a 
>> PR is merged, all tests are invalidated, and the autotester 
>> begins testing them again.
>>
>> There is also a priority affecting which PRs get tested first.
>>  Those that are labeled with "AutoMerge" are given a higher 
>> priority.
>
> I'm still rather puzzled.  My pull request remains with 8 tests 
> pending after several hours.  I can't find any confirmation on 
> the pulls display 
> https://auto-tester.puremagic.com/pulls.ghtml?projectid=1 that 
> it intends to run the tests.   Surely being listed in the "Old 
> results" table doesn't imply pending runs against a new SHA, 
> otherwise why would there be entries there dated 2017?

I give it a shot:

- Every time a PR is merged at dmd, druntime or phobos ALL 
auto-tester results get invalidated
- The auto-tester constantly tries to build PRs which have now 
results based on his queue (it's sorted after merge priority + 
last pushed timestamp)

The traffic is usually so high that your PR will always be at 
pending. That's really not a big deal. On the contrary, once your 
PR has been approved and gets on the merge queue, it will receive 
higher priority and usually will be merged within 1-3 hours.

Also all reviewers are familiar with auto-tester and pending 
tests have zero effect on the reviewers. All the other CIs 
already give a good feedback and if they want to, they can check 
the history of auto-tester for permanent failures

> I can't find any confirmation on the pulls display that it 
> intends to run the tests.

There's none. It _constantly_ reruns your PR.

>  Surely being listed in the "Old results" table doesn't imply 
> pending runs against a new SHA, otherwise why would there be 
> entries there dated 2017?

The timestamp you see on pulls.ghtml?projectid=1 is GitHub's last 
modified timestamp which essentially is timestamp of the last 
push to the respective branch.

And yep there are a few old PRs, but the auto-tester newer tests 
them as its working queue order is sorted by the "Last Commited" 
timestamp.

Did this clarify things for you?


More information about the Digitalmars-d mailing list