[GitHub] incubator-trafficcontrol pull request #247: [TC-124] - adds POST api/cdns/:i...

2017-02-01 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/247 [TC-124] - adds POST api/cdns/:id/queue_update route and tries to stay consisten… …t with the implementation of the other queue_update routes (server and cachegroup) You

[GitHub] incubator-trafficcontrol pull request #259: [TC-139] - adds necessary prefet...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/259 [TC-139] - adds necessary prefetching to /api/servers so no n+1 queries You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #260: [TC-64] - To exp ui - adds stati...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/260 [TC-64] - To exp ui - adds static dns entries and changes cta buttons You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #257: [TC-137] - adds dbix prefetch fo...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/257 [TC-137] - adds dbix prefetch for api/version/cachegroupparameters You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #243: [TC-125] Lowercase FQDNs generat...

2017-02-02 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/243#discussion_r99145205 --- Diff: traffic_ops/app/lib/UI/DeliveryService.pm --- @@ -78,7 +78,8 @@ sub get_cdn_domain { distinct =>

[GitHub] incubator-trafficcontrol pull request #253: [TC-134] - gets rid of sequentia...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/253 [TC-134] - gets rid of sequential queries caused by running a query inside a loo… …p. uses dbix prefetch instead to join 2 tables and retrieve the necessary information

[GitHub] incubator-trafficcontrol pull request #252: [TC-133] - uses dbix prefetch to...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/252 [TC-133] - uses dbix prefetch to eliminate n+1 queries and removes example urls … …intended for a single deliveryservice You can merge this pull request into a Git

[GitHub] incubator-trafficcontrol pull request #254: [TC-135] - joins ds_server on ds...

2017-02-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/254 [TC-135] - joins ds_server on ds table to avoid n+1 queries and improve route pe… …rformance 10x at least You can merge this pull request into a Git repository by running

[GitHub] incubator-trafficcontrol pull request #263: [TC-141] - adds more prefetches ...

2017-02-07 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/263 [TC-141] - adds more prefetches to api endpoints You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #244: [TC-126] - adds seed data lost w...

2017-02-01 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/244 [TC-126] - adds seed data lost when migration files were removed You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #240: [TC-99] api/{version}/cdns/:cdn_...

2017-01-31 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/240#discussion_r98784396 --- Diff: traffic_ops/app/lib/MojoPlugins/Health.pm --- @@ -159,6 +159,9 @@ sub register { cachegroups

[GitHub] incubator-trafficcontrol issue #141: Old - do not merge, will delete

2017-01-23 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/141 @dg4prez - you want to just close this since you have opened this? https://github.com/apache/incubator-trafficcontrol/pull/199 --- If your project is set up

[GitHub] incubator-trafficcontrol issue #200: [TC-68] - to support an HA TO, regex_re...

2017-01-23 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/200 @dewrich or @dangogh - can either of you review / merge? this needs to get into 2.0 so the path to HA TO is clear. thanks! --- If your project is set up for it, you can reply

[GitHub] incubator-trafficcontrol pull request #208: Fix TC-110 - parameterize ip_all...

2017-01-23 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/208#discussion_r97332624 --- Diff: traffic_ops/app/lib/UI/ConfigFiles.pm --- @@ -533,9 +550,8 @@ sub ip_allow_data

[GitHub] incubator-trafficcontrol pull request #214: [TC-94] - adds deliveryservice_r...

2017-01-23 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/214 [TC-94] - adds deliveryservice_regex crud capabilities You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #216: adds license details for experim...

2017-01-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/216 adds license details for experimental TO ui theme and loading bar use… …d in traffic portal and experimental TO UI You can merge this pull request into a Git repository

[GitHub] incubator-trafficcontrol pull request #219: removes self-signed certs provid...

2017-01-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/219 removes self-signed certs provided for convenience from TO UI experim… …ental You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafficcontrol pull request #217: updates main TC license file

2017-01-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/217 updates main TC license file You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol

[GitHub] incubator-trafficcontrol pull request #239: [TC-64] - hooks up experimental ...

