[GitHub] asfgit commented on issue #2148: add missing entry in capability table

2018-04-18 Thread GitBox
asfgit commented on issue #2148: add missing entry in capability table URL: https://github.com/apache/incubator-trafficcontrol/pull/2148#issuecomment-382542941 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficco

[GitHub] dewrich commented on issue #2103: Add a Location API

2018-04-18 Thread GitBox
dewrich commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-382541391 My only suggestion is `location` (ie: phys_location, and edge_locations) is overloaded in the CDN, I would have probably called it `coordinates`. O

[GitHub] dangogh opened a new pull request #2149: TO and TODB in docker front-to-back

2018-04-18 Thread GitBox
dangogh opened a new pull request #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149 Taking a different approach and removing rpm and postinstall from the equation. Traffic Ops (perl) and Traffic Ops (go) are each built separately

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-04-18 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382552603 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcont

[GitHub] rob05c commented on issue #2149: TO and TODB in docker front-to-back

2018-04-18 Thread GitBox
rob05c commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382553496 This seems like it adds a lot of complexity to the system, even if the interface is easy, it seems like there's a lot more here to g

[GitHub] dangogh commented on issue #2149: TO and TODB in docker front-to-back

2018-04-18 Thread GitBox
dangogh commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382564606 disagree on the complexity point, but this does serve a different purpose -- so I agree it should be made separate. I'll WIP thi

[GitHub] mtorluemke closed pull request #2132: better grouping for TP menu

2018-04-18 Thread GitBox
mtorluemke closed pull request #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a fore

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-19 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-382720726 @mitchell852 Some minot modification from your suggestions. Request your views before actually modifying those.

[GitHub] asfgit commented on issue #2108: create roles crud endpoints in golang

2018-04-19 Thread GitBox
asfgit commented on issue #2108: create roles crud endpoints in golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2108#issuecomment-382723202 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficco

[GitHub] asfgit commented on issue #2108: create roles crud endpoints in golang

2018-04-19 Thread GitBox
asfgit commented on issue #2108: create roles crud endpoints in golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2108#issuecomment-382724697 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficco

[GitHub] dewrich commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
dewrich commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382756580 @rob05c The only concern that I have is we need to build a TC Docker infrastructure that doesn't go stale. How do you do that? We

[GitHub] dewrich commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
dewrich commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382756580 @rob05c The only concern that I have is we need to build a TC Docker infrastructure that doesn't go stale. How do you do that? We

[GitHub] elsloo opened a new issue #2150: ISO generation tool has bond0 hardcoded

2018-04-19 Thread GitBox
elsloo opened a new issue #2150: ISO generation tool has bond0 hardcoded URL: https://github.com/apache/incubator-trafficcontrol/issues/2150 The ISO generation tool checks explicitly for bond0 instead of /^bond\d+/. This is a

[GitHub] elsloo opened a new pull request #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150.

2018-04-19 Thread GitBox
elsloo opened a new pull request #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150. URL: https://github.com/apache/incubator-trafficcontrol/pull/2151 This is an automated message from the Apache

[GitHub] elsloo closed pull request #2146: [Issue 2096] Fix TR issue with capital letters in CDN domain. fixes #2096

2018-04-19 Thread GitBox
elsloo closed pull request #2146: [Issue 2096] Fix TR issue with capital letters in CDN domain. fixes #2096 URL: https://github.com/apache/incubator-trafficcontrol/pull/2146 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] elsloo closed issue #2096: TR creates two files in /opt/traffic_router/var/auto-zones for each DS when CDN domain contains capitals

2018-04-19 Thread GitBox
elsloo closed issue #2096: TR creates two files in /opt/traffic_router/var/auto-zones for each DS when CDN domain contains capitals URL: https://github.com/apache/incubator-trafficcontrol/issues/2096 This is an automated mes

[GitHub] asfgit commented on issue #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150.

2018-04-19 Thread GitBox
asfgit commented on issue #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150. URL: https://github.com/apache/incubator-trafficcontrol/pull/2151#issuecomment-382772439 Refer to this link for build results (access rights to CI server needed): https://build

[GitHub] dneuman64 closed pull request #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150.

