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

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.

Jenkins build is back to normal : incubator-trafficcontrol-PR-rat #729

See

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

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

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

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #730

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1442 originally caused by: GitHub pull request #2108 of commit

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

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):

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

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

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

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

Build failed in Jenkins: incubator-trafficcontrol-rat #669

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 714 originally caused by: Started by an SCM change [EnvInject] - Loading node

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

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

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

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

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #731

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1443 originally caused by: GitHub pull request #2151 of commit

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

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):

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

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

Jenkins build is back to normal : incubator-trafficcontrol-PR-rat #732

See

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

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

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

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,

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

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

Build failed in Jenkins: incubator-trafficcontrol-rat #670

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 715 originally caused by: Started by an SCM change [EnvInject] - Loading node

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

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/

Jenkins build is back to normal : incubator-trafficcontrol-rat #671

See

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

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

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

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] dangogh closed pull request #2152: added license header

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

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

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

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

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):

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

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):

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

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

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

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

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

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

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

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

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

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

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):

Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #557

See Changes: [neuman] Fixes hardcoded bond0 check in the ISO generation API. This fixes #2150. [dangogh] added license header -- [...truncated

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

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

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

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

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #734

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1446 originally caused by: GitHub pull request #2104 of commit

Jenkins build is back to normal : incubator-trafficcontrol-PR-rat #735

See

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

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/

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

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

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

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):

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #738

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1450 originally caused by: GitHub pull request #2149 of commit

Jenkins build is back to normal : incubator-trafficcontrol-PR #1449

See

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

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):

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

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

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

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

Jenkins build is back to normal : incubator-trafficcontrol-traffic_ops-test #558

See

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

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

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #737

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1449 originally caused by: GitHub pull request #2104 of commit