[GitHub] incubator-trafficcontrol issue #754: Configure session tracking on TO side

2017-08-16 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/754
  
There's 2 fields- a boolean and a list of the keys to use from the cookie. 
The list is expected to be short, just a few entries long. No session metadata 
is stored in the TO DB. 

I agree we need better approach to DS options, but I don't think this is 
the PR to hinge things on. 








---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #816: Fix doc syntax for commit fe0029...

2017-08-15 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/816

Fix doc syntax for commit  fe0029399408c27341ff74a3e63fe2330e0b905a



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol doc_fix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/816.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 #816


commit cc24843de36d6de5ed22d00803603bd546a456ac
Author: Eric Friedrich 
Date:   2017-08-15T17:47:17Z

Fix doc syntax for commit  fe0029399408c27341ff74a3e63fe2330e0b905a




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #802: [TC-187] fix Traffic Ops UT

2017-08-14 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/802
  
test 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #781: WIP

2017-08-07 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/781
  
ok to test


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #754: Configure session tracking on TO side

2017-07-26 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/754
  
Thanks John-
  The diff is quite large, mostly because of whitespace and some formatting 
related changes. 

Can you please try to minimize the size of the diff to just changes 
directly related to the Session Tracking code so that its easier to review? 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #715: Update README.md for TC 2.0.0 re...

2017-07-06 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/715

Update README.md for TC 2.0.0 release



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol patch-2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/715.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 #715


commit c3ecfd127cc6bc72059005c13cf9b81a94712aff
Author: Eric Friedrich 
Date:   2017-07-07T01:41:17Z

Update README.md for TC 2.0.0 release




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-20 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/567
  
Thanks Amir-
Perfect sense! Just to make sure I understood correctly, we will keep 
both the auth server and API gateway as optional components that could be 
switched out for production deployments if desired?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-20 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/567
  
This look pretty nice Amir-
  Have you considered using an open source proxy like nginx or varnish in 
place of writing our own? Both pieces of software can almost certainly validate 
JWT and perform remaps. They have the advantage of already providing support 
for a wide range of HTTP transformations that webfront doesn't yet have. For 
example, I might like to use this proxy in front of grafana to perform 
authentication (based on setting some HTTP request headers). With nginx or 
varnish, I could easily add those headers. I'm sure we'll have many other use 
cases arise where we'd want to continously add more functionality to the API GW 
proxy. 

Also, those other open source proxies are much better tested and have many 
more miles on their tires- they could be a quick path to a very stable 
component here. 

Are there certain reasons why we prefer writing our own proxy to using one 
of several alternatives?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #686: [Backport TC-370] Address IPMC Comments

2017-06-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/686
  
merged


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #681: [TC-370] Address IPMC Comments

2017-06-15 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/681


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #682: [TC-370] Address IPMC Comments

2017-06-15 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/682

[TC-370] Address IPMC Comments

(cherry picked from commit 9ce7aa92f6ba8e78506e6bb080f6829df4a26f77)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol 
tc370-ipmc-comments

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/682.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 #682


commit e343d0f91df11f5c6b1f871d48712fa7bfdb66fc
Author: Eric Friedrich 
Date:   2017-06-15T14:54:50Z

[TC-370] Address IPMC Comments

(cherry picked from commit 9ce7aa92f6ba8e78506e6bb080f6829df4a26f77)




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #681: [TC-370] Address IPMC Comments

2017-06-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/681
  
@dangogh 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #681: [TC-370] Address IPMC Comments

2017-06-15 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/681

[TC-370] Address IPMC Comments



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol 
2.0.x-ipmc-comments

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/681.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 #681


commit 9ce7aa92f6ba8e78506e6bb080f6829df4a26f77
Author: Eric Friedrich 
Date:   2017-06-15T14:54:50Z

[TC-370] Address IPMC Comments




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #680: [TC-381] postinstall errors if "secrets...

2017-06-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/680
  
merged


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #679: [Backport] updates TO install docs; add...

2017-06-14 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/679
  
merged


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #674: [BACKPORT] Introduces a new top-level T...

2017-06-14 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/674
  
merged


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #671: [Backport TC-356] update .rat-excludes ...

2017-06-13 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/671
  
merged. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #634: [TC-361] Added a read timeout to TCP DN...

2017-05-31 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/634
  
merged!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #630: [BACKPORT TC-352] upgrade jdk do...

2017-05-31 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/630

[BACKPORT TC-352] upgrade jdk download for tm/tr build

(cherry picked from commit 3c79673851a1c3a7e8349b49673184ba853f1dcb)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol tc-352-1.8.x

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/630.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 #630


commit cc2c900e23d507e7e34f2f5d9d14aeb66419cb15
Author: Dan Kirkwood 
Date:   2017-05-23T16:03:52Z

upgrade jdk download for tm/tr build

(cherry picked from commit 3c79673851a1c3a7e8349b49673184ba853f1dcb)




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #626: [Backport TC-228] postinstall improveme...

