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


   ## What does this PR (Pull Request) do?
   
   - [x] This PR is not related to any Issue
   
   This PR fixes some experimental/miscellaneous Go tools and/or their tests 
that had compilation issues. They were:
   
   - experimental TR in Go
   - test/router's DNSSEC test suite
   - the `traffic_vault_util` program under `tools/`
   - the semi-experimental `t3c` tool in `traffic_ops_ort/`
   
   The size of this PR is nearly exclusively due to moving vendored packages 
from `traffic_ops/vendor` into `vendor/` so that `traffic_vault_util` would 
compile. They might look like big changes, but the files were just renamed 
without changes and the Go tests/CiaB Action should demonstrate this had no 
effect on compiling Traffic Ops.
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Ops ORT (t3c thing)
   
   ## What is the best way to verify this PR?
   The checks should be able to do the heavy lifting, but you could also check 
each tool I listed above and make sure it compiles/passes its own tests/ 
whatever else. Or don't; they're largely experimental.
   
   ## If this is a bug fix, what versions of Traffic Control are affected?
   - master
   
   ## The following criteria are ALL met by this PR
   - [x] This PR fixes existing tests
   - [x] Documentation is unnecessary for bug fixes
   - [x] An update to CHANGELOG.md is not necessary for changes to experimental 
things.
   - [x] This PR includes any and all required license headers
   - [x] This PR does not include a database migration
   - [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.

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


Reply via email to