[GitHub] asfgit commented on issue #2475: Remove db from interface

asfgit commented on issue #2475: Remove db from interface URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401804023 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1954/ Test PASSed.

[GitHub] asfgit commented on issue #2475: Remove db from interface

asfgit commented on issue #2475: Remove db from interface URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401813626 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1956/ Test PASSed.

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

See -- Started by upstream project "trafficcontrol-PR" build number 1956 originally caused by: GitHub pull request #2475 of commit 97de0f73d95cdee2fe5ca6b5eb05b58accd2e230, no

[GitHub] asfgit commented on issue #2338: Add TO Go users

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

Build failed in Jenkins: trafficcontrol-master-build #883

See Changes: [dewrich] Add TO Go delete deliveryservice_server [dewrich] Add TO Go delete ds_servers client funcs, tests -- [...truncated 19.27 MB...]

Build failed in Jenkins: trafficcontrol-PR #1973

See Changes: [rob] Add TO Go cdns/capacity -- GitHub pull request #2306 of commit 9336d7863413112986b8cab44027bfab25141d88, no merge conflicts. Setting status of

[GitHub] asfgit commented on issue #2306: Add TO Go cdns/capacity

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

[GitHub] asfgit commented on issue #2492: Refactor cachegroups to use coordinates table

asfgit commented on issue #2492: Refactor cachegroups to use coordinates table URL: https://github.com/apache/trafficcontrol/pull/2492#issuecomment-401929841 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: trafficcontrol-PR #1972

See Changes: [rob05c] Conversion Utility to modify Traffic Ops profile based on conversion [friede] remove DISCLAIMER -- no longer an incubating project [friede] remove -incubating from source tarball name

[GitHub] dewrich commented on a change in pull request #2307: Add TO Go cdns/routing

dewrich commented on a change in pull request #2307: Add TO Go cdns/routing URL: https://github.com/apache/trafficcontrol/pull/2307#discussion_r199619450 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -104,7 +104,7 @@ func Routes(d ServerData) ([]Route,

[GitHub] rob05c opened a new pull request #2495: Fix TO CRConfig perms back to ops/readonly

rob05c opened a new pull request #2495: Fix TO CRConfig perms back to ops/readonly URL: https://github.com/apache/trafficcontrol/pull/2495 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rob05c commented on issue #2353: Add TO Go regions/name

rob05c commented on issue #2353: Add TO Go regions/name URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401955221 @dewrich Fixed This is an automated message from the Apache Git Service. To respond to the

[GitHub] rawlinp opened a new issue #2496: TO Go PUT ASNs, regions, and tenants endpoints now require "id" field in request (possibly breaking clients)

rawlinp opened a new issue #2496: TO Go PUT ASNs, regions, and tenants endpoints now require "id" field in request (possibly breaking clients) URL: https://github.com/apache/trafficcontrol/issues/2496 The following TO Go PUT endpoints now appear to require `"id": ` in the request body

[GitHub] mitchell852 commented on a change in pull request #2307: Add TO Go cdns/routing

mitchell852 commented on a change in pull request #2307: Add TO Go cdns/routing URL: https://github.com/apache/trafficcontrol/pull/2307#discussion_r199622126 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -104,7 +104,7 @@ func Routes(d ServerData) ([]Route,

[GitHub] dewrich commented on issue #2339: Add TO Go users/id/deliveryservices

dewrich commented on issue #2339: Add TO Go users/id/deliveryservices URL: https://github.com/apache/trafficcontrol/pull/2339#issuecomment-401953317 API Tests are broken: ```Dewayne Richardson [4:09 PM] 2339 the API tests are broken ```DEBUG: userdeliveryservices_test.go:33:

[GitHub] asfgit commented on issue #2339: Add TO Go users/id/deliveryservices

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

Build failed in Jenkins: trafficcontrol-PR #1974

See Changes: [rob] Add TO Go users/id/deliveryservices [rob] Add TO Go deliveryservice/users tests -- GitHub pull request #2339 of commit

[GitHub] asfgit commented on issue #2475: Remove db from interface

asfgit commented on issue #2475: Remove db from interface URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401941178 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1975/ Test PASSed.

Jenkins build is back to normal : trafficcontrol-PR #1975

See

[GitHub] asfgit commented on issue #2495: Fix TO CRConfig perms back to ops/readonly

asfgit commented on issue #2495: Fix TO CRConfig perms back to ops/readonly URL: https://github.com/apache/trafficcontrol/pull/2495#issuecomment-401952561 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1976/

[GitHub] dewrich edited a comment on issue #2339: Add TO Go users/id/deliveryservices

dewrich edited a comment on issue #2339: Add TO Go users/id/deliveryservices URL: https://github.com/apache/trafficcontrol/pull/2339#issuecomment-401953317 API Tests are broken: ```Dewayne Richardson [4:09 PM] 2339 the API tests are broken ```DEBUG:

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

See Changes: [neuman] Fix TO CRConfig perms back to ops/readonly -- [...truncated 1.90 MB...] unit_1 | Parse errors: No plan found in

