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

See Changes: [jeffrey_elsloo] Fix TO CRconfig changelog -- [...truncated 2.24 MB...] unit_1 | Use of uninitialized value in join or string

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

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

[GitHub] asfgit commented on issue #2281: Increase hostname and domain name length limit in Traffic Portal

asfgit commented on issue #2281: Increase hostname and domain name length limit in Traffic Portal URL: https://github.com/apache/incubator-trafficcontrol/pull/2281#issuecomment-389681832 Refer to this link for build results (access rights to CI server needed):

[GitHub] rawlinp commented on issue #2279: Increase server hostname max length

rawlinp commented on issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279#issuecomment-389679003 Sure, just opened #2281 to increase both limits in TP to 100.

[GitHub] rawlinp opened a new pull request #2281: Increase hostname and domain name length limit in Traffic Portal

rawlinp opened a new pull request #2281: Increase hostname and domain name length limit in Traffic Portal URL: https://github.com/apache/incubator-trafficcontrol/pull/2281 This changes is from 45 to 100. The limit itself could probably be removed altogether since the DB type is `text`,

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

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 763 originally caused by: Started by an SCM change Started by an SCM change

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

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

[GitHub] asfgit commented on issue #2280: Fix TO CRconfig changelog

asfgit commented on issue #2280: Fix TO CRconfig changelog URL: https://github.com/apache/incubator-trafficcontrol/pull/2280#issuecomment-389676041 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 commented on issue #2279: Increase server hostname max length

mitchell852 commented on issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279#issuecomment-389673406 there's probably a maxLength attribute in TP. you want to remove that @rawlinp ?

[GitHub] elsloo closed pull request #2280: Fix TO CRconfig changelog

elsloo closed pull request #2280: Fix TO CRconfig changelog URL: https://github.com/apache/incubator-trafficcontrol/pull/2280 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

[GitHub] rob05c opened a new pull request #2280: Fix TO CRconfig changelog

rob05c opened a new pull request #2280: Fix TO CRconfig changelog URL: https://github.com/apache/incubator-trafficcontrol/pull/2280 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rawlinp commented on issue #2279: Increase server hostname max length

rawlinp commented on issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279#issuecomment-389668866 Yeah I think since the DB type is just `text` we should be able to bump up the artificial limit in TP.

[GitHub] jhg03a commented on issue #2279: Increase server hostname max length

jhg03a commented on issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279#issuecomment-389668324 Yes. I am able to enter the data in the old TO interface, but that makes the server uneditable in the current TP.

[GitHub] mitchell852 commented on issue #2279: Increase server hostname max length

mitchell852 commented on issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279#issuecomment-389652233 Are you referring to the field in traffic portal? This is

[GitHub] jhg03a opened a new issue #2279: Increase server hostname max length

jhg03a opened a new issue #2279: Increase server hostname max length URL: https://github.com/apache/incubator-trafficcontrol/issues/2279 As a CDN SRE I want to be able to supply an arbitrarily long hostname. Today I actually only need 5 more characters, but this seems like a hidden limit

[GitHub] rob05c closed issue #2243: Routes.go file should include type capture groups to enforce route parameter types

rob05c closed issue #2243: Routes.go file should include type capture groups to enforce route parameter types URL: https://github.com/apache/incubator-trafficcontrol/issues/2243 This is an automated message from the Apache

[GitHub] DylanVolz commented on issue #2277: Allow the capture group for a route parameter to be defined

DylanVolz commented on issue #2277: Allow the capture group for a route parameter to be defined URL: https://github.com/apache/incubator-trafficcontrol/pull/2277#issuecomment-389624176 After analysis of the legacy routes we will be porting from perl to go this feature is unnecessary. By

[GitHub] DylanVolz closed pull request #2277: Allow the capture group for a route parameter to be defined

DylanVolz closed pull request #2277: Allow the capture group for a route parameter to be defined URL: https://github.com/apache/incubator-trafficcontrol/pull/2277 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] DylanVolz commented on issue #2243: Routes.go file should include type capture groups to enforce route parameter types

DylanVolz commented on issue #2243: Routes.go file should include type capture groups to enforce route parameter types URL: https://github.com/apache/incubator-trafficcontrol/issues/2243#issuecomment-389624250 After analysis of the legacy routes we will be porting from perl to go this

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

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

[GitHub] asfgit commented on issue #2247: Add an Origin API

asfgit commented on issue #2247: Add an Origin API URL: https://github.com/apache/incubator-trafficcontrol/pull/2247#issuecomment-389582209 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1585/

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

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

[GitHub] knutsel commented on issue #2246: Add HitCount and FreshFor

knutsel commented on issue #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#issuecomment-389554197 @rob05c - like this? Sorry that the diff in rfc/rules.go is a bit bigger than needed, hard to get it all in the right place.

[GitHub] asfgit commented on issue #2246: Add HitCount and FreshFor

asfgit commented on issue #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#issuecomment-389553758 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] asfgit commented on issue #2246: Add HitCount and FreshFor

asfgit commented on issue #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#issuecomment-389552212 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] asfgit commented on issue #2246: Add HitCount and FreshFor

asfgit commented on issue #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#issuecomment-389530137 Refer to this link for build results (access rights to CI server needed):

[GitHub] knutsel commented on a change in pull request #2246: Add HitCount and FreshFor

knutsel commented on a change in pull request #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#discussion_r188635843 ## File path: grove/plugin/http_cacheinspector.go ## @@ -169,16 +170,17 @@ func cacheinspect(icfg