[GitHub] incubator-trafficcontrol pull request #175: updates repo url in traffic port...

2017-01-10 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/175 updates repo url in traffic portal build instructions You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-trafficcontrol pull request #167: Refactor and cleanup the influxd...

2017-01-10 Thread sbogacz
Github user sbogacz commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/167#discussion_r95499191 --- Diff: traffic_stats/influxdb_tools/sync/sync_test.go --- @@ -0,0 +1,230 @@ +/* +Licensed to the Apache Software Foundation

[GitHub] incubator-trafficcontrol pull request #167: Refactor and cleanup the influxd...

2017-01-10 Thread PSUdaemon
Github user PSUdaemon commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/167#discussion_r95498458 --- Diff: traffic_stats/influxdb_tools/sync/sync_test.go --- @@ -0,0 +1,230 @@ +/* +Licensed to the Apache Software Foundation

[GitHub] incubator-trafficcontrol pull request #167: Refactor and cleanup the influxd...

2017-01-10 Thread sbogacz
Github user sbogacz commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/167#discussion_r95489426 --- Diff: traffic_stats/influxdb_tools/sync/sync_test.go --- @@ -0,0 +1,230 @@ +/* +Licensed to the Apache Software Foundation

[jira] [Assigned] (TC-95) POST/PUT api/deliveryservices does not create necessary location params

2017-01-10 Thread Jeremy Mitchell (JIRA)
[ https://issues.apache.org/jira/browse/TC-95?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jeremy Mitchell reassigned TC-95: - Assignee: Jeremy Mitchell > POST/PUT api/deliveryservices does not create necessary location params

[jira] [Commented] (TC-92) Add RPM build for goTM

2017-01-10 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TC-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15815970#comment-15815970 ] ASF GitHub Bot commented on TC-92: -- Github user dneuman64 commented on a diff in the pull request:

[jira] [Commented] (TC-92) Add RPM build for goTM

2017-01-10 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TC-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15815971#comment-15815971 ] ASF GitHub Bot commented on TC-92: -- Github user dneuman64 commented on a diff in the pull request:

[GitHub] incubator-trafficcontrol pull request #173: [TC-92] -- Add RPM build for goT...

2017-01-10 Thread dneuman64
Github user dneuman64 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/173#discussion_r95441968 --- Diff: traffic_monitor/experimental/traffic_monitor/srvhttp/srvhttp.go --- @@ -134,22 +136,29 @@ func ParametersStr(params

[GitHub] incubator-trafficcontrol pull request #173: [TC-92] -- Add RPM build for goT...

2017-01-10 Thread dneuman64
Github user dneuman64 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/173#discussion_r95441958 --- Diff: traffic_monitor/experimental/traffic_monitor/srvhttp/srvhttp.go --- @@ -134,22 +136,29 @@ func ParametersStr(params

[jira] [Created] (TC-94) Add Deliveryservice Regex CRUD capabilities

2017-01-10 Thread Jeremy Mitchell (JIRA)
Jeremy Mitchell created TC-94: - Summary: Add Deliveryservice Regex CRUD capabilities Key: TC-94 URL: https://issues.apache.org/jira/browse/TC-94 Project: Traffic Control Issue Type: Improvement

[GitHub] incubator-trafficcontrol issue #167: Refactor and cleanup the influxdb_tools...

2017-01-10 Thread sbogacz
Github user sbogacz commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/167 @rob05c Yeah, that's what I'm talking about. That functionality (and since greater versions of go1.6, the tooling as well) supports the _existence_ of a vendor folder, and knows to

[jira] [Commented] (TC-92) Add RPM build for goTM

2017-01-10 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TC-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15815681#comment-15815681 ] ASF GitHub Bot commented on TC-92: -- Github user rob05c commented on a diff in the pull request:

[GitHub] incubator-trafficcontrol pull request #173: [TC-92] -- Add RPM build for goT...

2017-01-10 Thread rob05c
Github user rob05c commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/173#discussion_r95421273 --- Diff: traffic_monitor/experimental/traffic_monitor/srvhttp/srvhttp.go --- @@ -134,22 +136,29 @@ func ParametersStr(params url.Values)

[GitHub] incubator-trafficcontrol pull request #173: [TC-92] -- Add RPM build for goT...

2017-01-10 Thread rob05c
Github user rob05c commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/173#discussion_r95421340 --- Diff: traffic_monitor/experimental/traffic_monitor/srvhttp/srvhttp.go --- @@ -134,22 +136,29 @@ func ParametersStr(params url.Values)

[GitHub] incubator-trafficcontrol pull request #172: [TC-91] TM2 - Add support for pa...

2017-01-10 Thread rob05c
Github user rob05c commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/172#discussion_r95404995 --- Diff: traffic_monitor/experimental/traffic_monitor/manager/datarequest.go --- @@ -198,6 +204,16 @@ func NewCacheStatFilter(params

[jira] [Commented] (TC-91) Add support for path based filtering to relevant legacy APIs

2017-01-10 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TC-91?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15815434#comment-15815434 ] ASF GitHub Bot commented on TC-91: -- Github user asfgit closed the pull request at:

[GitHub] incubator-trafficcontrol pull request #172: [TC-91] TM2 - Add support for pa...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/172 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] incubator-trafficcontrol issue #167: Refactor and cleanup the influxdb_tools...

2017-01-10 Thread rob05c
Github user rob05c commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/167 +1 on built-in vendoring, -1 on godep unless there's a compelling reason. I'd also be +1 on Makefiles iff there's a compelling reason for them. Go build tools are minimal,