GitHub user khalidhuseynov reopened a pull request:
https://github.com/apache/zeppelin/pull/1625
[ZEPPELIN-1657] Private/public mode for user note creation/import
### What is this PR for?
In multi-user environment when users create a notebook normally they would
expect that notebook be listed only in their list/workbench. Currently Zeppelin
creates/imports notes as public by default. There should be at least a way to
pass through configuration to make this behaviour private be default. Should
discuss whether it's public or private by default.
### What type of PR is it?
Improvement
### Todos
* [x] - set permissions on create/import
* [x] - test
* [x] - review, feedback, decide whether public/private by default
* [x] - pass through env. config
### What is the Jira issue?
[ZEPPELIN-1657](https://issues.apache.org/jira/browse/ZEPPELIN-1657)
### How should this be tested?
1. set `zeppelin.notebook.public` property as false in `zeppelin-site.xml`
2. login as user1 and create noteA, note should appear in your list of notes
3. logout and login as user2, shouldn't be able to see noteA
### Screenshots (if appropriate)
TBD
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? maybe
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/khalidhuseynov/incubator-zeppelin
feat/public-note-create
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1625.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1625
----
commit b7f28b30616419d22b62d116f70568821c6f3711
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-11T09:13:18Z
add property to site.xml
commit 4062af09c9a585f7a9dd164bf3d8b90ceb3bde75
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-11T10:03:54Z
add var to env.sh
commit 97324096f674d61dee732561d986db728985d2fa
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-11T10:52:23Z
add variable to ZeppelinConfiguration
commit 44297a40fd67ce3fe25756417091e4fa6ccf9176
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-13T13:15:13Z
set new note permissions
commit 28abffac6a4f6fd8514fdea706af12a42958406e
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-13T13:15:51Z
set permissions on note create and import
commit 496b80c0823494d9c532574bcd69538b8bb2a3d8
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-13T16:57:51Z
read env var from conf file directly
commit 1a945e8e183ec1baab807b5c9ce522274093b061
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-13T16:58:04Z
add test
commit 876b79880ad0426f691e390c5e8b25baef35b380
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-14T03:39:40Z
fix vfs testSave npe
commit f862ae1e875d51c18c353b9eda9cd2d0734f5a12
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-14T04:30:01Z
fix InterpreterFactory test
commit fc91b2f15725e91db1cbdf0439e8dd8779fc4f68
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-16T08:14:44Z
add description to install config
commit 010c26200d274620eea65a414f58dad8bbaa47f0
Author: Khalid Huseynov <[email protected]>
Date: 2016-11-17T05:53:10Z
fix test: set state back
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