[GitHub] rawlinp opened a new issue #2494: Multiple Go API endpoints don't return "joined" fields on PUT/POST

rawlinp opened a new issue #2494: Multiple Go API endpoints don't return "joined" fields on PUT/POST URL: https://github.com/apache/trafficcontrol/issues/2494 This is a regression from the Perl API, where keys like `parentCachegroupName` are returned with the name of the cachegroup rather

Build failed in Jenkins: trafficcontrol-PR #1977

See Changes: [dewrich] Add TO Go delete deliveryservice_server [dewrich] Add TO Go delete ds_servers client funcs, tests [jrushford] Modify the grove/build/build_rpm.sh script to incorporate the last 9

Build failed in Jenkins: trafficcontrol-PR #1976

See -- GitHub pull request #2495 of commit d4298d654280fa8fc4702d8910c2f69e3e33b9ce, no merge conflicts. Setting status of d4298d654280fa8fc4702d8910c2f69e3e33b9ce to PENDING with url

[GitHub] dneuman64 closed pull request #2495: Fix TO CRConfig perms back to ops/readonly

dneuman64 closed pull request #2495: Fix TO CRConfig perms back to ops/readonly URL: https://github.com/apache/trafficcontrol/pull/2495 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

[GitHub] asfgit commented on issue #2485: Modify the grove rpm version format

asfgit commented on issue #2485: Modify the grove rpm version format URL: https://github.com/apache/trafficcontrol/pull/2485#issuecomment-401952657 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1977/ Test

[GitHub] dewrich commented on issue #2353: Add TO Go regions/name

dewrich commented on issue #2353: Add TO Go regions/name URL: https://github.com/apache/trafficcontrol/pull/2353#issuecomment-401954484 @rob05c Doesn't compile either due to package changes: ```rob05c-to-go-regions-name-2353 ᐅ go build && ./traffic_ops_golang -cfg ~/cdn.conf -dbcfg

Build failed in Jenkins: trafficcontrol-master-build #884

See Changes: [neuman] Fix TO CRConfig perms back to ops/readonly -- [...truncated 18.35 MB...] traffic_portal_build_1 | | +-- path-is-absolute@1.0.1

[GitHub] asfgit commented on issue #2353: Add TO Go regions/name

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

[GitHub] asfgit commented on issue #2400: To go cdns sslkeys

asfgit commented on issue #2400: To go cdns sslkeys URL: https://github.com/apache/trafficcontrol/pull/2400#issuecomment-401869906 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1964/ Test PASSed.

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

See -- Started by upstream project "trafficcontrol-PR" build number 1964 originally caused by: GitHub pull request #2400 of commit 7e93215fbb2637b979a57768c4facaee2fdcb46e, no

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

See -- Started by upstream project "trafficcontrol-PR" build number 1969 originally caused by: GitHub pull request #2369 of commit 31eadad712e5278b5175cc005b796e87f6771c9e, no

