ocket8888 opened a new pull request #6004:
URL: https://github.com/apache/trafficcontrol/pull/6004


   ## What does this PR (Pull Request) do?
   - [x] This PR is not related to any Issue
   
   This PR removes the `-v` flag from calls to `go test` in the Go unit test, 
TO Go client/API integration tests, and TM integration test GitHub Actions CI 
workflows. That verbose output makes finding the actual problems unnecessarily 
difficult.
   
   Additionally, the TV and TO logs for the TO Go client/API integration tests 
workflow are now uploaded as artifacts on failure, rather than interleaved with 
test output, to make reading the output easier. TV build output is still output 
directly.
   
   ## Which Traffic Control components are affected by this PR?
   - CI tests
   
   ## What is the best way to verify this PR?
   Make sure the CI tests still pass, verify that passing TO Go client/API 
integration tests output non-verbose testing logs, and don't upload artifacts 
of the the TV and TO logs. Then, break something on your own branch with these 
changes and push it, and make sure that only the part that's broken is logged, 
and that TV and TO logs are uploaded as artifacts for failing runs.
   
   ## The following criteria are ALL met by this PR
   - [x] This PR includes tests
   - [x] Documentation is unnecessary
   - [x] An update to CHANGELOG.md is not necessary
   - [x] This PR includes any and all required license headers
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY**


-- 
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