[GitHub] incubator-trafficcontrol issue #742: [TC-428] Ds tenancy validation

2017-07-25 Thread nir-sopher
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 record.
This is implemented in a branch pending to PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #742: [TC-428] Ds tenancy validation

2017-07-25 Thread mitchell852
Github user mitchell852 commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/742
  
just so i'm clear. to fully implement tenancy you should set the parameters 
like this?

ignore-tenancy=0
ignore_ds_tmuser_assignment=1





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafficcontrol issue #742: [TC-428] Ds tenancy validation

2017-07-24 Thread mitchell852
Github user mitchell852 commented on the issue:

https://github.com/apache/incubator-trafficcontrol/pull/742
  
I'm a little worried that I don't see an entry in the seeds.sql file for 
the ignore-tenancy parameter. I feel like if somebody want's to disable tenancy 
they will not know to add that parameter and set it to 1.

what do you think? should we have an entry in seeds.sql with 
ignore-tenancy=0?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---