[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

asfgit commented on issue #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401879116 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1969/ Test PASSed.

Build failed in Jenkins: trafficcontrol-PR #1971

See Changes: [mitchell852] testing passing contents of a file from the server to TP and prompting a -- GitHub pull request #2493 of commit

[GitHub] asfgit commented on issue #2493: testing passing contents of a file from the server to TP and promptin…

asfgit commented on issue #2493: testing passing contents of a file from the server to TP and promptin… URL: https://github.com/apache/trafficcontrol/pull/2493#issuecomment-401914171 Refer to this link for build results (access rights to CI server needed):

[GitHub] alficles commented on a change in pull request #2369: To go deletednsseckeys

alficles commented on a change in pull request #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369#discussion_r199563530 ## File path: traffic_ops/traffic_ops_golang/cdn/dnssec.go ## @@ -0,0 +1,58 @@ +package cdn + +/* + * Licensed to the

[GitHub] rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/trafficcontrol/issues/2232#issuecomment-398070296 # Table Usage ``` get /api/1.1/cdns/domains - DB: Profile, CDN get

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

See

[GitHub] asfgit commented on issue #2395: To go ds sslkeys generate

asfgit commented on issue #2395: To go ds sslkeys generate URL: https://github.com/apache/trafficcontrol/pull/2395#issuecomment-401870784 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1965/ Test PASSed.

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

See -- Started by upstream project "trafficcontrol-PR" build number 1965 originally caused by: GitHub pull request #2395 of commit b94fe3534a13f7677997f6bdc4bc6cd4a38b620d, no

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

See -- Started by upstream project "trafficcontrol-PR" build number 1970 originally caused by: GitHub pull request #2338 of commit 134bb62e4f865a77435592f602146e05576afabd, no

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

See Changes: [dewrich] Add TO Go delete deliveryservice_server [dewrich] Add TO Go delete ds_servers client funcs, tests -- [...truncated 1.91

[GitHub] rob05c commented on a change in pull request #2369: To go deletednsseckeys

rob05c commented on a change in pull request #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369#discussion_r199567779 ## File path: traffic_ops/traffic_ops_golang/cdn/dnssec.go ## @@ -0,0 +1,58 @@ +package cdn + +/* + * Licensed to the

[GitHub] asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete

asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete URL: https://github.com/apache/trafficcontrol/pull/2421#issuecomment-401876049 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1966/

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

See -- Started by upstream project "trafficcontrol-PR" build number 1966 originally caused by: GitHub pull request #2421 of commit b6f6be49d5759a811ba7737cdc3855523a742a69, no

[GitHub] dewrich closed pull request #2441: Add TO Go delete deliveryservice_server

dewrich closed pull request #2441: Add TO Go delete deliveryservice_server URL: https://github.com/apache/trafficcontrol/pull/2441 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

[GitHub] asfgit commented on issue #2475: Remove db from interface

asfgit commented on issue #2475: Remove db from interface URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401969608 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1980/ Test FAILed.

[GitHub] DylanVolz commented on issue #2475: Remove db from interface

DylanVolz commented on issue #2475: Remove db from interface URL: https://github.com/apache/trafficcontrol/pull/2475#issuecomment-401969216 retest this please This is an automated message from the Apache Git Service. To

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

See -- Started by upstream project "trafficcontrol-PR" build number 1968 originally caused by: GitHub pull request #2432 of commit e7454eb22b8fe2c3592052b9f67ef9da27ba36af, no

[GitHub] asfgit commented on issue #2432: Add TO Go deliveryservices urlkeys

asfgit commented on issue #2432: Add TO Go deliveryservices urlkeys URL: https://github.com/apache/trafficcontrol/pull/2432#issuecomment-401878332 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1968/ Test

[GitHub] DylanVolz opened a new pull request #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang

DylanVolz opened a new pull request #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang URL: https://github.com/apache/trafficcontrol/pull/2491 This is an automated message

[GitHub] rawlinp opened a new pull request #2492: Refactor cachegroups to use coordinates table

rawlinp opened a new pull request #2492: Refactor cachegroups to use coordinates table URL: https://github.com/apache/trafficcontrol/pull/2492 Replace cachegroup lat/long columns with a coordinate FK. On cg create, create a coordinate and set the coordinate ID in the cg. On cg update,

[GitHub] dewrich closed pull request #2233: Fixed the Go import package alignment because the github project moved

dewrich closed pull request #2233: Fixed the Go import package alignment because the github project moved URL: https://github.com/apache/trafficcontrol/pull/2233 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] asfgit commented on issue #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang

asfgit commented on issue #2491: add cpu profiling process, and memory profiling and db stats endpoints to traffic_ops_golang URL: https://github.com/apache/trafficcontrol/pull/2491#issuecomment-401860049 Refer to this link for build results (access rights to CI server needed):

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

See -- Started by upstream project "trafficcontrol-PR" build number 1962 originally caused by: GitHub pull request #2382 of commit 8257ecf8fb2501f6ed5a002e5a7662b8dba11965, no

[GitHub] asfgit commented on issue #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP]

asfgit commented on issue #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP] URL: https://github.com/apache/trafficcontrol/pull/2490#issuecomment-401847027 Can one of the admins verify this patch?

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

See

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

See -- Started by upstream project "trafficcontrol-PR" build number 1957 originally caused by: GitHub pull request #2358 of commit 92083f6ad0189a85a272c3fc861e64a5186fa757, no

[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

asfgit commented on issue #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401867312 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1961/ Test FAILed.

Build failed in Jenkins: trafficcontrol-PR #1961

See -- GitHub pull request #2369 of commit 03752ab61205d6422d1314188bd09f9708c66205, no merge conflicts. Setting status of 03752ab61205d6422d1314188bd09f9708c66205 to PENDING with url

[GitHub] dangogh commented on a change in pull request #2485: Modify the grove rpm version format

dangogh commented on a change in pull request #2485: Modify the grove rpm version format URL: https://github.com/apache/trafficcontrol/pull/2485#discussion_r199207507 ## File path: grove/build/build_rpm.sh ## @@ -19,6 +19,7 @@ cd "$ROOTDIR/grove"

[GitHub] rivasj opened a new pull request #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP]

rivasj opened a new pull request #2490: Update gen_iso endpoint to have option to stream bits instead of writing them to disk [WIP] URL: https://github.com/apache/trafficcontrol/pull/2490 This is an automated message from

[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-401848624 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1957/ Test

[GitHub] asfgit commented on issue #2492: Refactor cachegroups to use coordinates table

asfgit commented on issue #2492: Refactor cachegroups to use coordinates table URL: https://github.com/apache/trafficcontrol/pull/2492#issuecomment-401866456 Refer to this link for build results (access rights to CI server needed):

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

See -- Started by upstream project "trafficcontrol-PR" build number 1963 originally caused by: GitHub pull request #2369 of commit 2476da071419ecd136bfd0e5c388b8c495720799, no

[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

asfgit commented on issue #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369#issuecomment-401869032 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/1963/ Test PASSed.