[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-388325302 what happens for the following cases: POST /api/1.2/cachegroup_fallbacks [ fallback11, fallback12 ] followed

[GitHub] smalenfant opened a new issue #2257: TP: Bandwidth Reports Edge + Mid, should be only Edge

smalenfant opened a new issue #2257: TP: Bandwidth Reports Edge + Mid, should be only Edge URL: https://github.com/apache/incubator-trafficcontrol/issues/2257 Was looking at my bandwidth report vs what it shows on Traffic Portal and looks like Mid Cache bandwidth is displaying when Mids

[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-388325302 what happens for the following cases: POST /api/1.2/cachegroup_fallbacks [ fallback11, fallback12 ] followed

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

See -- GitHub pull request #2229 of commit b60533903c6c908ba8e931c3a81cdc3e62f382da, no merge conflicts. Setting status of b60533903c6c908ba8e931c3a81cdc3e62f382da to

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388390622 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388392249 Refer to this link for build results (access rights to CI server needed):

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

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

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

See -- GitHub pull request #2229 of commit ebe93b0f27a5dfe3dfb8df65c6e7e3da5c645e8f, no merge conflicts. Setting status of ebe93b0f27a5dfe3dfb8df65c6e7e3da5c645e8f to

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

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

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

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

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388396509 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] mitchell852 commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

mitchell852 commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388388718 I tested all these Go routes through the TP and they all work fine. @DylanVolz @dangogh @dewrich - want to

[GitHub] dewrich commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

dewrich commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388389657 ``` PASS | ReadHandler (0.00s) START| UpdateHandler FAIL | --- FAIL: TestUpdateHandler (0.00s) |

[GitHub] dewrich commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

dewrich commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388389507 Unit Tests are broken ``` master ᐅ gt BUILD|

[GitHub] rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-388391453 @dewrich Fixed. This is an automated message from the

[GitHub] asfgit commented on issue #2102: Add TO all 1.2 CDNs routes, remove perl

asfgit commented on issue #2102: Add TO all 1.2 CDNs routes, remove perl URL: https://github.com/apache/incubator-trafficcontrol/pull/2102#issuecomment-388392501 Refer to this link for build results (access rights to CI server needed):

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

See -- GitHub pull request #2102 of commit e41865aa606ad02f9380469f97324d095456a398, no merge conflicts. Setting status of e41865aa606ad02f9380469f97324d095456a398 to

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

See

[GitHub] asfgit commented on issue #2102: Add TO all 1.2 CDNs routes, remove perl

asfgit commented on issue #2102: Add TO all 1.2 CDNs routes, remove perl URL: https://github.com/apache/incubator-trafficcontrol/pull/2102#issuecomment-388394530 Refer to this link for build results (access rights to CI server needed):

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

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1554 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-388397857 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c commented on a change in pull request #2229: Add Traffic Ops Golang deliveryservicesregexes

rob05c commented on a change in pull request #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#discussion_r187632509 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -214,6 +215,14 @@ func

[GitHub] mitchell852 commented on issue #2255: Traffic Ops API endpoint for generating SSL keys needs validation

mitchell852 commented on issue #2255: Traffic Ops API endpoint for generating SSL keys needs validation URL: https://github.com/apache/incubator-trafficcontrol/issues/2255#issuecomment-388373656 If you generate ssl keys using TP

[GitHub] asfgit commented on issue #2258: Update traffic_router_api.rst

asfgit commented on issue #2258: Update traffic_router_api.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2258#issuecomment-388450881 Can one of the admins verify this patch? This is an automated message

[GitHub] asfgit commented on issue #2258: Update traffic_router_api.rst

asfgit commented on issue #2258: Update traffic_router_api.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2258#issuecomment-388451152 Can one of the admins verify this patch? This is an automated message

[GitHub] killedbywork opened a new pull request #2258: Update traffic_router_api.rst

killedbywork opened a new pull request #2258: Update traffic_router_api.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2258 Updated syntax of API queries (removed colons and added <> for input parameters).

[GitHub] rob05c commented on a change in pull request #2010: Go login

rob05c commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r187747195 ## File path: traffic_ops/traffic_ops_golang/config/config.go ## @@ -226,3 +250,22 @@ func ParseConfig(cfg Config) (Config,

[GitHub] rob05c commented on issue #2010: Go login

rob05c commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-388500648 The LDAP timeout should have a reasonable default. It's defaulting to 0 now, and my LDAP (which is probably misconfigured) is hanging indefinitely. I'd

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

See Changes: [dewrich] Fix TO Go ASNs unit tests for renamed symbols [rob05c] added DeliveryServicesService Get API in Go [rob05c] added api to DeliveryserviceServer Get in Go [rob05c] removed

[GitHub] asfgit commented on issue #2236: update portal info bubbles

asfgit commented on issue #2236: update portal info bubbles URL: https://github.com/apache/incubator-trafficcontrol/pull/2236#issuecomment-388464804 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: incubator-trafficcontrol-master-build #757

See Changes: [neuman] Update traffic_router_api.rst -- Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM

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

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

[GitHub] mitchell852 opened a new issue #2259: Add ?orderby= to all api calls TP makes to fetch a collection

mitchell852 opened a new issue #2259: Add ?orderby= to all api calls TP makes to fetch a collection URL: https://github.com/apache/incubator-trafficcontrol/issues/2259 This will ensure the order of the results. This is an

[GitHub] guzzijason opened a new issue #2260: TP: sorting of cache group list is no longer alphanumeric

guzzijason opened a new issue #2260: TP: sorting of cache group list is no longer alphanumeric URL: https://github.com/apache/incubator-trafficcontrol/issues/2260 Traffic Portal Version: 2.3.0-8247.fb69f019.el7 Cache groups used to sort by alpha, but now appear to be unsorted,

[GitHub] dneuman64 commented on issue #2258: Update traffic_router_api.rst

dneuman64 commented on issue #2258: Update traffic_router_api.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2258#issuecomment-388462105  thanks Pavel! This is an automated message from the Apache Git

[GitHub] dneuman64 closed pull request #2258: Update traffic_router_api.rst

dneuman64 closed pull request #2258: Update traffic_router_api.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2258 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

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

See Changes: [neuman] Update traffic_router_api.rst -- [...truncated 463.96 KB...] Successfully installed Sub-Exporter-ForMethods-0.100052

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

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

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

See -- GitHub pull request #2108 of commit d35459c362f984aa73eb71fbd2e0219578bc31f1, no merge conflicts. Setting status of d35459c362f984aa73eb71fbd2e0219578bc31f1 to

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

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

[GitHub] asfgit commented on issue #2234: Add TO Go staticdnsentries endpoint

asfgit commented on issue #2234: Add TO Go staticdnsentries endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2234#issuecomment-388491293 Refer to this link for build results (access rights to CI server needed):

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

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

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

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

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

asfgit commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-388497590 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c commented on issue #2010: Go login

rob05c commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-388499921 I think this needs to check for the `ldap.conf` file in the same way the existing Perl does

[GitHub] rob05c commented on issue #2010: Go login

rob05c commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-388500648 The LDAP timeout should have a reasonable default. It's defaulting to 0 now, and my LDAP (which is probably misconfigured) is hanging indefinitely. I'd