[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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