[GitHub] zeppelin issue #1223: [WIP Zeppelin-945] Interpreter authorization

2016-08-01 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
I'll do this at the another 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] zeppelin issue #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-25 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@prabhjyotsingh Thank you for share your great idea.
This PR is for interpreter authorization but your idea seems include user 
management.
I think it should be separated and I agree with @Leemoonsoo's idea of 
giving the same user experience for the authorization inside of Zeppelin.
and I also agree with @prabhjyotsingh's idea of the difference with 
notebook's because interpreter authorization is n:n. (it's hard to show with 
notebook's way.)
so What do you guys think about following way?

- when you want to grant permission of interpreters to all users 

![image](https://cloud.githubusercontent.com/assets/3348133/17102852/6a7622b6-52b6-11e6-98ec-06f120413ee5.png)

- when you want to grant permission of interpreters to specific user

![image](https://cloud.githubusercontent.com/assets/3348133/17102416/7184d4be-52b4-11e6-87b4-74211dd60c90.png)



---
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] zeppelin issue #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-25 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@Leemoonsoo I would say, since the use case for this is different than that 
of notebook, and idea for this is to map access to users with interpreters. 

While reading the title/description of this PR this is what I had in my mind



![user 
setting](https://cloud.githubusercontent.com/assets/674497/17094158/21487652-526b-11e6-8752-9c6797c614da.png)

Also think about the case where user is getting auth using LDAP/AD where 
there are 100s of user, so this will require pagination. So, we need to have a 
group/profile for users that can have preset for grouping interpreters together.


![user 
profile](https://cloud.githubusercontent.com/assets/674497/17094169/37694a4c-526b-11e6-9dc9-f2c9ad1a10f1.png)

And once this is set, IMO, then user who don't have permission to a 
particular interpreter, should not see that interpreter in the notebook page.



---
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] zeppelin issue #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-23 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@Leemoonsoo Thank you for your review and I think your opinion makes sense. 
I'll change the UI.


---
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] zeppelin issue #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@astroshim Thanks for the contribution.

Regarding UI, why don't we use the same UI with notebook authorization?
That can give the same user experience for all the authorization inside of 
Zeppelin.

Than it'll look something like


![image](https://cloud.githubusercontent.com/assets/1540981/17080923/92df8ace-517d-11e6-8b96-4c51989ce641.png)

What do you think?


---
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.
---