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

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

[GitHub] dangogh closed pull request #2530: Fix Coordinate integration fixture

2018-07-11 Thread GitBox
dangogh closed pull request #2530: Fix Coordinate integration fixture URL: https://github.com/apache/trafficcontrol/pull/2530 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

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [dangogh] Fix Coordinate integration fixture -- Started by an SCM change [EnvInject] - Loading node environment variables. Building

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

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

[GitHub] rawlinp opened a new issue #2532: TO Go GET /server/{id}/deliveryservices endpoint missing fields from TO Perl version

2018-07-11 Thread GitBox
rawlinp opened a new issue #2532: TO Go GET /server/{id}/deliveryservices endpoint missing fields from TO Perl version URL: https://github.com/apache/trafficcontrol/issues/2532 Fields *missing* from TO Go version that are in the Perl version: ``` "anonymousBlockingEnabled"

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [mitchell852] Add TO Go cdns/name/sslkeys -- Started by an SCM change [EnvInject] - Loading node environment variables. Building

[GitHub] jrushford opened a new pull request #2533: Modify the grovetccfg rpm build script so that it uses the trafficcontrol versioning scheme.

2018-07-11 Thread GitBox
jrushford opened a new pull request #2533: Modify the grovetccfg rpm build script so that it uses the trafficcontrol versioning scheme. URL: https://github.com/apache/trafficcontrol/pull/2533 Update the grovetccfg build script to match the trafficcontrol versioning scheme.

[GitHub] mitchell852 closed issue #2483: GET /api/1.3/deliveryservices/{{id}}/servers/eligible returns integers as strings

2018-07-11 Thread GitBox
mitchell852 closed issue #2483: GET /api/1.3/deliveryservices/{{id}}/servers/eligible returns integers as strings URL: https://github.com/apache/trafficcontrol/issues/2483 This is an automated message from the Apache Git

[GitHub] mitchell852 closed pull request #2400: To go cdns sslkeys

2018-07-11 Thread GitBox
mitchell852 closed pull request #2400: To go cdns sslkeys URL: https://github.com/apache/trafficcontrol/pull/2400 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 request

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [rob05c] Fix compiler error where using tc.DSType is used as a string without a -- Started by an SCM change [EnvInject] - Loading

[GitHub] dangogh commented on issue #2522: ensure tenant set where needed; add not null constraint

2018-07-11 Thread GitBox
dangogh commented on issue #2522: ensure tenant set where needed; add not null constraint URL: https://github.com/apache/trafficcontrol/pull/2522#issuecomment-404257284 added WIP label as we need to evaluate this approach. Please do not merge.

[GitHub] dewrich opened a new pull request #2531: added staticdnsentry struct

2018-07-11 Thread GitBox
dewrich opened a new pull request #2531: added staticdnsentry struct URL: https://github.com/apache/trafficcontrol/pull/2531 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] dangogh closed pull request #2531: added staticdnsentry struct

2018-07-11 Thread GitBox
dangogh closed pull request #2531: added staticdnsentry struct URL: https://github.com/apache/trafficcontrol/pull/2531 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 pull request #2529: adds back menu item to view static dns entries for a ds. also hides t…

2018-07-11 Thread GitBox
dewrich closed pull request #2529: adds back menu item to view static dns entries for a ds. also hides t… URL: https://github.com/apache/trafficcontrol/pull/2529 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [dangogh] added staticdnsentry struct -- [...truncated 1.97 MB...] unit_1 | Parse errors: No plan found in TAP output

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

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

[GitHub] rob05c commented on a change in pull request #2513: Add TO Go plugin system

2018-07-11 Thread GitBox
rob05c commented on a change in pull request #2513: Add TO Go plugin system URL: https://github.com/apache/trafficcontrol/pull/2513#discussion_r201757739 ## File path: traffic_ops/traffic_ops_golang/plugin/hello_world.go ## @@ -0,0 +1,34 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2513: Add TO Go plugin system

2018-07-11 Thread GitBox
rob05c commented on a change in pull request #2513: Add TO Go plugin system URL: https://github.com/apache/trafficcontrol/pull/2513#discussion_r201757739 ## File path: traffic_ops/traffic_ops_golang/plugin/hello_world.go ## @@ -0,0 +1,34 @@ +package plugin + +/* +

[GitHub] mitchell852 closed pull request #2432: Add TO Go deliveryservices urlkeys

2018-07-11 Thread GitBox
mitchell852 closed pull request #2432: Add TO Go deliveryservices urlkeys URL: https://github.com/apache/trafficcontrol/pull/2432 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] mitchell852 commented on a change in pull request #2400: To go cdns sslkeys

