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 2019/02/14 03:02:18 UTC

[GitHub] Shihta commented on issue #3281: CIAB: Fixed the method of obtaining IP

Shihta commented on issue #3281: CIAB: Fixed the method of obtaining IP
URL: https://github.com/apache/trafficcontrol/pull/3281#issuecomment-463468261
 
 
   @JBevillC thanks for helping me review it!
   
   The 2 problems you found are not caused by this commit.
   As you mentioned that there are some files in `infrastructure/cdn-in-a-box/traffic_ops_data/servers`.
   They will be modified by `infrastructure/cdn-in-a-box/traffic_ops/set-to-ips-from-dns.sh` and then be sent by container enroller.
   
   In fact, I have sent a PR #3264 trying to solve these 2 problems.
   In PR #3264, it add a retry mechanism in obtaining IP and fix the netmask issue.
   I didn't highlight it in slack because I'm not sure it's a right direction.
   
   Should we just fix them by the PR #3264?
   or should we let each service goes to enroll by themselves? like your opinion
   
   It seems like the option 2 is better than 1.
   If we want to select option 2, I'll close PR #3264 first and work the selection 2 in the future.
   
   ------------
   
   Back to this PR
   It aim to fix the sequence of registering DNS record, not service enrollment of TO.
   I don't know why, but registering DNS record of DB sometimes just fails in my environment.
   That why I sent this PR

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services