[GitHub] asfgit commented on issue #2103: Add a Location API

2018-04-13 Thread GitBox
asfgit commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-381267629 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1419/

[GitHub] asfgit commented on issue #2136: Properly unmarshal null DeepCachingType value as "NEVER"

2018-04-13 Thread GitBox
asfgit commented on issue #2136: Properly unmarshal null DeepCachingType value as "NEVER" URL: https://github.com/apache/incubator-trafficcontrol/pull/2136#issuecomment-381263781 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 opened a new pull request #2137: adds chart link and goto link (only for rascal servers) to each serve…

2018-04-13 Thread GitBox
mitchell852 opened a new pull request #2137: adds chart link and goto link (only for rascal servers) to each serve… URL: https://github.com/apache/incubator-trafficcontrol/pull/2137 …r table fixes #1946 This is an

[GitHub] dangogh closed pull request #2134: TP - fixes possible null pointer exception

2018-04-13 Thread GitBox
dangogh closed pull request #2134: TP - fixes possible null pointer exception URL: https://github.com/apache/incubator-trafficcontrol/pull/2134 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

[GitHub] mitchell852 commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mitchell852 commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381217520 I struggled with how to organize these groups and the items in them so i defaulted to alphabetical. With the exception of Dashboard

[GitHub] asfgit commented on issue #2134: TP - fixes possible null pointer exception

2018-04-13 Thread GitBox
asfgit commented on issue #2134: TP - fixes possible null pointer exception URL: https://github.com/apache/incubator-trafficcontrol/pull/2134#issuecomment-381246723 Refer to this link for build results (access rights to CI server needed):

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

2018-04-13 Thread Apache Jenkins Server
See Changes: [dewrich] adds external links to drop down values in TP forms -- Started by an SCM change Started by an SCM change Started by an SCM

[GitHub] rawlinp opened a new pull request #2136: Properly unmarshal null DeepCachingType value as "NEVER"

2018-04-13 Thread GitBox
rawlinp opened a new pull request #2136: Properly unmarshal null DeepCachingType value as "NEVER" URL: https://github.com/apache/incubator-trafficcontrol/pull/2136 If the client received DeliveryService JSON that contained a `null` value in `deepCachingType`, it was unable to unmarshal

[GitHub] asfgit commented on issue #2135: TP - adds external links to drop down values in TP forms

2018-04-13 Thread GitBox
asfgit commented on issue #2135: TP - adds external links to drop down values in TP forms URL: https://github.com/apache/incubator-trafficcontrol/pull/2135#issuecomment-381255502 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed issue #1298: All static dns entries are displayed when viewing for a delivery service

2018-04-13 Thread GitBox
mitchell852 closed issue #1298: All static dns entries are displayed when viewing for a delivery service URL: https://github.com/apache/incubator-trafficcontrol/issues/1298 This is an automated message from the Apache Git

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

2018-04-13 Thread Apache Jenkins Server
See

[GitHub] dewrich closed pull request #2135: TP - adds external links to drop down values in TP forms

2018-04-13 Thread GitBox
dewrich closed pull request #2135: TP - adds external links to drop down values in TP forms URL: https://github.com/apache/incubator-trafficcontrol/pull/2135 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] dewrich closed issue #1885: Location detail of server missing on traffic-portal from Physical location tab in server information

2018-04-13 Thread GitBox
dewrich closed issue #1885: Location detail of server missing on traffic-portal from Physical location tab in server information URL: https://github.com/apache/incubator-trafficcontrol/issues/1885 This is an automated

[GitHub] mitchell852 opened a new pull request #2135: TP - adds external links to drop down values in TP forms

2018-04-13 Thread GitBox
mitchell852 opened a new pull request #2135: TP - adds external links to drop down values in TP forms URL: https://github.com/apache/incubator-trafficcontrol/pull/2135 fixes #1885 This is an automated message from the

[GitHub] asfgit commented on issue #2131: Fix org_server_fqdn trailing slash issue

2018-04-13 Thread GitBox
asfgit commented on issue #2131: Fix org_server_fqdn trailing slash issue URL: https://github.com/apache/incubator-trafficcontrol/pull/2131#issuecomment-381236214 Refer to this link for build results (access rights to CI server needed):

[GitHub] limited commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
limited commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381216418 Definitely and improvement. The "configuration" tab seems like it would be used less frequently (or at least not first by a new user).

[GitHub] asfgit commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
asfgit commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381218443 Refer to this link for build results (access rights to CI server needed):

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

2018-04-13 Thread Apache Jenkins Server
See Changes: [dangogh] fixes possible null pointer exception -- Started by an SCM change Started by an SCM change Started by an SCM change Started

[GitHub] mitchell852 commented on issue #1885: Location detail of server missing on traffic-portal from Physical location tab in server information

2018-04-13 Thread GitBox
mitchell852 commented on issue #1885: Location detail of server missing on traffic-portal from Physical location tab in server information URL: https://github.com/apache/incubator-trafficcontrol/issues/1885#issuecomment-381233386 @tyagian - see the example on the PR attached to this

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

2018-04-13 Thread Apache Jenkins Server
See

[GitHub] asfgit commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
asfgit commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381195860 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 commented on issue #1301: Manage delivery service static dns entries

2018-04-13 Thread GitBox
mitchell852 commented on issue #1301: Manage delivery service static dns entries URL: https://github.com/apache/incubator-trafficcontrol/issues/1301#issuecomment-381237281 @rawlinp - the reason i added the 'View Static DNS entries' menu item to ds, cachegroup and type (where

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

