{"action":"created","issue":{"url":"https://api.github.com/repos/apache/trafficcontrol/issues/2566","repository_url":"https://api.github.com/repos/apache/trafficcontrol","labels_url":"https://api.github.com/repos/apache/trafficcontrol/issues/2566/labels{/name}","comments_url":"https://api.github.com/repos/apache/trafficcontrol/issues/2566/comments","events_url":"https://api.github.com/repos/apache/trafficcontrol/issues/2566/events","html_url":"https://github.com/apache/trafficcontrol/issues/2566","id":342879203,"node_id":"MDU6SXNzdWUzNDI4NzkyMDM=","number":2566,"title":"Traffic Ops API endpoint /api/1.2/cachegroups missing key requirement","user":{"login":"ocket8888","id":6013378,"node_id":"MDQ6VXNlcjYwMTMzNzg=","avatar_url":"https://avatars0.githubusercontent.com/u/6013378?v=4","gravatar_id":"","url":"https://api.github.com/users/ocket8888","html_url":"https://github.com/ocket8888","followers_url":"https://api.github.com/users/ocket8888/followers","following_url":"https://api.github.com/users/ocket8888/following{/other_user}","gists_url":"https://api.github.com/users/ocket8888/gists{/gist_id}","starred_url":"https://api.github.com/users/ocket8888/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ocket8888/subscriptions","organizations_url":"https://api.github.com/users/ocket8888/orgs","repos_url":"https://api.github.com/users/ocket8888/repos","events_url":"https://api.github.com/users/ocket8888/events{/privacy}","received_events_url":"https://api.github.com/users/ocket8888/received_events","type":"User","site_admin":false},"labels":[{"id":675814919,"node_id":"MDU6TGFiZWw2NzU4MTQ5MTk=","url":"https://api.github.com/repos/apache/trafficcontrol/labels/Traffic%20Ops","name":"Traffic Ops","color":"808080","default":false},{"id":1070217405,"node_id":"MDU6TGFiZWwxMDcwMjE3NDA1","url":"https://api.github.com/repos/apache/trafficcontrol/labels/Traffic%20Ops%20API%202.0","name":"Traffic Ops API 2.0","color":"e99695","default":false},{"id":435396616,"node_id":"MDU6TGFiZWw0MzUzOTY2MTY=","url":"https://api.github.com/repos/apache/trafficcontrol/labels/bug","name":"bug","color":"ee0701","default":true},{"id":678937474,"node_id":"MDU6TGFiZWw2Nzg5Mzc0NzQ=","url":"https://api.github.com/repos/apache/trafficcontrol/labels/major","name":"major","color":"d93f0b","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-07-19T20:34:07Z","updated_at":"2018-10-19T19:50:18Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"For certain operations (including snapshot generation) cache groups MUST have geomagnetic coordinates (latitude & longitude), which is enforced in the Traffic Portal UI but not by the backing Traffic Ops endpoint.\r\n\r\nYou really ought to have coordinates on a production CDN, as they can be very helpful.\r\n\r\nPossible Solutions:\r\n\r\n1. Enforce that new cache groups **must** have coordinates\r\n2. Provide some default coordinates, either at cache group creation or at snapshot time."},"comment":{"url":"https://api.github.com/repos/apache/trafficcontrol/issues/comments/431479593","html_url":"https://github.com/apache/trafficcontrol/issues/2566#issuecomment-431479593","issue_url":"https://api.github.com/repos/apache/trafficcontrol/issues/2566","id":431479593,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMTQ3OTU5Mw==","user":{"login":"rawlinp","id":6724053,"node_id":"MDQ6VXNlcjY3MjQwNTM=","avatar_url":"https://avatars2.githubusercontent.com/u/6724053?v=4","gravatar_id":"","url":"https://api.github.com/users/rawlinp","html_url":"https://github.com/rawlinp","followers_url":"https://api.github.com/users/rawlinp/followers","following_url":"https://api.github.com/users/rawlinp/following{/other_user}","gists_url":"https://api.github.com/users/rawlinp/gists{/gist_id}","starred_url":"https://api.github.com/users/rawlinp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rawlinp/subscriptions","organizations_url":"https://api.github.com/users/rawlinp/orgs","repos_url":"https://api.github.com/users/rawlinp/repos","events_url":"https://api.github.com/users/rawlinp/events{/privacy}","received_events_url":"https://api.github.com/users/rawlinp/received_events","type":"User","site_admin":false},"created_at":"2018-10-19T19:50:18Z","updated_at":"2018-10-19T19:50:18Z","author_association":"CONTRIBUTOR","body":"Yes, however the Cachegroup API only creates a Coordinate under the hood from the lat/long in the request and sets cachegroup.coordinate to the ID of the created coordinate. In API 2.x we should make Cachegroups require a Coordinate, possibly making the Cachegroup API require a Coordinate ID rather than the lat/long."}}
{"actor":{"display_login":"rawlinp"}}