2018-04-19 Thread GitBox
dneuman64 closed pull request #2151: Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150. URL: https://github.com/apache/incubator-trafficcontrol/pull/2151 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below f

[GitHub] dneuman64 closed issue #2150: ISO generation tool has bond0 hardcoded

2018-04-19 Thread GitBox
dneuman64 closed issue #2150: ISO generation tool has bond0 hardcoded URL: https://github.com/apache/incubator-trafficcontrol/issues/2150 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] dewrich opened a new pull request #2152: added license header

2018-04-19 Thread GitBox
dewrich opened a new pull request #2152: added license header URL: https://github.com/apache/incubator-trafficcontrol/pull/2152 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] dangogh commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
dangogh commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382775842 @dewrich we can develop this independently of the current docker structure, and that actually allows us more freedom without disru

[GitHub] dangogh closed pull request #2152: added license header

2018-04-19 Thread GitBox
dangogh closed pull request #2152: added license header URL: https://github.com/apache/incubator-trafficcontrol/pull/2152 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull r

[GitHub] rob05c commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
rob05c commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382776596 @dewrich As above, I have a need for a Docker container which * uses the RPM(s) * is a single container with all of Traffic Ops

[GitHub] rob05c commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
rob05c commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382776596 @dewrich As above, I have a need for a Docker container which * uses the RPM(s) * is a single container with all of Traffic Ops

[GitHub] asfgit commented on issue #2152: added license header

2018-04-19 Thread GitBox
asfgit commented on issue #2152: added license header URL: https://github.com/apache/incubator-trafficcontrol/pull/2152#issuecomment-382777180 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1444/

[GitHub] rivasj opened a new issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile

2018-04-19 Thread GitBox
rivasj opened a new issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile URL: https://github.com/apache/incubator-trafficcontrol/issues/1595 The endpoint at: /api/1.3/servers/$hostname_short/update_status returns an

[GitHub] rawlinp commented on issue #2103: Add a Location API

2018-04-19 Thread GitBox
rawlinp commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-382790333 @dewrich thanks, I was able to pull a lot of this straight out of the Cachegroups API since it already had name/lat/long, and I plan to move the ca

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382795526 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcont

[GitHub] DylanVolz commented on issue #2035: TO API - Provide ability to associate roles w/ capabilities

2018-04-19 Thread GitBox
DylanVolz commented on issue #2035: TO API - Provide ability to associate roles w/ capabilities URL: https://github.com/apache/incubator-trafficcontrol/issues/2035#issuecomment-382811758 I think these can both be implemented on an update to a Role, by adding a field that is an array of Ca

[GitHub] dg4prez commented on issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile

2018-04-19 Thread GitBox
dg4prez commented on issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile URL: https://github.com/apache/incubator-trafficcontrol/issues/1595#issuecomment-382825008 It looks to me like the go version of the API is

[GitHub] dg4prez commented on issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile

2018-04-19 Thread GitBox
dg4prez commented on issue #1595: Traffic Ops update_status endpoint returns empty response without use_reval_pending parameter set in global profile URL: https://github.com/apache/incubator-trafficcontrol/issues/1595#issuecomment-382834228 correction - not the 1.2 api: the UI server statu

[GitHub] asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them

2018-04-19 Thread GitBox
asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them URL: https://github.com/apache/incubator-trafficcontrol/pull/2104#issuecomment-382836764 Refer to this link for build results (access rights to CI server needed): https://builds.apache

[GitHub] dewrich commented on issue #2103: Add a Location API

2018-04-19 Thread GitBox
dewrich commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-382837077 @rawlinp Also don't forget to update the CHANGELOG.MD 😁 This is an automated messa

[GitHub] rawlinp commented on issue #2103: Add a Location API

2018-04-19 Thread GitBox
rawlinp commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-382840740 @dewrich aye, this would indeed require that. Done This is an automated message fro

[GitHub] ajschmidt opened a new issue #2153: Read method on api/1.3/types does not return 'lastUpdated'

2018-04-19 Thread GitBox
ajschmidt opened a new issue #2153: Read method on api/1.3/types does not return 'lastUpdated' URL: https://github.com/apache/incubator-trafficcontrol/issues/2153 api/1.3/types returns an empty value for the 'lastUpdated' field where version api/1.2 returns a valid time stamp. The 'lastUpd