2017-01-30 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/239 [TC-64] - hooks up experimental TO UI to ds regex crud api You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol issue #230: Allowing variables in traffic-server's ...

2017-01-25 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/230 @nir-sopher - if you rename this PR "[TC-121] Allowing variables in traffic-server's profile derived parameters" I think it will link it to the jira issue... -

[GitHub] incubator-trafficcontrol pull request #231: [TC-103] - casts 0|1 to bools in...

2017-01-25 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/231 [TC-103] - casts 0|1 to bools in GET /update/:host_name route You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol issue #295: [TC-158] daemonize() leaves parent proc...

2017-02-20 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/295 @elsloo or @dneuman64 - maybe one of you guys feel qualified to review / test this PR. I, unfortunately, do not. --- If your project is set up for it, you can reply

[GitHub] incubator-trafficcontrol pull request #278: New Overview Page

2017-02-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/278#discussion_r101674626 --- Diff: docs/source/overview/introduction.rst --- @@ -15,14 +15,34 @@ Introduction -Traffic Control

[GitHub] incubator-trafficcontrol pull request #307: [tc-164] - $self needs to be pas...

2017-02-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/307 [tc-164] - $self needs to be passed to these methods for dbix You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #309: [TC-164-Backport] - $self needs ...

2017-02-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/309 [TC-164-Backport] - $self needs to be passed to these methods for dbix (cherry picked from commit f1a912f1ae4296eabf8b966fb852b2f76f91979e) You can merge this pull request

[GitHub] incubator-trafficcontrol pull request #284: [TC-147] - $self was being used ...

2017-02-14 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/284 [TC-147] - $self was being used incorrectly limiting reuse of create_dnssec_keys() method You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-trafficcontrol pull request #278: New Overview Page

2017-02-13 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/278#discussion_r100897910 --- Diff: docs/source/overview/introduction.rst --- @@ -15,14 +15,34 @@ Introduction -Traffic Control

[GitHub] incubator-trafficcontrol pull request #280: [TC-146] - adds osversions.cfg t...

2017-02-13 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/280 [TC-146] - adds osversions.cfg to traffic ops RPM post install which is needed f… …or /geniso route You can merge this pull request into a Git repository by running

[GitHub] incubator-trafficcontrol pull request #276: [TC-141] - need to clarify ambig...

2017-02-13 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/276 [TC-141] - need to clarify ambiguous id when adding query prefetches (joins) You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafficcontrol pull request #277: [TC-144] - need to clarify ambig...

2017-02-13 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/277 [TC-144] - need to clarify ambiguous id when adding prefetches (joins) You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #213: [TC-117] - creates a default del...

2017-01-23 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/213 [TC-117] - creates a default deliveryservice_regex when a new ds is created at set_number=0 in the format .\*\.xml-id\..\* You can merge this pull request into a Git

[GitHub] incubator-trafficcontrol pull request #199: [TC-32] Addition of ATS Config A...

2017-01-23 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/199#discussion_r97381236 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1941 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #181: [TC-96] - adds group_by to dbix ...

2017-01-17 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/181#discussion_r96448937 --- Diff: traffic_ops/app/lib/API/Cdn.pm --- @@ -264,7 +264,8 @@ sub get_traffic_monitor_config { { 'cdn.name

[GitHub] incubator-trafficcontrol pull request #191: [TC-95] - ports some functionali...

