bryancall commented on PR #13787:
URL: https://github.com/apache/trafficserver/pull/13787#issuecomment-6332705792

   @RajaMuhammadAwais master moved the end-to-end tests from `tests/gold_tests` 
to `tests/uranium_tests` in #13545, and the old AuTest runner no longer runs 
there. GitHub still shows this PR as mergeable, but `git merge-tree` reports 
both new files as `CONFLICT (file location)`. If merged as is, 
`splitdns_failover.test.py` and `splitdns_failover_no_rr.test.py` would land in 
`tests/uranium_tests/dns/` where nothing collects them, so the failover fix 
would ship without its tests running.
   
   Could you merge master and port the two tests to Uranium? The collector 
picks up `test_*.py` files (or `*.test.yaml`). 
`tests/uranium_tests/dns/test_splitdns.py` and `test_dns_down_nameserver.py` 
show the fixtures for a SplitDNS config with a dead nameserver. Once the ported 
tests pass in CI, this looks ready to merge.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to