Nir Sopher created TC-460:
-
Summary: Delivery-service tenancy based access control - DS/Server
Assignment
Key: TC-460
URL: https://issues.apache.org/jira/browse/TC-460
Project: Traffic Control
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/742
If you set neither both features will work simultaneously.
In order to access a DS you'll need to be in the proper tenant (if DS has
tenancy) as well as have a ds/tm_user
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129162314
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -153,21 +158,26 @@ sub show {
my $current_user = $self
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129148538
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -153,21 +158,26 @@ sub show {
my $current_user = $self
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129144946
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -909,8 +945,13 @@ sub capacity {
my $id = $self->param(
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129143451
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -292,7 +308,10 @@ sub update {
}
#setting
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129120463
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -54,7 +54,10 @@ sub index {
$criteria{'me.logs_enabled
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129119425
--- Diff: traffic_ops/app/lib/Utils/Tenant.pm ---
@@ -64,6 +64,10 @@ sub new {
->get_column('value')->
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742#discussion_r129117364
--- Diff: traffic_ops/app/lib/API/Deliveryservice.pm ---
@@ -54,7 +54,10 @@ sub index {
$criteria{'me.logs_enabled
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/742
[TC-428] Ds tenancy validation
Verifying the DS tenancy on delivery-service API
You can merge this pull request into a Git repository by running:
$ git pull https
Nir Sopher created TC-428:
-
Summary: Tenancy based access control - Delivery Service API
Key: TC-428
URL: https://issues.apache.org/jira/browse/TC-428
Project: Traffic Control
Issue Type
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/735#discussion_r128364144
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -477,6 +506,13 @@ sub update_current {
return $self->alert( "
Nir Sopher created TC-427:
-
Summary: Tenancy Based Access Control - User Resource
Key: TC-427
URL: https://issues.apache.org/jira/browse/TC-427
Project: Traffic Control
Issue Type: Improvement
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/735
User tenancy validation
Updating the user API to enforce user tenancy access control.
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/718
Tenant in DS table - tenant name in API
Adding the tenant name to DS listing APIs
You can merge this pull request into a Git repository by running:
$ git pull https
[
https://issues.apache.org/jira/browse/TC-412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16076233#comment-16076233
]
Nir Sopher commented on TC-412:
---
Hi William,
Regarding the addition of owner to each DS.
Are you going to use
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124880183
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -131,21 +130,74 @@ sub update {
my $is_active = $params->{act
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/644
file move moved as required
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/644
True.
Note that there is also a matter of consistency: Tenant update can also be
done only by the parent. Would you like to change that as well?
On Jun 29, 2017
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r124647087
--- Diff: docs/source/development/traffic_ops_api/v12/deliveryservice.rst
---
@@ -168,6 +168,8 @@ Delivery Service
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124640284
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -248,7 +326,17 @@ sub delete {
if ( !defined($tenant
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124639190
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -131,21 +130,74 @@ sub update {
my $is_active = $params->{act
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124638570
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -131,21 +130,74 @@ sub update {
my $is_active = $params->{act
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124637268
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -121,7 +115,12 @@ sub update
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644#discussion_r124635238
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -31,34 +32,28 @@ sub getTenantName {
return defined($tenant_id) ? $self->
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/691
[TC-398] "create user" API - moving password hashing to scrypt
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/
Nir Sopher created TC-398:
-
Summary: Create User API - Broken due to merge issue
Key: TC-398
URL: https://issues.apache.org/jira/browse/TC-398
Project: Traffic Control
Issue Type: Bug
[
https://issues.apache.org/jira/browse/TC-384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nir Sopher updated TC-384:
--
Description:
Due to a missing "profile-type" in the 1.7 json, the profiles cannot be
imported.
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/644
Tenant utils
Adding the "TenantUtils" class allowing to check if resource is accessible
(tenancy wise).
The class provides further tenants related functionalit
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/403
Rebase was done.
The below tests are failing regardless to the PR
t/api/1.1/deliveryservice/ssl_keys.t t/user.t t/deliveryserviceserver.t
t/api/1.2
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/230
PR was rebased so it cam be merged. However, it still needs to be verified
again by adding a record to records.config and checking the value
---
If your project is set up
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/370
The change of the "role" broke the UT and was reverted.
It changes current behavior of user-update.
I prefer not to make such changes in this PR
---
If yo
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/370
Rebase was done a PR review changes were made, except for a test for
"username" in the beginning of the create function. The test is done as the
value is in use by th
Nir Sopher created TC-332:
-
Summary: Heatlh tab never loads if there are no traffic monitors
Key: TC-332
URL: https://issues.apache.org/jira/browse/TC-332
Project: Traffic Control
Issue Type: Bug
Nir Sopher created TC-317:
-
Summary: ORT syncds bug
Key: TC-317
URL: https://issues.apache.org/jira/browse/TC-317
Project: Traffic Control
Issue Type: Bug
Reporter: Nir Sopher
Based
Nir Sopher created TC-304:
-
Summary: Ping each Riak server
Key: TC-304
URL: https://issues.apache.org/jira/browse/TC-304
Project: Traffic Control
Issue Type: Improvement
Components
Nir Sopher created TC-299:
-
Summary: TO Change Log. Expand status to human readable string
Key: TC-299
URL: https://issues.apache.org/jira/browse/TC-299
Project: Traffic Control
Issue Type
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r113034286
--- Diff: traffic_ops/app/t/api/1.2/user_admin.t ---
@@ -0,0 +1,126 @@
+package main;
+#
+#
+# Licensed under
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r113034309
--- Diff: traffic_ops/app/t/api/1.2/user_admin.t ---
@@ -0,0 +1,126 @@
+package main;
+#
+#
+# Licensed under
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r113028771
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -226,6 +226,120 @@ sub update {
}
+# Create
+sub create
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/379
This PR became old and might be irrelevant.
Closing it
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user nir-sopher closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/379
---
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 user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/433
Following a discussion with @mitchell852 I'm closing this PR.
New PR will be provided, supplying an API that supports tenants table
caching to reduce calls to DB when the API
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/403
@mitchell852 Please note that after rebasing the branch deliveryservice UT
fail.
For some reason the changes I do does not work well with the changes you
did, moving from
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/403
File was renamed as required
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112417979
--- Diff:
traffic_ops/app/db/migrations/2017040400_delivery_service_tenancy.sql ---
@@ -0,0 +1,30 @@
+/*
--- End diff
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112418273
--- Diff: traffic_ops/app/lib/Schema/Result/Deliveryservice.pm ---
@@ -278,11 +278,22 @@ __PACKAGE__->table("deliver
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112420550
--- Diff: traffic_ops/app/t/api/1.2/deliveryservice.t ---
@@ -30,302 +30,407 @@ use Test::TestHelper;
BEGIN { $ENV{MOJO_MODE
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403#discussion_r112409155
--- Diff: docs/source/development/traffic_ops_api/v12/deliveryservice.rst
---
@@ -168,6 +168,8 @@ Delivery Service
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15972417#comment-15972417
]
Nir Sopher edited comment on TC-217 at 4/18/17 12:42 PM:
-
Hi,
I believe we need
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15972417#comment-15972417
]
Nir Sopher commented on TC-217:
---
Hi,
I believe we need to distinguish between the 2 terminologies:
"desce
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110659889
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -226,6 +226,120 @@ sub update {
}
+# Create
+sub create
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110659653
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -226,6 +226,120 @@ sub update {
}
+# Create
+sub create
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/402
In this phase we are dealing with "org-tenancy".
Initially I thought some CDN tenancy is required in this phase - as the
content provider creating a DS needs
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110366117
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -218,6 +218,115 @@ sub update {
}
+# Create
+sub create
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370#discussion_r110366079
--- Diff: docs/source/development/traffic_ops_api/v12/user.rst ---
@@ -214,6 +214,153 @@ Users
|
+
+**POST
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/370
Indeed
On Apr 7, 2017 6:16 AM, "Jeremy Mitchell" <notificati...@github.com> wrote:
> *@mitchell852* commented o
Github user nir-sopher closed the pull request at:
https://github.com/apache/incubator-trafficcontrol/pull/322
---
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 user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/322
Almost all changes are already added by other PRs.
Closing
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/439
org tenancy - verify tenant user test stability
Verifying the order of items in the list to ensure test stability
You can merge this pull request into a Git repository
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/433
Tenancy validation tenant
WIP.
First step in tenancy verification and tests - treating "tenant" as a
resource and verify who can write/read it.
WIP as this
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/429
Org tenancy - initial configuration
This PR deals with adding a root tenant.
seed.sql was added with a root user and a util script was added to assign a
user to a tenant
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954994#comment-15954994
]
Nir Sopher edited comment on TC-217 at 4/5/17 8:59 AM:
---
Further thinking of the issue I
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954994#comment-15954994
]
Nir Sopher edited comment on TC-217 at 4/5/17 8:57 AM:
---
Further thinking of the issue I
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/403
All required merges were done.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109747255
--- Diff: traffic_ops/app/db/migrations/2017031502_cdn_tenancy.sql ---
@@ -0,0 +1,31 @@
+/*
--- End diff --
Done
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15955009#comment-15955009
]
Nir Sopher commented on TC-217:
---
Ryan,
Per your comment, please see my comments and edits to
https
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954994#comment-15954994
]
Nir Sopher commented on TC-217:
---
Further thinking of the issue I would like to revise the earlier suggestion I
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109313737
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -277,6 +285,7 @@ sub current {
@data
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109302513
--- Diff: traffic_ops/app/t/api/1.2/tenant.t ---
@@ -152,6 +152,12 @@ ok $t->delete_ok('/api/1.2/tenants/' .
$tenantE_id)->sta
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109302190
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -174,7 +179,9 @@ sub update {
registration_sent
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109292359
--- Diff: traffic_ops/app/lib/Fixtures/Cdn.pm ---
@@ -33,6 +34,17 @@ my %definition_for = (
id =>
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109292282
--- Diff: traffic_ops/app/lib/API/Cdn.pm ---
@@ -181,19 +189,25 @@ sub update {
return $self->alert( "a cdn wit
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109292241
--- Diff: traffic_ops/app/lib/API/Cdn.pm ---
@@ -123,10 +126,14 @@ sub create {
return $self->alert( "a cdn wit
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109292194
--- Diff: traffic_ops/app/lib/API/Cdn.pm ---
@@ -123,10 +126,14 @@ sub create {
return $self->alert( "a cdn wit
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402#discussion_r109292111
--- Diff: traffic_ops/app/db/migrations/2017031502_cdn_tenancy.sql ---
@@ -0,0 +1,31 @@
+/*
--- End diff
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109291385
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -159,6 +161,9 @@ sub update {
return $self->not_fo
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109291378
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -159,6 +161,9 @@ sub update {
return $self->not_fo
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109291260
--- Diff: traffic_ops/app/db/migrations/2017031501_user_tenancy.sql ---
@@ -0,0 +1,31 @@
+/*
--- End diff
[
https://issues.apache.org/jira/browse/TC-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944602#comment-15944602
]
Nir Sopher commented on TC-217:
---
Hi,
Please let me know if the below satisfy you need:
Adding an api whose
[
https://issues.apache.org/jira/browse/TC-215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944597#comment-15944597
]
Nir Sopher commented on TC-215:
---
Please note that the solution proposed in TC-55 practically allows you to use
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/403
Org tenancy - delivery services
This PR adds the tenancy to the delivery services, and adjusts the API.
No tenancy based authorization yet.
Note that the PR can
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/402
Org tenancy - adding tenant to cdn
This PR adds tenant to the CDN table and adjusts the API.
It can be pulled by itself but merging it with PR-398 (adding the user
tenancy
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/385
Yes I am.
Thank you for adding it.
Nir
On Wed, Mar 22, 2017 at 7:18 PM, Jeremy Mitchell <notificati...@github.com>
wrote:
> @nir-soph
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/371
I'll add the test, also as all other fields in the tenant module are tested
upon update, so it is more consistent.
---
If your project is set up for it, you can reply
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/371
Thanks Jeremy for your comments and help with documentation.
Most AIs are in.
Still need to further discuss "Active" input as required comment
---
If your proj
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106616105
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -0,0 +1,247 @@
+package API::Tenant;
+#
+#
+# Licensed under
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106613842
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -0,0 +1,247 @@
+package API::Tenant;
+#
+#
+# Licensed under
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106613824
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -0,0 +1,247 @@
+package API::Tenant;
+#
+#
+# Licensed under
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/371#discussion_r106613763
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -0,0 +1,247 @@
+package API::Tenant;
+#
+#
+# Licensed under
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/378
t_integration/server.t test & UploadCsv - Aligning the csv file header
This PR was created while trying to fix the subject test, but it is not
required for the
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/377
t_integration/server.t test fix
This PR tries to fix the failures of the subject test.
It seems not like a test issue, but rather a bug. Still, the below fix
feels like
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/371
Tenancy table creation
This PR is the first step in adding "org tenancy" to TC. It comes to start
and replace PR-322
It define the tenants table, including an &qu
GitHub user nir-sopher opened a pull request:
https://github.com/apache/incubator-trafficcontrol/pull/370
Adding a "create user" to the api
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nir-sopher/incubator-traf
[
https://issues.apache.org/jira/browse/TC-184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926224#comment-15926224
]
Nir Sopher commented on TC-184:
---
Hi Ryan,
I'm currently working on a PR focusing on items 1-4 of the JIRA
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/322
Current status:
All UTs passed.
First batch of PR remarks was fixed.
The API supports tenancy for CDn / Delivery Service / Users
UTs are added to cover
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/322
The \ is still in my local repo. Will be pushed on next update
On Mar 6, 2017 8:35 PM, "Jeremy Mitchell" <notificati...@github.com> wrote:
Github user nir-sopher commented on the issue:
https://github.com/apache/incubator-trafficcontrol/pull/322
This PR can stand for its own. It passed the unit tests beside
delivery-service related failures, which as far as I can see are not
related to my change. Will be able
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/322#discussion_r103793015
--- Diff: traffic_ops/app/t/api/1.2/tenant.t ---
@@ -0,0 +1,156 @@
+package main;
+#
+#
+# Licensed under the Apache
Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/322#discussion_r103788980
--- Diff: traffic_ops/app/lib/API/Tenant.pm ---
@@ -0,0 +1,225 @@
+package API::Tenant;
+#
+#
+# Licensed under
101 - 200 of 221 matches
Mail list logo