[GitHub] mitchell852 closed pull request #2217: Fix Traffic Ops Go CRConfig to implement maxmind override logic

mitchell852 closed pull request #2217: Fix Traffic Ops Go CRConfig to implement maxmind override logic URL: https://github.com/apache/incubator-trafficcontrol/pull/2217 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] rob05c opened a new pull request #2221: Fix TO Go CRConfig to omit null missLocations

rob05c opened a new pull request #2221: Fix TO Go CRConfig to omit null missLocations URL: https://github.com/apache/incubator-trafficcontrol/pull/2221 This fixed the Golang CRConfig to do what Perl does: omit the `missLocation` field if it's null in the database. Specifically,

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

See

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

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

[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-386330729 Refer to this link for build results (access rights to CI server needed):

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

See

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

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

[GitHub] dewrich closed pull request #2103: Add a Coordinates API

dewrich closed pull request #2103: Add a Coordinates API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103 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

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

See

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

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

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

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1515 originally caused by: GitHub pull request #2103 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-386410306 Refer to this link for build results (access rights to CI server needed):

[GitHub] dneuman64 opened a new issue #2223: Traffic Router leaks file handles when it cannot get SSL certificates

dneuman64 opened a new issue #2223: Traffic Router leaks file handles when it cannot get SSL certificates URL: https://github.com/apache/incubator-trafficcontrol/issues/2223 Traffic Router is leaking file handles when it cannot retrieve SSL certificates. This will eventually lead to a

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

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

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

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

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

See

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

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

Build failed in Jenkins: incubator-trafficcontrol-PR #1514

See -- GitHub pull request #2103 of commit 4127d6019b026ca0df2578a71177b7f3e32ad0cc, no merge conflicts. [EnvInject] - Loading node environment variables. Building remotely

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

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

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

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

[GitHub] dg4prez opened a new issue #2224: Need to generate tm_host using tm.url instead of host header on crconfig generation

dg4prez opened a new issue #2224: Need to generate tm_host using tm.url instead of host header on crconfig generation URL: https://github.com/apache/incubator-trafficcontrol/issues/2224 RIght now when a crconfig is generated the tm_host field is generated using the host header. This

[GitHub] asfgit commented on issue #2221: Fix TO Go CRConfig to omit null missLocations

asfgit commented on issue #2221: Fix TO Go CRConfig to omit null missLocations URL: https://github.com/apache/incubator-trafficcontrol/pull/2221#issuecomment-386404540 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] rob05c opened a new pull request #2222: Add TO Go 1.1 routes

rob05c opened a new pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/ Specifically, changes routes to be 1.1, which implicitly serve 1.2+, and adds the one different 1.1 ASNs route.

[GitHub] asfgit commented on issue #2222: Add TO Go 1.1 routes

asfgit commented on issue #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#issuecomment-386422316 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1513/

[GitHub] mitchell852 opened a new issue #2220: When fetching a user thru the API, capabilities should be added to the response

mitchell852 opened a new issue #2220: When fetching a user thru the API, capabilities should be added to the response URL: https://github.com/apache/incubator-trafficcontrol/issues/2220 when fetching a user object via one of these routes: GET /api/$version/user/current GET

[GitHub] rivasj opened a new pull request #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride

rivasj opened a new pull request #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride URL: https://github.com/apache/incubator-trafficcontrol/pull/2219 This is an automated message from the Apache Git

[GitHub] DylanVolz opened a new pull request #2218: remove perl test of route moved from perl to go

DylanVolz opened a new pull request #2218: remove perl test of route moved from perl to go URL: https://github.com/apache/incubator-trafficcontrol/pull/2218 This is an automated message from the Apache Git Service. To

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

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

[GitHub] asfgit commented on issue #2218: remove perl test of route moved from perl to go

asfgit commented on issue #2218: remove perl test of route moved from perl to go URL: https://github.com/apache/incubator-trafficcontrol/pull/2218#issuecomment-386352157 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c closed pull request #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride

rob05c closed pull request #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride URL: https://github.com/apache/incubator-trafficcontrol/pull/2219 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] asfgit commented on issue #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride

asfgit commented on issue #2219: renamed maxmind.default.override parameter to maxmindDefaultOverride URL: https://github.com/apache/incubator-trafficcontrol/pull/2219#issuecomment-386364584 Can one of the admins verify this patch?

[GitHub] rob05c commented on issue #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling

rob05c commented on issue #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2225#issuecomment-386455359 Tested in a Docker TO with a prod db, matches Perl output when that

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

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

[GitHub] rob05c opened a new pull request #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling

rob05c opened a new pull request #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2225 This is an automated

[GitHub] asfgit commented on issue #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling

asfgit commented on issue #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2225#issuecomment-386458506 Refer to this link for build results (access rights to CI server