You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by rob05c <gi...@git.apache.org> on 2017/01/03 18:00:09 UTC
[GitHub] incubator-trafficcontrol pull request #151: Add experimental Go ATS config
GitHub user rob05c opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/151
Add experimental Go ATS config
Adds experimental Go ATS config generator, as a library which can be used as a CLI app or HTTP service.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rob05c/incubator-trafficcontrol atsconfig
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-trafficcontrol/pull/151.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #151
----
commit 7b4958c110134912eb5ea2886a65e43efe580289
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-08T21:43:50Z
Add TM2 cache.PrecomputeTest
commit 5c1210a01ec8d32127e48002f460b19e82a8d563
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-08T22:24:44Z
Add TM2 cache.TestStatsMarshall
commit f9827e534ef74bcd867d82e66daa1a598af5b23f
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-08T22:44:42Z
Change TM2 to reduce CPU usage
Changes Traffic Monitor 2.0 pollers to not lock the goroutine to the
OS thread, changes pollers to sleep for 100ms when the poll
queue is empty (and amortize back to the poll interval by reducing
the set interval by the same amount), changes GOMAXPROCS from 3x to
1x the core count, with the aim of reducing CPU usage.
commit 2c4d3f45339c71045d3a2e25e5ef7afed0a9e08b
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-09T22:05:22Z
Add TM2 fetch error ID and URL info
commit e70dd9f66bfaae5e418e139d6d5f96baec3711cb
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-09T22:05:41Z
Add TM2 enum EDGE_VECTOR
commit d5b0850f0cb4017de3f291e5853407de32ff2765
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-09T22:06:21Z
Add TM2 stat and health history zero check
This prevents an out-of-range trying to prune the history to 0.
commit 3bb2ba211648411fdc8979755fb2155f575eb68e
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-09T22:42:48Z
Fix TM2 /publish/stats to strings to match TM1
commit f812a62cee10ec93cfb837b5256d307ae338d951
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-22T23:06:42Z
Add TM2 towrap URL, User, Servers, Parameters
commit 31b496eb1d44b4466ed176eda6d8cb4df4761762
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-22T23:12:48Z
Change experimental to/ats_config to use to/client
commit 1b31dfa0891b9dbb6981f391c41fecb42b7a190f
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-22T23:15:34Z
Move experimental ats_config apps to /cli /service
commit 518ad68960be0e20f6210f8214c448f4ac3b515f
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-22T23:51:35Z
Add experimental ats_config volume.config
commit 3161bc4501ef5a39c0bd68acc9d0f14aa30604b1
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-23T00:24:43Z
Add experimental ats_config logs_xml.config
commit 970c61dce9997b121cb168dc0fc10dd360ad805f
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-23T21:02:19Z
Add TM2 towrap.DeliveryServices()
commit 6beb68be62ba772107bf5e97d0ef59a99538a2df
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-23T21:04:55Z
Add experimental ats_config cacheurl.config
Changes config file dispatch to a regex map, necessary for
`cacheurl_(.*).config` files.
commit dd2e1d2a840c2d6a51e59b0429e7c0a1c04083d3
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-23T21:09:26Z
Add experimental ats_config storage.config file
Moves storage.config creator func to its own file.
commit 4e6b3c51162f1b686d5b3b9cf07bdf59b6c33183
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-24T00:41:31Z
Add experimental ats_config generic, hosting cfgs
commit c85d4c9e74d4c58abe7075fed9c6456b02000827
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-24T02:56:58Z
Add experimental ats_config 50-ats.rules
commit 8fcf7fd8abfaaac12ae50006447d53d351f9e375
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-24T03:26:14Z
Add experimental ats_config cache.config
commit cb3f08e66863d411c078d08007dcea784c58397d
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-26T04:06:38Z
Add experimental ats_config remap.config
commit a7267598b0be314c411f0b4ae777d5c933209568
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-27T00:35:10Z
Add experimental ats_config parent.config
commit 2a2ba6677cf1731aee098269d198d1b71f98c6a6
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-27T06:03:35Z
Add experimental ats_config ip_allow.config
commit ab93563ba1c30a3195f343f705683cea6362e364
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-27T20:48:59Z
Add experimental ats_config 12M_facts
commit 1183fba518f16aa5980eb1707e073795e29da531
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-27T21:35:57Z
Add experimental ats_config drop_qstring.config
commit 7a9c44f6ead9aba593ba4f9807b5948c9e8d3b12
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-27T21:52:53Z
Add experimental ats_config bg_fetch.config
commit afddc0a8d1d6a4b55c682739a4de2ef01e044cbf
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-28T00:19:00Z
Add experimental ats_config url_sig.config
commit f5a0ebeb88fae0a5b0cca2c0695941a9423b1f1e
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-28T00:50:25Z
Add experimental ats_config hdr_rw_.config
commit 7f769d52e00adc7b0437b382287ab10a1c6f77b2
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-28T00:58:26Z
Add experimental ats_config set_dscp_.config
commit 07c20d882621d5595e2200e848ba0a002c678d8b
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-28T04:29:43Z
Add experimental ats_config regex_remap.config
commit 9b8b69ed11a81d617e0a606f30aec73e92b9b9a7
Author: Robert Butts <ro...@gmail.com>
Date: 2016-12-28T05:15:46Z
Add experimental ats_config ssl_multicert.config
----
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #151: Add experimental Go ATS config
Posted by rob05c <gi...@git.apache.org>.
Github user rob05c commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/151
@knutsel Added.
I'd still like to see it merged, yes. Separating ATS configuration into its own app is both a step toward microservices and away from monoliths, and a step toward decoupling Traffic Ops and our cache architecture.
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol pull request #151: Add experimental Go ATS config
Posted by rob05c <gi...@git.apache.org>.
Github user rob05c commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/151#discussion_r94869522
--- Diff: traffic_monitor/experimental/traffic_monitor/enum/enum.go ---
@@ -44,6 +43,8 @@ type CacheType string
const (
// CacheTypeEdge represents an edge cache.
CacheTypeEdge = CacheType("EDGE")
+ // CacheTypeEdgeVector represents a vector (teak) cache.
+ CacheTypeEdgeVector = CacheType("EDGE_VECTOR")
--- End diff --
This has to be removed before it can be merged, Vector is implementation-specific.
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #151: Add experimental Go ATS config
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/151
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/34/
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #151: Add experimental Go ATS config
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/151
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-traffic_ops-test-PR/19/
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #151: Add experimental Go ATS config
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/151
Can one of the admins verify this patch?
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-trafficcontrol issue #151: Add experimental Go ATS config
Posted by knutsel <gi...@git.apache.org>.
Github user knutsel commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/151
@rob05c - should we still merge this? If so, can you add the Apache license?
---
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---