Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/813
retest this please
---
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
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/821
@mitchell852 Please close. Thanks.
---
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
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/830
[TC-514] ORT: Use CLI provided traffic_ops_hosts
- Script was fetching tm.url from GLOBAL profile. If the URL was different
(multiple TO for example), it was failing on cookies
Github user smalenfant closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/797
---
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
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/813
@dg4prez I'll conduct more testing here but I'd like you to review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/813
[TC-353] Can't use CNAME hostname with "ccr" prefix
- Replaced regex from ccr to http_routing
You can merge this pull request into a Git repository by running:
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/804
CDU/CHR Checks: Formatting float as integer
- Convertion was auto-applied by MySQL, but not with Postgresql resulting
in errors in logs
You can merge this pull request into a
Github user smalenfant closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/318
---
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
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/318
This is already merged from another PR. Closing.
---
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
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/803
Update master version to 2.2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/smalenfant/incubator-trafficcontrol
update
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/797
[BACKPORT][TC-511] Remove reference to public database
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/smalenfant
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/796
[TC-511] Remove reference to public database
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/smalenfant/incubator
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/616
Centos 7.3
---
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
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/616
@elsloo Tested on Centos 6 (single interface) and Centos 7 (single and
bond). All looks good. :shipit:
---
If your project is set up for it, you can reply to this email and
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/616
That's what I was thinking about first time I looked at this. Not sure why
we needed to iterate all the slaves. We can put this through the test, but
it looks good
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/319
[TC-172] Add CURLOPT_FOLLOWLOCATION
Note: Not setting a limit and it is unlimited by default
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/318
[TC-90] Recovered changes for @elsloo
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/smalenfant/incubator-trafficcontrol
GitHub user smalenfant reopened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/281
[TC-132] Fix proper RP for sync
This is still very static, but does what is intended to do.
- daily_stats: Put data in indefinite instead of DEFAULT
Github user smalenfant closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/281
---
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
Github user smalenfant commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/281#discussion_r101363315
--- Diff: traffic_stats/influxdb_tools/sync/sync_ts_databases.go ---
@@ -176,16 +177,24 @@ func syncDailyDb(ch chan string
Github user smalenfant commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/281#discussion_r101362585
--- Diff: traffic_stats/influxdb_tools/sync/sync_ts_databases.go ---
@@ -298,8 +307,9 @@ func syncDailyStat(sourceClient influx.Client
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/281
[TC-132] Fix proper RP for sync
This is still very static, but does what is intended to do.
- daily_stats: Put data in indefinite instead of DEFAULT
GitHub user smalenfant opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/279
[TC-145] Add defines to compile against multiple version of TS
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/smalenfant
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/256
There is a possibility that you might have some patches against your 5.3.x
that might make this work. But it won't compile against 5.3.2 provided by
Traffic Server.
-
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/256
@elsloo Sorry I couldn't get to this before, but it still fails compiling
on 5.3.2. Please look at the PR I've made against yours to prevents this.
```
Github user smalenfant commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/256
@elsloo I haven't tried but I believe it will fail on... Have you taken a
look at the PR I made against yours?
```
if (TSPluginRegister(&info) != T
26 matches
Mail list logo