[GitHub] mitchell852 commented on issue #2153: Read method on api/1.3/types does not return 'lastUpdated'

2018-04-19 Thread GitBox
mitchell852 commented on issue #2153: Read method on api/1.3/types does not return 'lastUpdated' URL: https://github.com/apache/incubator-trafficcontrol/issues/2153#issuecomment-382847543 @ajschmidt will be working on this T

[GitHub] dewrich closed pull request #2104: assigns routes the appropriate version number and organizes them

2018-04-19 Thread GitBox
dewrich closed pull request #2104: assigns routes the appropriate version number and organizes them URL: https://github.com/apache/incubator-trafficcontrol/pull/2104 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[GitHub] asfgit commented on issue #2103: Add a Location API

2018-04-19 Thread GitBox
asfgit commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-382856776 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1447/ T

[GitHub] asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them

2018-04-19 Thread GitBox
asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them URL: https://github.com/apache/incubator-trafficcontrol/pull/2104#issuecomment-382870803 Refer to this link for build results (access rights to CI server needed): https://builds.apache

[GitHub] dangogh commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
dangogh commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382882841 I've moved these files to `infrastructure/cdn-in-a-box` to more readily indicate the intent.. open to other names, but I'd like

[GitHub] asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them

2018-04-19 Thread GitBox
asfgit commented on issue #2104: assigns routes the appropriate version number and organizes them URL: https://github.com/apache/incubator-trafficcontrol/pull/2104#issuecomment-382891086 Refer to this link for build results (access rights to CI server needed): https://builds.apache

[GitHub] mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-19 Thread GitBox
mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-382891910 @Vijay-1 - i think that makes sense fetch list Cache group for a fallbackId Example: GET api/{version}/

[GitHub] mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-19 Thread GitBox
mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-382891910 @Vijay-1 - i think that makes sense fetch list Cache group for a fallbackId Example: GET api/{version}/

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-04-19 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-382898433 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcont

[GitHub] dewrich commented on issue #2108: create roles crud endpoints in golang

