You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2020/10/29 23:04:40 UTC

[GitHub] [trafficcontrol] rawlinp opened a new pull request #5227: Fix CIAB mid enrollment

rawlinp opened a new pull request #5227:
URL: https://github.com/apache/trafficcontrol/pull/5227


   ## What does this PR (Pull Request) do?
   The `hostname` command, depending on particular CentOS versions, will sometimes return the FQDN or the short hostname. This fixes the mid enrollment to use `--short` in order to not try to find cachegroups with the suffix `infra.ciab.test` (which don't exist).
   
   ## Which Traffic Control components are affected by this PR?
   - CDN in a Box
   
   ## What is the best way to verify this PR?
   Verify the CIAB check completes successfully. If you _really_ want to verify, modify the `mid` Dockerfile to use CentOS 7.6.1810 (the version I happen to be using).
   
   ## If this is a bug fix, what versions of Traffic Control are affected?
   - master
   - 5.0
   
   ## The following criteria are ALL met by this PR
   
   - [x] ciab is a test
   - [x] docs unnecessary
   - [x] changelog unnecessary
   - [x] This PR includes any and all required license headers
   - [x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY** (see [the Apache Software Foundation's security guidelines](https://www.apache.org/security/) for details)
   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [trafficcontrol] zrhoffman merged pull request #5227: Fix CIAB mid enrollment

Posted by GitBox <gi...@apache.org>.
zrhoffman merged pull request #5227:
URL: https://github.com/apache/trafficcontrol/pull/5227


   


----------------------------------------------------------------
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:
users@infra.apache.org