2017-01-17 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/191 [TC-95] - ports some functionality over from ui namespace. specifically creatin… …g location params (if don't exist) for header rewrites, regex_remap and cacheurl plus

[GitHub] incubator-trafficcontrol pull request #181: [TC-96] - adds group_by to dbix ...

2017-01-17 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/181#discussion_r96442726 --- Diff: traffic_ops/app/lib/API/Cdn.pm --- @@ -264,7 +264,8 @@ sub get_traffic_monitor_config { { 'cdn.name

[GitHub] incubator-trafficcontrol pull request #197: [TC-25] - removes ort routes tha...

2017-01-18 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/197 [TC-25] - removes ort routes that appear to have been broken... ...since 10/5/15 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafficcontrol issue #309: [TC-164-Backport] - $self needs to be p...

2017-02-28 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/309 @knutsel - yes, I agree jan...but...$self is not what it appears to be in this case as the parent method is being called from the UI namespace AND the API namespace

[GitHub] incubator-trafficcontrol pull request #322: Adding tenancy to the database

2017-02-28 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/322#discussion_r103570488 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -0,0 +1,225 @@ +package API::Tenant; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol pull request #127: [TC-52] Integration Tests for Tr...

2016-12-08 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/127#discussion_r91540680 --- Diff: traffic_ops/app/lib/API/Deliveryservice.pm --- @@ -295,7 +295,7 @@ sub update { my @response

[GitHub] incubator-trafficcontrol pull request #129: [EXP] adds placeholder functiona...

2016-12-07 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/129 [EXP] adds placeholder functionality for experimental TO UI You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #127: [TC-52] Integration Tests for Tr...

2016-12-07 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/127#discussion_r91391206 --- Diff: traffic_ops/app/lib/API/Deliveryservice.pm --- @@ -295,7 +295,7 @@ sub update { my @response

[GitHub] incubator-trafficcontrol issue #131: [TC-66] - Stores, diffs and fetches CRC...

2016-12-13 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/131 files were merged but asfgit didn't feel like closing this PR so I will do so manually. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-trafficcontrol pull request #131: [TC-66] - Stores, diffs and fetc...

2016-12-13 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/131 --- 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 #131: [TC-66] - Stores, diffs and fetc...

2016-12-12 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/131#discussion_r92041791 --- Diff: traffic_ops/app/db/migrations/2016120800_alter_cdn_name_not_null.sql --- @@ -0,0 +1,23

[GitHub] incubator-trafficcontrol pull request #131: [TC-66] - Stores, diffs and fetc...

2016-12-09 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/131 [TC-66] - Stores, diffs and fetches CRConfig.json from the db instead of the file system This is one step toward making TO HA capable (highly-available) You can merge

[GitHub] incubator-trafficcontrol pull request #131: [TC-66] - Stores, diffs and fetc...

2016-12-09 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/131#discussion_r91789262 --- Diff: traffic_ops/app/lib/UI/Topology.pm --- @@ -554,52 +554,30 @@ sub gen_crconfig_json { return ($data_obj

[GitHub] incubator-trafficcontrol issue #183: fix any_map bug introduced in 1.8

2017-01-11 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/183 just want to make sure removing this code (&& $ds_type ne 'ANY_MAP') doesn't reintroduce a previous bug... https://github.com/apache/incubator-trafficcontro

[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 https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #181: [TC-96] - adds group_by to dbix ...

2017-01-11 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/181 [TC-96] - adds group_by to dbix query to satisfy postgres You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #160: [TC-64] - Adds more views to exp...

2017-01-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/160 [TC-64] - Adds more views to experimental TO UI You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #161: [TC-64] - More views for experim...

2017-01-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/161 [TC-64] - More views for experimental UI You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #165: [TC-88] - checks for ip and ip6 ...

2017-01-06 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/165 [TC-88] - checks for ip and ip6 / profile uniqueness on server create / update this is a constraint in the database. checking it in the api layer. You can merge this pull

[GitHub] incubator-trafficcontrol pull request #140: [TC-75] - adds cdn and cachegrou...

2016-12-19 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/140 [TC-75] - adds cdn and cachegroup filters to api/1.2/servers You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #142: [PSQL] - removes expensive disti...

2016-12-20 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/142 [PSQL] - removes expensive distinct from query You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-21 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93525547 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1959 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-23 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93779696 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1959 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-23 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93786024 --- Diff: traffic_ops/app/lib/UI/ConfigFiles.pm --- @@ -312,6 +312,9 @@ sub param_data { if ( $row->parameter->n

[GitHub] incubator-trafficcontrol pull request #146: [TC-80] - adds cachegroup filter...

2016-12-28 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/146 [TC-80] - adds cachegroup filter to parameters and asns You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-21 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93565394 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1959 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-22 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93647261 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1959 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-22 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93649730 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -0,0 +1,1959 @@ +package API::Configs::ApacheTrafficServer

[GitHub] incubator-trafficcontrol pull request #141: [Please Review] [TC-32] Addition...

2016-12-22 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93661807 --- Diff: traffic_ops/app/lib/Schema/Result/DeliveryServiceInfoForCdnList.pm --- @@ -0,0 +1,101 @@ + +# +# +# Licensed

[GitHub] incubator-trafficcontrol issue #400: tenant api - fixes boolean active flag ...

2017-03-23 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/400 so check it out. you have to "cast" the booleans in index and show methods but not in create / update... I don't know why to be honest... --- If yo

[GitHub] incubator-trafficcontrol pull request #389: TO UI - lower edge heath table r...

2017-03-21 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/389 TO UI - lower edge heath table refresh rate from 10s to 30s You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol issue #385: [TC-184] - adds parent tenant name

2017-03-22 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/385 @nir-sopher - you ok with this? parentName is just a convenience added to the API. It's not a table on the tenant table. --- If your project is set up for it, you can reply

[GitHub] incubator-trafficcontrol pull request #395: [BACKPORT] - Change TO monitorin...

2017-03-22 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/395 [BACKPORT] - Change TO monitoring.json to raw SQL This is 3-10x faster, and this endpoint is one of the slowest. (cherry picked from commit

[GitHub] incubator-trafficcontrol pull request #395: [BACKPORT] - Change TO monitorin...

2017-03-22 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/395 --- 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 #399: [TC-205] - fixes null pointer wh...

2017-03-22 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/399 [TC-205] - fixes null pointer when user logged in as ldap plus copy/paste errors… … plus moved up invalid check You can merge this pull request into a Git repository

[GitHub] incubator-trafficcontrol pull request #400: tenant api - fixes boolean activ...

2017-03-22 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/400 tenant api - fixes boolean active flag on update and create You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #401: [TC-206] - adds the ability in t...

2017-03-22 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/401 [TC-206] - adds the ability in the API to filter profiles by cdn You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #419: updates docs for ds apis

2017-03-31 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/419 updates docs for ds apis You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol ds-docs

[GitHub] incubator-trafficcontrol pull request #420: fixes datatypes on ds api docs

2017-03-31 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/420 fixes datatypes on ds api docs You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol ds

[GitHub] incubator-trafficcontrol pull request #374: [BACKPORT] - adds back exampleur...

2017-03-16 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/374 [BACKPORT] - adds back exampleurls to deliveryservice in /api/version/deliveryserv… …ices (cherry picked from commit dddb3a67095b2c07df46e3cd166fdb1843614c67

[GitHub] incubator-trafficcontrol pull request #373: adds back exampleurls to deliver...

2017-03-16 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/373 adds back exampleurls to deliveryservice in /api/version/deliveryserv… …ices You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafficcontrol pull request #385: adds parent tenant name

2017-03-20 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/385 adds parent tenant name You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol

[GitHub] incubator-trafficcontrol pull request #368: [BACKPORT] - fixes regression - ...

2017-03-15 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/368 --- 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 #362: [BACKPORT] [TC-189] - fixes 2 mi...

2017-03-15 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/362 --- 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 #356: Security Enhancement - secures D...

2017-03-14 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/356#discussion_r106014345 --- Diff: traffic_ops/app/lib/UI/GenDbDump.pm --- @@ -17,11 +17,17 @@ package UI::GenDbDump; # use Mojo::Base 'Mojolicious

[GitHub] incubator-trafficcontrol pull request #367: [TC-194] Add federation view fai...

2017-03-14 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/367 [TC-194] Add federation view failed to load DS's due to bad orderby value You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #356: Security Enhancement - secures D...

2017-03-14 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/356#discussion_r106014242 --- Diff: traffic_ops/app/lib/UI/GenDbDump.pm --- @@ -17,11 +17,17 @@ package UI::GenDbDump; # use Mojo::Base 'Mojolicious

[GitHub] incubator-trafficcontrol pull request #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106541559 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -0,0 +1,247 @@ +package API::Tenant; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol pull request #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106541595 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -0,0 +1,247 @@ +package API::Tenant; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol issue #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/371 Also, you forgot one thingdocumentation. I'll help you get started on that...I'll send a PR your way. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-trafficcontrol pull request #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106573144 --- Diff: traffic_ops/app/lib/Fixtures/Tenant.pm --- @@ -0,0 +1,46 @@ +package Fixtures::Tenant; +# +# +# Licensed

[GitHub] incubator-trafficcontrol pull request #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106541500 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -0,0 +1,247 @@ +package API::Tenant; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol issue #371: [TC-184] Tenancy table creation

2017-03-17 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/371 As far as the "active" flag being required on update, it's no big deal. either was is fine with me. i guess i was just thinking it might be confusing to a user that

[GitHub] incubator-trafficcontrol pull request #390: [BACKPORT] [TC-204] - only reloa...

2017-03-21 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/390 [BACKPORT] [TC-204] - only reload health table if browser window is active lightens the load on the database You can merge this pull request into a Git repository by running

[GitHub] incubator-trafficcontrol pull request #390: [BACKPORT] [TC-204] - only reloa...

2017-03-21 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/390 --- 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 #368: [BACKPORT] - fixes regression - ...

2017-03-15 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/368 [BACKPORT] - fixes regression - unable to create federations (cherry picked from commit 6685b67d373562849e94cda3ee5c4ca630ee57fb) You can merge this pull request into a Git

[GitHub] incubator-trafficcontrol pull request #369: fixes broken cdn test. domainNam...

2017-03-15 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/369 fixes broken cdn test. domainName is required on put. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #299: Instant Invalidate for ORT and T...

2017-03-15 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/299#discussion_r106264761 --- Diff: traffic_ops/app/lib/API/Configs/ApacheTrafficServer.pm --- @@ -76,7 +76,9 @@ sub get_config_metadata

[GitHub] incubator-trafficcontrol pull request #299: Instant Invalidate for ORT and T...

2017-03-15 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/299#discussion_r106267760 --- Diff: traffic_ops/app/lib/UI/Server.pm --- @@ -915,20 +916,53 @@ sub readupdate

[GitHub] incubator-trafficcontrol pull request #299: Instant Invalidate for ORT and T...

2017-03-15 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/299#discussion_r106263950 --- Diff: traffic_ops/app/db/migrations/20170221190747_add_reval_pending.sql --- @@ -0,0 +1,26 @@ +/* + --- End diff

[GitHub] incubator-trafficcontrol pull request #299: Instant Invalidate for ORT and T...

2017-03-15 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/299#discussion_r106269894 --- Diff: traffic_ops/app/lib/UI/Server.pm --- @@ -980,7 +1018,14 @@ sub postupdate { my $update_server

[GitHub] incubator-trafficcontrol pull request #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106575505 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -0,0 +1,247 @@ +package API::Tenant; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol issue #371: [TC-184] Tenancy table creation

2017-03-16 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/371 I created a PR into your branch for the documentation - https://github.com/nir-sopher/incubator-trafficcontrol/pull/4 If it looks ok to you, just hit the merge button

[GitHub] incubator-trafficcontrol pull request #398: [TC-184] Org tenancy - adding te...

2017-04-01 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109293617 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -174,7 +179,9 @@ sub update { registration_sent

[GitHub] incubator-trafficcontrol pull request #398: [TC-184] Org tenancy - adding te...

2017-04-01 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109293498 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -174,7 +179,9 @@ sub update { registration_sent

[GitHub] incubator-trafficcontrol pull request #398: [TC-184] Org tenancy - adding te...

2017-04-01 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109293677 --- Diff: traffic_ops/app/lib/UI/Utils.pm --- @@ -42,7 +42,7 @@ use constant ADMIN => 30; our %EXPORT_TAGS = ( '

[GitHub] incubator-trafficcontrol pull request #427: [TC-222] - adds unique constrain...

2017-04-04 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/427 [TC-222] - adds unique constraint to a number of tables to: parameter (name, config_file, value) job_status (name) job_agent (name) role (name) status

  1   2   3   4   5   >