2018-04-13 Thread Apache Jenkins Server
See Changes: [jeffrey_elsloo] Fix org_server_fqdn trailing slash issue -- Started by an SCM change Started by an SCM change Started by an SCM

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

2018-04-13 Thread Apache Jenkins Server
See -- GitHub pull request #2131 of commit 325533e7a8bac34491685646d34ec6e863c5c33e, no merge conflicts. [EnvInject] - Loading node environment variables. Building remotely

[GitHub] asfgit commented on issue #2131: Fix org_server_fqdn trailing slash issue

2018-04-13 Thread GitBox
asfgit commented on issue #2131: Fix org_server_fqdn trailing slash issue URL: https://github.com/apache/incubator-trafficcontrol/pull/2131#issuecomment-381207518 Refer to this link for build results (access rights to CI server needed):

[GitHub] tyagian commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
tyagian commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381217074 New sidebar looks good. Do you think `servers` can be in `Configure`. It is used more frequently so handy if keep on top drop-down menu

[GitHub] mitchell852 opened a new pull request #2134: TP - fixes possible null pointer exception

2018-04-13 Thread GitBox
mitchell852 opened a new pull request #2134: TP - fixes possible null pointer exception URL: https://github.com/apache/incubator-trafficcontrol/pull/2134 This is an automated message from the Apache Git Service. To respond

[GitHub] mitchell852 commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mitchell852 commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381217520 I struggled with how to organize these groups and the items in them so i defaulted to alphabetical

[GitHub] mitchell852 commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mitchell852 commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381217520 I struggled with how to organize these groups and the items in them so i defaulted to alphabetical. With the exception of Dashboard

[GitHub] asfgit commented on issue #2137: adds chart link and goto link (only for rascal servers) to each serve…

2018-04-13 Thread GitBox
asfgit commented on issue #2137: adds chart link and goto link (only for rascal servers) to each serve… URL: https://github.com/apache/incubator-trafficcontrol/pull/2137#issuecomment-381275817 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich closed pull request #2130: Traffic Ops - golang API servers CRUD tests

2018-04-13 Thread GitBox
dewrich closed pull request #2130: Traffic Ops - golang API servers CRUD tests URL: https://github.com/apache/incubator-trafficcontrol/pull/2130 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

[GitHub] asfgit commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
asfgit commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381170560 Refer to this link for build results (access rights to CI server needed):

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

2018-04-13 Thread Apache Jenkins Server
See -- GitHub pull request #2132 of commit 67bfd1a598407f1c56e7c91918ecb26a297a331b, no merge conflicts. PR merge status couldn't be retrieved, maybe GitHub hasn't settled

[GitHub] limited commented on issue #2130: Traffic Ops - golang API servers CRUD tests

2018-04-13 Thread GitBox
limited commented on issue #2130: Traffic Ops - golang API servers CRUD tests URL: https://github.com/apache/incubator-trafficcontrol/pull/2130#issuecomment-381114758 test this please This is an automated message from the

[GitHub] DylanVolz commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-04-13 Thread GitBox
DylanVolz commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-381161255 @mitchell852 I think this looks good now, I verified it works as expected. The only outstanding issue is the merging

[GitHub] mitchell852 opened a new pull request #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mitchell852 opened a new pull request #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #2130: Traffic Ops - golang API servers CRUD tests

2018-04-13 Thread GitBox
asfgit commented on issue #2130: Traffic Ops - golang API servers CRUD tests URL: https://github.com/apache/incubator-trafficcontrol/pull/2130#issuecomment-381134315 Refer to this link for build results (access rights to CI server needed):

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

2018-04-13 Thread Apache Jenkins Server
See Changes: [dewrich] Added golang API CRUD tests for servers [dewrich] Updated golang API servers tests [dewrich] golang API servers test code cleanup

[GitHub] mtorluemke commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mtorluemke commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381182086 I think the new structure looks great. This is an automated message

[GitHub] dg4prez commented on issue #1546: deliveryservice.multi_site_origin_algorithm column isn't accessible via any UI

2018-04-13 Thread GitBox
dg4prez commented on issue #1546: deliveryservice.multi_site_origin_algorithm column isn't accessible via any UI URL: https://github.com/apache/incubator-trafficcontrol/issues/1546#issuecomment-381182353 This is set in the DS profile now, so it doesn't need to have a column.

[GitHub] mitchell852 commented on issue #2132: better grouping for TP menu

2018-04-13 Thread GitBox
mitchell852 commented on issue #2132: better grouping for TP menu URL: https://github.com/apache/incubator-trafficcontrol/pull/2132#issuecomment-381183510 retest this please This is an automated message from the Apache Git

[GitHub] dg4prez opened a new issue #2133: single-layer CDNs need MSO functionality

2018-04-13 Thread GitBox
dg4prez opened a new issue #2133: single-layer CDNs need MSO functionality URL: https://github.com/apache/incubator-trafficcontrol/issues/2133 Currently, there is no way for a single-layer CDN to use a multisite configuration. This is something we should add.

[GitHub] elsloo closed pull request #2131: Fix org_server_fqdn trailing slash issue

2018-04-13 Thread GitBox
elsloo closed pull request #2131: Fix org_server_fqdn trailing slash issue URL: https://github.com/apache/incubator-trafficcontrol/pull/2131 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] elsloo closed issue #2062: Trailing slash in DS origin URL breaks MSO parent.config

2018-04-13 Thread GitBox
elsloo closed issue #2062: Trailing slash in DS origin URL breaks MSO parent.config URL: https://github.com/apache/incubator-trafficcontrol/issues/2062 This is an automated message from the Apache Git Service. To respond to