[GitHub] saoyag001 opened a new issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN

saoyag001 opened a new issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN URL: https://github.com/apache/trafficcontrol/issues/2460 Steps to reproduce. Create a DS with a mid header rewrite in cdn a check parameters that are created.

[GitHub] rob05c commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN

rob05c commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN URL: https://github.com/apache/trafficcontrol/issues/2460#issuecomment-399965864 This can't be Go, because DS creation where it was observed isn't in Go yet

[GitHub] mitchell852 closed pull request #2462: Remove duplicate API routes for coordinates and origins

mitchell852 closed pull request #2462: Remove duplicate API routes for coordinates and origins URL: https://github.com/apache/trafficcontrol/pull/2462 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

[GitHub] asfgit commented on issue #2358: Created the static dns entry API

asfgit commented on issue #2358: Created the static dns entry API URL: https://github.com/apache/trafficcontrol/pull/2358#issuecomment-399945074 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1902/ Test

[GitHub] guzzijason commented on issue #2461: Mid header rewrite parameters being automatically assigned to wrong CDN

guzzijason commented on issue #2461: Mid header rewrite parameters being automatically assigned to wrong CDN URL: https://github.com/apache/trafficcontrol/issues/2461#issuecomment-399939924 Dup of #2460 This is an automated

[GitHub] guzzijason closed issue #2461: Mid header rewrite parameters being automatically assigned to wrong CDN

guzzijason closed issue #2461: Mid header rewrite parameters being automatically assigned to wrong CDN URL: https://github.com/apache/trafficcontrol/issues/2461 This is an automated message from the Apache Git Service. To

[GitHub] dneuman64 closed pull request #2442: Improve TR logging when a certificate from TO fails to decode

dneuman64 closed pull request #2442: Improve TR logging when a certificate from TO fails to decode URL: https://github.com/apache/trafficcontrol/pull/2442 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

[GitHub] mitchell852 commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN

mitchell852 commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN URL: https://github.com/apache/trafficcontrol/issues/2460#issuecomment-399966818 may be related to #2176 as well

[GitHub] asfgit commented on issue #2305: Add TO Go cdns/health

asfgit commented on issue #2305: Add TO Go cdns/health URL: https://github.com/apache/trafficcontrol/pull/2305#issuecomment-400030778 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1907/ Test PASSed.

[GitHub] mitchell852 commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN

mitchell852 commented on issue #2460: mid header rewrite location parameter creation is sending to the incorrect CDN URL: https://github.com/apache/trafficcontrol/issues/2460#issuecomment-34302 @saoyag001 - need to know what version of TO this bug was observed in.

[GitHub] asfgit commented on issue #2091: Fix TM to warn not err on matchset without regex

asfgit commented on issue #2091: Fix TM to warn not err on matchset without regex URL: https://github.com/apache/trafficcontrol/pull/2091#issuecomment-33903 Refer to this link for build results (access rights to CI server needed):

[GitHub] dangogh closed pull request #2091: Fix TM to warn not err on matchset without regex

dangogh closed pull request #2091: Fix TM to warn not err on matchset without regex URL: https://github.com/apache/trafficcontrol/pull/2091 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

[GitHub] asfgit commented on issue #2300: Add TO Go cachegroups/id/deliveryservices

asfgit commented on issue #2300: Add TO Go cachegroups/id/deliveryservices URL: https://github.com/apache/trafficcontrol/pull/2300#issuecomment-400031647 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1908/

[GitHub] asfgit commented on issue #2278: Add TO Go caches/stats endpoint

asfgit commented on issue #2278: Add TO Go caches/stats endpoint URL: https://github.com/apache/trafficcontrol/pull/2278#issuecomment-400025951 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1906/ Test

Build failed in Jenkins: trafficcontrol-rat #823

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

[GitHub] dewrich commented on a change in pull request #2451: Fix delivery service validation

dewrich commented on a change in pull request #2451: Fix delivery service validation URL: https://github.com/apache/trafficcontrol/pull/2451#discussion_r197866808 ## File path: lib/go-tc/tovalidate/rules.go ## @@ -46,3 +48,19 @@ func IsOneOfStringICase(set ...string)

[GitHub] dewrich closed pull request #2451: Fix delivery service validation

dewrich closed pull request #2451: Fix delivery service validation URL: https://github.com/apache/trafficcontrol/pull/2451 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] dewrich closed issue #2373: Missing validation in Golang Delivery Service API

dewrich closed issue #2373: Missing validation in Golang Delivery Service API URL: https://github.com/apache/trafficcontrol/issues/2373 This is an automated message from the Apache Git Service. To respond to the message,