2018-04-20 Thread GitBox
dewrich commented on issue #2108: create roles crud endpoints in golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2108#issuecomment-383098025 @DylanVolz I thought we decided to have roles "seeded"? This is

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-20 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383123063 @mitchell852 Looks like you missed the Order parameter: PUT Request [ { "cacheGroupId": 1, "fallbackName":

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-04-20 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-383146367 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcont

[GitHub] asfgit commented on issue #2103: Add a Coordinates API

2018-04-20 Thread GitBox
asfgit commented on issue #2103: Add a Coordinates API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-383231138 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1452/

[GitHub] rob05c commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
rob05c commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383231321 Adding WIP label: As far as I'm aware, this is complete, but I haven't thoroughly tested. Delivery Services endpoints: * cre

[GitHub] rob05c commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
rob05c commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383231321 Adding WIP label: As far as I'm aware, this is complete, but I haven't thoroughly tested. Delivery Services endpoints: * cre

[GitHub] asfgit commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
asfgit commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383233243 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontro

[GitHub] asfgit commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
asfgit commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383233372 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontro

[GitHub] asfgit commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
asfgit commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383243101 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontro

[GitHub] dangogh commented on issue #2124: Add TO Go deliveryservices routes

2018-04-20 Thread GitBox
dangogh commented on issue #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-383244417 RAT flagged a couple of files for lack of license -- don't forget to check those.. -

[GitHub] asfgit commented on issue #2103: Add a Coordinates API

2018-04-20 Thread GitBox
asfgit commented on issue #2103: Add a Coordinates API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-383250628 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1456/

[GitHub] mitchell852 opened a new pull request #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-21 Thread GitBox
mitchell852 opened a new pull request #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154 fixes #1470 This is an automated message from the

[GitHub] asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-21 Thread GitBox
asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154#issuecomment-383328450 Refer to this link for build results (access rights to CI server needed): https://builds.apache.

[GitHub] asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-21 Thread GitBox
asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154#issuecomment-383335163 Refer to this link for build results (access rights to CI server needed): https://builds.apache.

[GitHub] dangogh commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-21 Thread GitBox
dangogh commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154#issuecomment-383353028 RAT points out the license is missing on this file: apache-trafficcontrol-2.3.0-incubating/tr

[GitHub] asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-22 Thread GitBox
asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154#issuecomment-383408344 Refer to this link for build results (access rights to CI server needed): https://builds.apache.

[GitHub] asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name

2018-04-22 Thread GitBox
asfgit commented on issue #2154: vendors jsdiff and moment.js and adds versions to the file name URL: https://github.com/apache/incubator-trafficcontrol/pull/2154#issuecomment-383410853 Refer to this link for build results (access rights to CI server needed): https://builds.apache.

[GitHub] ajschmidt opened a new pull request #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl

2018-04-22 Thread GitBox
ajschmidt opened a new pull request #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl URL: https://github.com/apache/incubator-trafficcontrol/pull/2155 This PR fixes issue: https://github.com/apache/incubator-trafficcontrol/issues/2153 ---

[GitHub] asfgit commented on issue #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl

2018-04-22 Thread GitBox
asfgit commented on issue #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl URL: https://github.com/apache/incubator-trafficcontrol/pull/2155#issuecomment-383424364 Can one of the admins verify this patch? -

[GitHub] asfgit commented on issue #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl

2018-04-22 Thread GitBox
asfgit commented on issue #2155: Fixed missing 'lastUpdate' field in Types Go API and remove Perl URL: https://github.com/apache/incubator-trafficcontrol/pull/2155#issuecomment-38342 Can one of the admins verify this patch? -

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383509104 @mitchell852 I spent some time in the GoLang version implementation today and i have some doubts on GoLang versi

[GitHub] mitchell852 commented on issue #2055: deleting inactive delivery services requires double confirmation

2018-04-23 Thread GitBox
mitchell852 commented on issue #2055: deleting inactive delivery services requires double confirmation URL: https://github.com/apache/incubator-trafficcontrol/issues/2055#issuecomment-383586243 #3 should be 3. then assign to you and "fulfill" (not delete) -

[GitHub] mitchell852 commented on issue #2055: deleting inactive delivery services requires double confirmation

2018-04-23 Thread GitBox
mitchell852 commented on issue #2055: deleting inactive delivery services requires double confirmation URL: https://github.com/apache/incubator-trafficcontrol/issues/2055#issuecomment-383586243 3 should be 3. then assign to you and "fulfill" (not delete) --

[GitHub] guzzijason opened a new issue #2156: When renaming a host in TC, the hashId retains the value of the previous hostname

2018-04-23 Thread GitBox
guzzijason opened a new issue #2156: When renaming a host in TC, the hashId retains the value of the previous hostname URL: https://github.com/apache/incubator-trafficcontrol/issues/2156 Observed in TP 2.3.0-8247.fb69f019.el7 Possible hashId naming collision if previous hostname valu

[GitHub] mitchell852 commented on issue #2156: When renaming a host in TC, the hashId retains the value of the previous hostname

2018-04-23 Thread GitBox
mitchell852 commented on issue #2156: When renaming a host in TC, the hashId retains the value of the previous hostname URL: https://github.com/apache/incubator-trafficcontrol/issues/2156#issuecomment-383621185 This appears to be because in the crconfig.json file, the hashId is derived fr

[GitHub] asfgit commented on issue #2103: Add a Coordinates API

2018-04-23 Thread GitBox
asfgit commented on issue #2103: Add a Coordinates API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-383629882 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1461/

[GitHub] rawlinp commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
rawlinp commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383638835 I reviewed the TR-side of the PR which looks good to me. Once the API-side is solid I'll pull this down and try out the

[GitHub] rivasj opened a new pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
rivasj opened a new pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157 Added json response field use_reval_pending to update_status endpoint in golang API

[GitHub] asfgit commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
asfgit commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157#issuecomment-383690560 Can one of the admins verify this patch? ---

[GitHub] asfgit commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
asfgit commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157#issuecomment-383691672 Can one of the admins verify this patch? ---

[GitHub] dg4prez opened a new pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-23 Thread GitBox
dg4prez opened a new pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158 Updates ORT to work with the revised 1.3 update status API. If the use_reval_pending flag is not present, reverts to the U

[GitHub] dg4prez commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
dg4prez commented on issue #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157#issuecomment-383707470 Verified and tested against the ORT code in https://github.com/apache/incubator-tra

[GitHub] rivasj opened a new pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
rivasj opened a new pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157 Added json response field use_reval_pending to update_status endpoint in golang API

[GitHub] dg4prez closed pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
dg4prez closed pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] dg4prez closed pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595

2018-04-23 Thread GitBox
dg4prez closed pull request #2157: Added json response field use_reval_pending to update_status endpoint. Fixes #1595 URL: https://github.com/apache/incubator-trafficcontrol/pull/2157 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] asfgit commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-23 Thread GitBox
asfgit commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#issuecomment-383709033 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-23 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-383798798 @rawlinp Thank you. Will let you know once we get the API endpoints right. -

[GitHub] mitchell852 commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-24 Thread GitBox
mitchell852 commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#discussion_r183714002 ## File path: traffic_ops/bin/traffic_ops_ort.pl ## @@ -783,6 +771,

[GitHub] mitchell852 commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-24 Thread GitBox
mitchell852 commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#discussion_r183715686 ## File path: traffic_ops/bin/traffic_ops_ort.pl ## @@ -783,6 +771,

[GitHub] dg4prez commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-24 Thread GitBox
dg4prez commented on a change in pull request #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#discussion_r183716652 ## File path: traffic_ops/bin/traffic_ops_ort.pl ## @@ -783,6 +771,53 @

[GitHub] dneuman64 opened a new issue #2159: Let's Encrypt support

2018-04-24 Thread GitBox
dneuman64 opened a new issue #2159: Let's Encrypt support URL: https://github.com/apache/incubator-trafficcontrol/issues/2159 Let's Encrypt now supports wildcard certs (https://community.letsencrypt.org/t/acme-v2-and-wildcard-certificate-support-is-live/55579). We should update tr

[GitHub] dg4prez commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-24 Thread GitBox
dg4prez commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#issuecomment-383951804 This is updated to not worry about backwards compatibility and only checks to be sure the updated 1.3 api

[GitHub] asfgit commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation.

2018-04-24 Thread GitBox
asfgit commented on issue #2158: modifies ORT to correctly parse api 1.3 for revalidation. URL: https://github.com/apache/incubator-trafficcontrol/pull/2158#issuecomment-383953079 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/

[GitHub] rivasj opened a new pull request #2160: Fixed failing TestGetServerUpdateStatus test

2018-04-24 Thread GitBox
rivasj opened a new pull request #2160: Fixed failing TestGetServerUpdateStatus test URL: https://github.com/apache/incubator-trafficcontrol/pull/2160 Fixed failing TestGetServerUpdateStatus test caused by PR #2157 This is a

[GitHub] dg4prez closed pull request #2160: Fixed failing TestGetServerUpdateStatus test

2018-04-24 Thread GitBox
dg4prez closed pull request #2160: Fixed failing TestGetServerUpdateStatus test URL: https://github.com/apache/incubator-trafficcontrol/pull/2160 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] elsloo opened a new issue #2161: Add ability to bulk change DNS TTLs on delivery services

2018-04-24 Thread GitBox
elsloo opened a new issue #2161: Add ability to bulk change DNS TTLs on delivery services URL: https://github.com/apache/incubator-trafficcontrol/issues/2161 Traffic Ops needs to have an API that will allow one to perform bulk changes to DNS TTLs on delivery services. Specifically, we need

[GitHub] elsloo opened a new issue #2162: NIC throughput not properly calculated after cache reboot

2018-04-24 Thread GitBox
elsloo opened a new issue #2162: NIC throughput not properly calculated after cache reboot URL: https://github.com/apache/incubator-trafficcontrol/issues/2162 If a cache is rebooted, when it comes back online, the NIC's bandwidth counter resets. Traffic Monitor does not handle this situati

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-04-24 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-384061802 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcont

  1   2   3   4   5   6   7   8   9   10   >