2017-05-31 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/626
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #619: Cherry pick for TO HTML Fix

2017-05-26 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/619
  
merged please close



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #611: BACKPORT: [TC-200] - The CCR DNS TTL fi...

2017-05-24 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/611
  
merged


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #603: [Backport TC-352] updates jdk version f...

2017-05-23 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/603
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #600: [Backport TC-349] update 2.0.x build sc...

2017-05-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/600
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #599: [Backport] license fixes to be backport...

2017-05-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/599
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #594: [Backport TC-345] source tarball has in...

2017-05-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/594
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #30: Add --skip-ssl option to post ins...

2017-05-19 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/30


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #28: [WIP] Travis ci

2017-05-19 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/28


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #583: [Backport TC-340] Remove TS unnecessary...

2017-05-19 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/583
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #532: [Backport TC-167] 2.0.x fix cachegroupp...

2017-05-19 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/532
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #531: [Backport] remove KEYS -- stored in dis...

2017-05-19 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/531
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #530: [Backport TC-150] 2.0 document cert gen...

2017-05-19 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/530
  
merged to 2.0.x, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #575: Add Jenkins Build Status icon to README...

2017-05-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/575
  
ok to test


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #575: Add Jenkins Build Status icon to...

2017-05-15 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/575

Add Jenkins Build Status icon to README.md



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol jenkins-icon

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/575.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 #575


commit 6f11eef16ceaf4c89a8b6a670372a29ced6cb589
Author: Eric Friedrich 
Date:   2017-05-15T16:07:18Z

Add Jenkins Build Status icon




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #574: Add Jenkins status icon

2017-05-15 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/574


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #574: Add Jenkins status icon

2017-05-15 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/574

Add Jenkins status icon

Add Jenkins build status icon to README.md

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol 
jenkins-status-icon

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/574.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 #574


commit 9e8e133a220411aa9231b6887ecbc24b2ffcdd4b
Author: Eric Friedrich 
Date:   2017-04-25T02:02:35Z

Adding KEYS required for release

(cherry picked from commit b3885f73e4c518b45368ecbdb9d730826b2bad06)

commit 0937f95701149c7f54c7cedb8e10505e55ca2879
Author: Eric Friedrich 
Date:   2017-05-15T15:56:41Z

Update README.md




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #497: [TC-262] Allow specification of the 'id...

2017-05-08 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/497
  

I understand the ATS storage.config parameter, but how is this ID 
configured/generated in Traffic Ops? Is there a per-disk ID parameter (i.e. if 
i have 72 disks there are now 72 new parameters)?

I looked at the referenced PR but it didnt seem to contain API changes




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #497: [TC-262] Allow specification of the 'id...

2017-04-29 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/497
  
What values would one typically set for the  *_Hash_Id? Does it need to 
correspond to a property of the disks?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #523: BACKPORT - fix fork_and_daemonize to co...

2017-04-28 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/523
  
merged to 2.0.x



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #347: Add global build script.

2017-04-26 Thread limited
Github user limited commented on a diff in the pull request:


https://github.com/apache/incubator-trafficcontrol/pull/347#discussion_r113432531
  
--- Diff: infrastructure/docker/build/clean_build.sh ---
@@ -0,0 +1,8 @@
+#!/usr/bin/env sh
--- End diff --

Needs an AL2 license header


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #347: Add global build script.

2017-04-26 Thread limited
Github user limited commented on a diff in the pull request:


https://github.com/apache/incubator-trafficcontrol/pull/347#discussion_r113432482
  
--- Diff: pkg ---
@@ -0,0 +1,132 @@
+#!/usr/bin/env bash
--- End diff --

This file needs a license header


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #508: Adding KEYS required for release

2017-04-25 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/508


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #508: Adding KEYS required for release

2017-04-24 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/508

Adding KEYS required for release

(cherry picked from commit b3885f73e4c518b45368ecbdb9d730826b2bad06)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/508.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 #508


commit 9e8e133a220411aa9231b6887ecbc24b2ffcdd4b
Author: Eric Friedrich 
Date:   2017-04-25T02:02:35Z

Adding KEYS required for release

(cherry picked from commit b3885f73e4c518b45368ecbdb9d730826b2bad06)




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #500: [BACKPORT TC-243] 2.0.x postinstall rem...

2017-04-23 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/500
  
Merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #490: [BACKPORT TC-228] collective changes to...

2017-04-20 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/490
  
merged to 2.0.x



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #469: [BACKPORT TC-223] - default regex is re...

2017-04-14 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/469
  
merged to 2.0.x please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #410: [BACKPORT] add login dispersion

2017-04-10 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/410
  
Sorry for delay, this is now merged to 2.0.x


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #443: Remove TO client ResumeSession, remove ...

2017-04-10 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/443
  
Merged to 2.0.x, please close this PR and mark the Jira as resolved


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #410: add login dispersion

2017-03-27 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/410
  