2018-07-11 Thread GitBox
mitchell852 commented on a change in pull request #2400: To go cdns sslkeys URL: https://github.com/apache/trafficcontrol/pull/2400#discussion_r201762476 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -108,6 +108,7 @@ func Routes(d ServerData) ([]Route,

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [dangogh] Fix Coordinate integration fixture -- [...truncated 19.22 MB...] traffic_portal_build_1 | | +-- object-assign@3.0.0

[GitHub] jhg03a commented on a change in pull request #2513: Add TO Go plugin system

2018-07-11 Thread GitBox
jhg03a commented on a change in pull request #2513: Add TO Go plugin system URL: https://github.com/apache/trafficcontrol/pull/2513#discussion_r201751485 ## File path: traffic_ops/traffic_ops_golang/plugin/hello_world.go ## @@ -0,0 +1,34 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2513: Add TO Go plugin system

2018-07-11 Thread GitBox
rob05c commented on a change in pull request #2513: Add TO Go plugin system URL: https://github.com/apache/trafficcontrol/pull/2513#discussion_r201757739 ## File path: traffic_ops/traffic_ops_golang/plugin/hello_world.go ## @@ -0,0 +1,34 @@ +package plugin + +/* +

Jenkins build is back to normal : trafficcontrol-master-build #921

2018-07-11 Thread Apache Jenkins Server
See

[GitHub] asfgit commented on issue #2533: Modify the grovetccfg rpm build script so that it uses the trafficcontrol versioning scheme.

2018-07-11 Thread GitBox
asfgit commented on issue #2533: Modify the grovetccfg rpm build script so that it uses the trafficcontrol versioning scheme. URL: https://github.com/apache/trafficcontrol/pull/2533#issuecomment-404283581 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2535: TO Go GET /user/current should have 'roleName' not 'rolename'

2018-07-11 Thread GitBox
mitchell852 closed pull request #2535: TO Go GET /user/current should have 'roleName' not 'rolename' URL: https://github.com/apache/trafficcontrol/pull/2535 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

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [mitchell852] TO Go GET /user/current should have 'roleName' not 'rolename' -- Started by an SCM change [EnvInject] - Loading node

[GitHub] asfgit commented on issue #2434: Range Slice mode for the range_req_handler

2018-07-11 Thread GitBox
asfgit commented on issue #2434: Range Slice mode for the range_req_handler URL: https://github.com/apache/trafficcontrol/pull/2434#issuecomment-404325757 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/2085/

[GitHub] mitchell852 closed pull request #2369: To go deletednsseckeys

2018-07-11 Thread GitBox
mitchell852 closed pull request #2369: To go deletednsseckeys URL: https://github.com/apache/trafficcontrol/pull/2369 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

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

2018-07-11 Thread Apache Jenkins Server
See Changes: [mitchell852] Add TO Go cdns/name/dnsseckeys/delete -- Started by an SCM change [EnvInject] - Loading node environment variables.

[GitHub] ocket8888 commented on issue #2483: GET /api/1.3/deliveryservices/{{id}}/servers/eligible returns integers as strings

2018-07-11 Thread GitBox
ocket commented on issue #2483: GET /api/1.3/deliveryservices/{{id}}/servers/eligible returns integers as strings URL: https://github.com/apache/trafficcontrol/issues/2483#issuecomment-404333604 Oops, sorry, I got that notification while I was on vacation, and I forgot to do it :P

[GitHub] rawlinp opened a new pull request #2534: Fix DSR PUT handlers for status/assignments

2018-07-11 Thread GitBox
rawlinp opened a new pull request #2534: Fix DSR PUT handlers for status/assignments URL: https://github.com/apache/trafficcontrol/pull/2534 The handlers reassign the struct to the existing struct but now need to save the ReqInfo into the new struct so that it's not lost during

[GitHub] asfgit commented on issue #2535: TO Go GET /user/current should have 'roleName' not 'rolename'

2018-07-11 Thread GitBox
asfgit commented on issue #2535: TO Go GET /user/current should have 'roleName' not 'rolename' URL: https://github.com/apache/trafficcontrol/pull/2535#issuecomment-404346838 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2534: Fix DSR PUT handlers for status/assignments

2018-07-11 Thread GitBox
asfgit commented on issue #2534: Fix DSR PUT handlers for status/assignments URL: https://github.com/apache/trafficcontrol/pull/2534#issuecomment-404336771 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/2086/