[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115894997 --- Diff: traffic_ops/app/t/api/1.2/api_capabilities.t --- @@ -0,0 +1,232 @@ +package main; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115854793 --- Diff: docs/source/development/traffic_ops_api/v12/api_capability.rst --- @@ -0,0 +1,367 @@ +.. +.. +.. Licensed under

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115894801 --- Diff: traffic_ops/app/lib/TrafficOpsRoutes.pm --- @@ -669,6 +669,22 @@ sub api_routes { # Supports ?orderby=key $r

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115895225 --- Diff: traffic_ops/app/t/api/1.2/capabilities.t --- @@ -0,0 +1,183 @@ +package main; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115855843 --- Diff: docs/source/development/traffic_ops_api/v12/api_capability.rst --- @@ -0,0 +1,367 @@ +.. +.. +.. Licensed under

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115859251 --- Diff: traffic_ops/app/db/migrations/2017040601_create_capabilities_and_roles.sql --- @@ -0,0 +1,91

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115856573 --- Diff: docs/source/development/traffic_ops_api/v12/capability.rst --- @@ -0,0 +1,271 @@ +.. +.. +.. Licensed under

[GitHub] incubator-trafficcontrol pull request #544: Authorization model

2017-05-10 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/544#discussion_r115855505 --- Diff: docs/source/development/traffic_ops_api/v12/api_capability.rst --- @@ -0,0 +1,367 @@ +.. +.. +.. Licensed under

[GitHub] incubator-trafficcontrol pull request #559: [TC-64] - to exp - changes mount...

2017-05-09 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/559 [TC-64] - to exp - changes mount point for ssl certs You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #558: [TC-64] - to exp ui - hooks up r...

2017-05-09 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/558 [TC-64] - to exp ui - hooks up routing methods component You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #550: adds timezone to last_seen_log b...

2017-05-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/550 adds timezone to last_seen_log because log.last_updated column is now of type timestamptz You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-trafficcontrol pull request #549: [TC-64] to ui exp dashboard

2017-05-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/549 [TC-64] to ui exp dashboard You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol tc-64

[GitHub] incubator-trafficcontrol pull request #548: adds back GET api/1.1/asns. the ...

2017-05-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/548 adds back GET api/1.1/asns. the only truly 1.1 api :) You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #546: fixes exampleurls on a collectio...

2017-05-04 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/546 fixes exampleurls on a collection and an individual ds You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #545: [TC-270] - cache_stats - query f...

2017-05-04 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/545 [TC-270] - cache_stats - query from summarized data (continuous queries) instead of raw data You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-05-03 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r114584874 --- Diff: traffic_ops/app/db/seeds.sql --- @@ -22,6 +22,257 @@ insert into role (id, name, description, priv_level) values (5

[GitHub] incubator-trafficcontrol pull request #536: [TC-64] - TO UI exp - places cac...

2017-05-02 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/536 [TC-64] - TO UI exp - places cachegroup markers on google map You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #529: [TC-64] - TO exp - fixes bug in ...

2017-04-28 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/529 [TC-64] - TO exp - fixes bug in change log poller. had to move it into a persistent mode… …l so the $interval would not get crushed. it is needed to cancel the interval

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

2017-04-27 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/497 @jschmerge - can you create a jira ticket for this? https://issues.apache.org/jira/browse/TC --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-trafficcontrol pull request #526: [TC-64] - TO exp - fixes respons...

2017-04-27 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/526 [TC-64] - TO exp - fixes responsive layout You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #524: [TC-64] TOUI experimental - adds...

2017-04-26 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/524 [TC-64] TOUI experimental - adds update status dialog and also changes return object of generic d… …ialog component You can merge this pull request into a Git repository

[GitHub] incubator-trafficcontrol pull request #513: [TC-235] - fixes max_dns_answers...

2017-04-25 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/513 [TC-235] - fixes max_dns_answers for all DNS* types You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #509: [TC-64] - adds the ability to un...

2017-04-24 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/509 [TC-64] - adds the ability to unlink profiles from parameters You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113050424 --- Diff: traffic_ops/app/db/seeds.sql --- @@ -22,6 +22,257 @@ insert into role (id, name, description, priv_level) values (5

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113049926 --- Diff: traffic_ops/app/db/seeds.sql --- @@ -22,6 +22,257 @@ insert into role (id, name, description, priv_level) values (5

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113049468 --- Diff: traffic_ops/app/db/migrations/2017040601_create_capabilities_and_roles.sql --- @@ -0,0 +1,89

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113049164 --- Diff: traffic_ops/app/db/migrations/2017040601_create_capabilities_and_roles.sql --- @@ -0,0 +1,89

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113056228 --- Diff: traffic_ops/app/lib/TrafficOpsRoutes.pm --- @@ -657,6 +657,23 @@ sub api_routes { # Supports ?orderby=key $r

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113053856 --- Diff: traffic_ops/app/lib/API/ApiCapability.pm --- @@ -0,0 +1,258 @@ +package API::ApiCapability

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113053641 --- Diff: traffic_ops/app/lib/API/ApiCapability.pm --- @@ -0,0 +1,258 @@ +package API::ApiCapability

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113056848 --- Diff: traffic_ops/app/lib/API/ApiCapability.pm --- @@ -0,0 +1,258 @@ +package API::ApiCapability

[GitHub] incubator-trafficcontrol pull request #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/435#discussion_r113052674 --- Diff: traffic_ops/app/db/seeds.sql --- @@ -22,6 +22,257 @@ insert into role (id, name, description, priv_level) values (5

[GitHub] incubator-trafficcontrol issue #435: Authorization model

2017-04-24 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/435 Hi @naamashoresh - I'm sorry to make you do this but can you reconcile your seeds.sql file? You might have to do a rebase as seeds.sql has changes significantly since you

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-24 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r113004733 --- Diff: traffic_ops/app/t/api/1.2/user_admin.t --- @@ -0,0 +1,126 @@ +package main; +# +# +# Licensed under

[GitHub] incubator-trafficcontrol pull request #498: [WIP] - moves demo data out of s...

2017-04-21 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/498 [WIP] - moves demo data out of seeds file into seeds_demo.sql and adds new co… …nvenience method to admin.pl You can merge this pull request into a Git repository

[GitHub] incubator-trafficcontrol issue #370: Adding a "create user" to the api

2017-04-20 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/370 you might want to run your code thru the perl formatter. I see some formatting errors. Here's more info about that: https://github.com/apache/incubator-trafficcontrol

[GitHub] incubator-trafficcontrol pull request #493: [TC-241] - adds api endpoint to ...

2017-04-20 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/493 [TC-241] - adds api endpoint to remove a cache from a ds You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol issue #433: [TC-184] Tenancy validation for resourc...

2017-04-20 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/433 make sure you rebase this when you get a chance. you have conflicts. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-trafficcontrol pull request #492: [TC-64] - hooks TOUI into up dbd...

2017-04-20 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/492 [TC-64] - hooks TOUI into up dbdump api You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol issue #491: [TC-240] updates db dump to custom form...

2017-04-19 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/491 works well for me in both the UI and the api. merging... --- 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

[GitHub] incubator-trafficcontrol pull request #486: [TC-82] - removes duplicate apis

2017-04-19 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/486 [TC-82] - removes duplicate apis You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol tc

[GitHub] incubator-trafficcontrol pull request #485: fixes broken asn test

2017-04-19 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/485 fixes broken asn test You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol fixes-asn

[GitHub] incubator-trafficcontrol pull request #484: fixes broken log tests - loads l...

2017-04-19 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/484 fixes broken log tests - loads log fixture data outside of the core You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #483: [TC-64] - hooks in change logs t...

2017-04-18 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/483 [TC-64] - hooks in change logs to TO experimental UI You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #482: [TC-238] - adds limit query para...

2017-04-18 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/482 [TC-238] - adds limit query param to GET api/../logs You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #481: [TC-237] - GET /api/../logs sets...

2017-04-18 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/481 [TC-237] - GET /api/../logs sets last_seen_log cookie when all change logs are viewed emulates the UI namespace You can merge this pull request into a Git repository

[GitHub] incubator-trafficcontrol pull request #433: [TC-184] Tenancy validation for ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/433#discussion_r112024314 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -117,14 +123,25 @@ sub update { my $is_active = $params->{act

[GitHub] incubator-trafficcontrol pull request #433: [TC-184] Tenancy validation for ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/433#discussion_r112025226 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -117,14 +123,25 @@ sub update { my $is_active = $params->{act

[GitHub] incubator-trafficcontrol issue #402: [TC-184] Org tenancy - adding tenant to...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/402 @nir-sopher - i agree. let's tackle tenancy on a deliveryservice to start and then we can circle back to tenancy at the cdn level... --- If your project is set up for it, you

[GitHub] incubator-trafficcontrol pull request #403: [TC-184] Org tenancy - delivery ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112016567 --- Diff: traffic_ops/app/lib/Schema/Result/Deliveryservice.pm --- @@ -278,11 +278,22 @@ __PACKAGE__->table("deliver

[GitHub] incubator-trafficcontrol pull request #403: [TC-184] Org tenancy - delivery ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112015576 --- Diff: traffic_ops/app/db/migrations/2017040400_delivery_service_tenancy.sql --- @@ -0,0 +1,30 @@ +/* --- End diff

[GitHub] incubator-trafficcontrol pull request #403: [TC-184] Org tenancy - delivery ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112016601 --- Diff: traffic_ops/app/lib/Schema/Result/Deliveryservice.pm --- @@ -390,8 +401,12 @@ __PACKAGE__->add_columns( { data_t

[GitHub] incubator-trafficcontrol pull request #403: [TC-184] Org tenancy - delivery ...

2017-04-18 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112014737 --- Diff: docs/source/development/traffic_ops_api/v12/deliveryservice.rst --- @@ -168,6 +168,8 @@ Delivery Service

[GitHub] incubator-trafficcontrol pull request #478: cleans up trafficopsroutes file ...

2017-04-18 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/478 cleans up trafficopsroutes file - reorganizes, adds parameter constra… …ints where necessary, additional comments, etc. You can merge this pull request into a Git

[GitHub] incubator-trafficcontrol pull request #464: [TC-231] - adds a query param (e...

2017-04-17 Thread mitchell852
Github user mitchell852 closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/464 --- 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 #472: fixes failing tests. ambiguous f...

2017-04-14 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/472 fixes failing tests. ambiguous fields need table qualifiers. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol issue #434: org tenancy - add tenant name to user a...

2017-04-14 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/434 @orifinkelman - can you take out all the "N/A"s and i'll get this merged? if tenant is not defined, just leave it as undef that is how we do it on the

[GitHub] incubator-trafficcontrol pull request #470: fixes failing job unit test

2017-04-14 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/470 fixes failing job unit test You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol fix-job

[GitHub] incubator-trafficcontrol pull request #469: [BACKPORT TC-223] - default rege...

2017-04-13 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/469 [BACKPORT TC-223] - default regex is required on a ds before dnssec keys can be created (… …if required) (cherry picked from commit

[GitHub] incubator-trafficcontrol pull request #464: [TC-231] - adds a query param (e...

2017-04-12 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/464 [TC-231] - adds a query param (exclude) that can prevent the fetching of the ver… …y expensive exampleURLs property. if not used (?exclude=exampleURLs), the api behaves

[GitHub] incubator-trafficcontrol pull request #460: [TC-64] - TO UI experimental - a...

2017-04-12 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/460 [TC-64] - TO UI experimental - adds queue updates and invalidate content views You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-trafficcontrol pull request #458: [TC-221] - responses to ds updat...

2017-04-12 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/458 [TC-221] - responses to ds update/create now mirror that of show You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #454: clarifies ports for toui and rem...

2017-04-11 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/454 clarifies ports for toui and removes unused proxyport You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #434: org tenancy - add tenant name to...

2017-04-11 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/434#discussion_r110946789 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -281,11 +284,14 @@ sub current { if ( _ldap($self

[GitHub] incubator-trafficcontrol pull request #453: [TC-227] - adds the ability to f...

2017-04-11 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/453 [TC-227] - adds the ability to fetch jobs thru api You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[GitHub] incubator-trafficcontrol pull request #451: [TC-193] - adds ability to filte...

2017-04-10 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/451 [TC-193] - adds ability to filter delivery services by profile You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol issue #444: Updated API POST /api/1.2/deliveryservi...

2017-04-10 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/444 can you do a search on that page for missLat and missLong? I see 2 more with 45 as the example, want to update those as well while you are in this file? --- If your project

[GitHub] incubator-trafficcontrol pull request #447: [TC-64] - to api destination is ...

2017-04-10 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/447 [TC-64] - to api destination is controlled at the server level You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol pull request #446: [TC-64] - fixes serverutils typo

2017-04-10 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/446 [TC-64] - fixes serverutils typo 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 #445: [TC-64] - To exp ui server.js fi...

2017-04-10 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/445 [TC-64] - To exp ui server.js fixes plus simplifying build You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852

[GitHub] incubator-trafficcontrol pull request #433: [TC-184] Tenancy validation for ...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/433#discussion_r110311804 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -44,14 +44,16 @@ sub index { my $orderby = $self->param('orderby') || &q

[GitHub] incubator-trafficcontrol pull request #433: [TC-184] Tenancy validation for ...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/433#discussion_r110311677 --- Diff: traffic_ops/app/lib/API/Tenant.pm --- @@ -44,14 +44,16 @@ sub index { my $orderby = $self->param('orderby') || &q

[GitHub] incubator-trafficcontrol issue #402: [TC-184] Org tenancy - adding tenant to...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/402 Do you want me to hold off on this one? Are you rethinking it? I still think we need tenancy at the CDN level. --- If your project is set up for it, you can reply

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110309859 --- Diff: docs/source/development/traffic_ops_api/v12/user.rst --- @@ -214,6 +214,153 @@ Users | + +**POST

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110310005 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -226,6 +226,120 @@ sub update { } +# Create +sub create

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110309625 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -226,6 +226,120 @@ sub update { } +# Create +sub create

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110309463 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -226,6 +226,120 @@ sub update { } +# Create +sub create

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r107505547 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -218,6 +218,115 @@ sub update { } +# Create +sub create

[GitHub] incubator-trafficcontrol pull request #370: Adding a "create user" to the ap...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110309119 --- Diff: docs/source/development/traffic_ops_api/v12/user.rst --- @@ -214,6 +214,153 @@ Users | + +**POST

[GitHub] incubator-trafficcontrol pull request #434: org tenancy - add tenant name to...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/434#discussion_r110308097 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -330,6 +336,7 @@ sub current { "phoneN

[GitHub] incubator-trafficcontrol pull request #434: org tenancy - add tenant name to...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/434#discussion_r110307660 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -98,6 +98,7 @@ sub index { "stateOrProvince&quo

[GitHub] incubator-trafficcontrol pull request #434: org tenancy - add tenant name to...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/434#discussion_r110307744 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -69,7 +69,7 @@ sub index { my $dbh; if ( defined $username

[GitHub] incubator-trafficcontrol pull request #434: org tenancy - add tenant name to...

2017-04-06 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/434#discussion_r110308262 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -134,6 +135,7 @@ sub show { "stateOrProvince&quo

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

2017-04-06 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/322 can this PR be closed? --- 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 #435: Authorization model

2017-04-06 Thread mitchell852
Github user mitchell852 commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/435 thanks @naamashoresh - i'll take a look at this asap. --- 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

[GitHub] incubator-trafficcontrol pull request #430: updates traffic portal install r...

2017-04-05 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/430 updates traffic portal install readme You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator

[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

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

2017-04-02 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109322384 --- Diff: traffic_ops/app/lib/API/Cdn.pm --- @@ -123,10 +126,14 @@ sub create { return $self->alert( &qu

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

2017-04-02 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109321173 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -277,6 +285,7 @@ sub current { @data

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

2017-04-02 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109320570 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -277,6 +285,7 @@ sub current { @data

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

2017-04-02 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109312553 --- Diff: traffic_ops/app/lib/API/User.pm --- @@ -277,6 +285,7 @@ sub current { @data

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

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

[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 #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 #421: fixes broken profile tests

2017-03-31 Thread mitchell852
GitHub user mitchell852 opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/421 fixes broken profile tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/mitchell852/incubator-trafficcontrol fix

[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 #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 #402: [TC-184] Org tenancy - adding te...

2017-03-31 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109164965 --- Diff: traffic_ops/app/db/migrations/2017031502_cdn_tenancy.sql --- @@ -0,0 +1,31 @@ +/* --- End diff

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

2017-03-31 Thread mitchell852
Github user mitchell852 commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109166684 --- Diff: traffic_ops/app/lib/API/Cdn.pm --- @@ -123,10 +126,14 @@ sub create { return $self->alert( &qu

<    1   2   3   4   5   >