Build failed in Jenkins: trafficcontrol-rat #824

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

[GitHub] DylanVolz commented on issue #2249: Add CRConfig DS Modified fields

DylanVolz commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/trafficcontrol/pull/2249#issuecomment-400080589 The migration needs to be renamed to a later date so that it will run. It also appears to have syntax errors: `Migrating database...

[GitHub] jhg03a commented on issue #2464: Add documentation generation to pkg

jhg03a commented on issue #2464: Add documentation generation to pkg URL: https://github.com/apache/trafficcontrol/pull/2464#issuecomment-400116732 ok to test This is an automated message from the Apache Git Service. To

[GitHub] dneuman64 commented on a change in pull request #2305: Add TO Go cdns/health

dneuman64 commented on a change in pull request #2305: Add TO Go cdns/health URL: https://github.com/apache/trafficcontrol/pull/2305#discussion_r197957877 ## File path: lib/go-tc/enum.go ## @@ -53,6 +53,8 @@ type DeliveryServiceName string // CacheType is the type (or

[GitHub] dneuman64 commented on issue #2233: Fixed the Go import package alignment because the github project moved

dneuman64 commented on issue #2233: Fixed the Go import package alignment because the github project moved URL: https://github.com/apache/trafficcontrol/pull/2233#issuecomment-400113929 @dewrich is this PR still valid? If so, can you please fix the merge issues?

[GitHub] dneuman64 commented on issue #2284: Remove (Incubating), and some other stuff not needed

dneuman64 commented on issue #2284: Remove (Incubating), and some other stuff not needed URL: https://github.com/apache/trafficcontrol/pull/2284#issuecomment-400113735 Hey @knutsel do you still want this PR merged? It looks like it has some conflicts.

Build failed in Jenkins: trafficcontrol-PR-rat #1181

See -- Started by upstream project "trafficcontrol-PR" build number 1911 originally caused by: GitHub pull request #2187 of commit f72c2b2d7a9a3e0dcc1c6f070c72320eb3bf36eb, no

[GitHub] asfgit commented on issue #2187: updating roles and capabilities seeds

asfgit commented on issue #2187: updating roles and capabilities seeds URL: https://github.com/apache/trafficcontrol/pull/2187#issuecomment-400112495 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1911/

[GitHub] dangogh opened a new pull request #2466: DOC: Update profile_parameter.rst

dangogh opened a new pull request #2466: DOC: Update profile_parameter.rst URL: https://github.com/apache/trafficcontrol/pull/2466 fixed a number of typos and grammatical errors This is an automated message from the Apache

[GitHub] dneuman64 commented on issue #2305: Add TO Go cdns/health

dneuman64 commented on issue #2305: Add TO Go cdns/health URL: https://github.com/apache/trafficcontrol/pull/2305#issuecomment-400113317 I would expect a heathtest.go to be a part of this PR is there a reason it is not included?

[GitHub] asfgit commented on issue #2464: Add documentation generation to pkg

asfgit commented on issue #2464: Add documentation generation to pkg URL: https://github.com/apache/trafficcontrol/pull/2464#issuecomment-400117790 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1912/ Test

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

See

Build failed in Jenkins: trafficcontrol-rat #825

See -- Started by upstream project "trafficcontrol-master-build" build number 870 originally caused by: Started by an SCM change [EnvInject] - Loading node environment variables.

[GitHub] dangogh commented on issue #2464: Add documentation generation to pkg

dangogh commented on issue #2464: Add documentation generation to pkg URL: https://github.com/apache/trafficcontrol/pull/2464#issuecomment-400115997 add to whitelist This is an automated message from the Apache Git Service.

Jenkins build is back to normal : trafficcontrol-rat #828

See

[GitHub] jhg03a opened a new issue #2463: Feature: Add documentation generation to pkg

jhg03a opened a new issue #2463: Feature: Add documentation generation to pkg URL: https://github.com/apache/trafficcontrol/issues/2463 I believe it would be worthwhile to enable new developers and technical writers the ability to easily compile our documentation. As a secondary bonus,

[GitHub] asfgit commented on issue #2451: Fix delivery service validation

asfgit commented on issue #2451: Fix delivery service validation URL: https://github.com/apache/trafficcontrol/pull/2451#issuecomment-400044860 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1909/ Test

[GitHub] asfgit commented on issue #2464: Add documentation generation to pkg

asfgit commented on issue #2464: Add documentation generation to pkg URL: https://github.com/apache/trafficcontrol/pull/2464#issuecomment-400094284 Can one of the admins verify this patch? This is an automated message from