looks low risk and useful enough so unless someone objects we can merge it 
into 2.0.x


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #395: [BACKPORT] - Change TO monitoring.json ...

2017-03-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/395
  
merged to 2.0.x

please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #394: Stats performance 2.0.x

2017-03-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/394
  
merged to 2.0.x. 

please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #388: [BACKPORT] fixed URL to fancybox v2.1.5...

2017-03-22 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/388
  
merged to 2.0.x, please close the PR


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #376: [BACKPORT] fixed issue where the docker...

2017-03-16 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/376
  
merged to 2.0.x, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #374: [BACKPORT] - adds back exampleurls to d...

2017-03-16 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/374
  
merged to 2.0.x, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #331: Fixing typo in traffic_router/build/pom...

2017-03-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/331
  
@hbeatty Can you please close this PR?



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #368: [BACKPORT] - fixes regression - unable ...

2017-03-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/368
  
merged to 2.0.x, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #362: [BACKPORT] [TC-189] - fixes 2 missLatts...

2017-03-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/362
  
merged to 2.0.x please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #358: [BACKPORT] -- This fixes a potential nu...

2017-03-15 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/358
  
merged, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #28: [WIP] Travis ci

2017-03-12 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/28
  
Probably need to revisit based on latest dockerbuild stuff. 

However, I was holding up on this because I thought someone, maybe 
@dneuman64 was looking at other CI options? 

I'll send a question out to dev@ and see if we can finally get something 
public setup.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #338: [TC-72] BACKPORT Dockerfile build local

2017-03-10 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/338
  
merged to 2.0.x!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #333: [TC-176] fix minor typo in --...

2017-03-10 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/333
  
Merged this in #331, please close this one


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #331: Fixing typo in traffic_router/build/pom...

2017-03-10 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/331
  
Merged this to 2.0.x


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #342: [BACKPORT] of PR #317

2017-03-09 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/342
  
fix merged to 2.0.x, please close


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #327: [TC-72] Dockerfile build local

2017-03-07 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/327
  
noted, will see how the merge looks. probably a go


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #333: [TC-176] fix minor typo in --...

2017-03-06 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/333
  
thanks for catching this, will merge into 2.0.x



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #304: Updates version number to 2.1

2017-02-23 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/304

Updates version number to 2.1



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol 
2.1_version_fix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/304.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 #304


commit 4882553612fed1f517b903845d11731bc8b9c0ba
Author: Eric Friedrich 
Date:   2017-02-24T02:05:29Z

Updates to version number to 2.1




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #301: TC-159 Remove AL2 from traffic_m...

2017-02-22 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/301

TC-159 Remove AL2 from traffic_monitor_config.js

JSON files do not support comments. 

This PR does not need to be merged unless another 1.8 RC will be cut

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol TC-159

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/301.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 #301


commit b445dad9e38672dbf2e8c09eba4cbe2214f740f5
Author: Eric Friedrich 
Date:   2017-02-22T19:43:49Z

TC-159: Remove AL2 from traffic_monitor_config.js

commit 0f52ab241d33575e0806e8444c7999e023017fbc
Author: Eric Friedrich 
Date:   2017-02-22T19:50:19Z

TC-159: Remove AL2 from JSON header




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #30: Add --skip-ssl option to post install to...

2016-12-13 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/30
  
Not yet, but Peter suggested adding a question to the input in 126 to allow 
a user to skip it. I would support that approach over this PR. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #117: [TC-61] Update LICENSE, NOTICE w...

2016-12-02 Thread limited
Github user limited closed the pull request at:

https://github.com/apache/incubator-trafficcontrol/pull/117


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #117: [TC-61] Update LICENSE, NOTICE w...

2016-12-01 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/117

[TC-61] Update LICENSE, NOTICE with Maxmind CCSA-4.0 IL and attribution



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol fork-1.8.x

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/117.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 #117


commit 5db3d2cdb247c05e90e41d5c485bb5c3d9f63b9b
Author: Eric Friedrich 
Date:   2016-12-01T20:44:36Z

Update LICENSE, NOTICE with Maxmind CCSA-4.0 IL and attribution




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol pull request #116: [TC-61] Update LICENSE, NOTICE w...

2016-12-01 Thread limited
GitHub user limited opened a pull request:

https://github.com/apache/incubator-trafficcontrol/pull/116

[TC-61] Update LICENSE, NOTICE with Maxmind CCSA-4.0 IL and attribution



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/limited/incubator-trafficcontrol mmdb-license

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafficcontrol/pull/116.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 #116


commit 964312affbdda3547bad18cf26e2c053aecdecde
Author: Eric Friedrich 
Date:   2016-12-01T20:44:36Z

Update LICENSE, NOTICE with Maxmind CCSA-4.0 IL and attribution




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #110: [TC-58] TR able to work without JDNSSEC

2016-11-30 Thread limited
Github user limited commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/110
  
test comment to check change in email list subscriptions, please ignore


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---