Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[Prabhjyot Singh]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/
---

(Updated Sept. 26, 2017, 4:10 p.m.)


Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit 
Choudhary, Sumit Mohanty, and venkat sairam.


Bugs: AMBARI-22052
https://issues.apache.org/jira/browse/AMBARI-22052


Repository: ambari


Description
---

Zeppelin's conf HDFS directory owner is wrong. However, on restart of Zeppelin 
server it takes the correct value

```
[zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
Found 4 items
drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 /user/zeppelin/notebook
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
```


Diffs
-

  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
 382566eff7 
  ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
f379f40534 


Diff: https://reviews.apache.org/r/62546/diff/1/


Testing
---

Manually


File Attachments (updated)


AMBARI-22052_branch-2.6_v1.patch
  
https://reviews.apache.org/media/uploaded/files/2017/09/26/562b4dfe-320d-44c7-b20e-a98be47f868f__AMBARI-22052_branch-2.6_v1.patch


Thanks,

Prabhjyot Singh

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[Prabhjyot Singh]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/
---

(Updated Sept. 26, 2017, 2:04 p.m.)


Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit 
Choudhary, Sumit Mohanty, and venkat sairam.


Bugs: AMBARI-22052
https://issues.apache.org/jira/browse/AMBARI-22052


Repository: ambari


Description
---

Zeppelin's conf HDFS directory owner is wrong. However, on restart of Zeppelin 
server it takes the correct value

```
[zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
Found 4 items
drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 /user/zeppelin/notebook
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
```


Diffs
-

  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
 382566eff7 
  ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
f379f40534 


Diff: https://reviews.apache.org/r/62546/diff/1/


Testing
---

Manually


File Attachments (updated)


AMBARI-22052_branch-2.6_v1.patch
  
https://reviews.apache.org/media/uploaded/files/2017/09/26/962742eb-f021-48f5-a6cf-5ba9e2b81596__AMBARI-22052_branch-2.6_v1.patch


Thanks,

Prabhjyot Singh

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[Gaurav Nagar]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/#review186216
---


Ship it!




Ship It!

- Gaurav Nagar


On Sept. 25, 2017, 3:02 p.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62546/
> ---
> 
> (Updated Sept. 25, 2017, 3:02 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22052
> https://issues.apache.org/jira/browse/AMBARI-22052
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Zeppelin's conf HDFS directory owner is wrong. However, on restart of 
> Zeppelin server it takes the correct value
> 
> ```
> [zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
> Found 4 items
> drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
> drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 
> /user/zeppelin/notebook
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
> ```
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  382566eff7 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> f379f40534 
> 
> 
> Diff: https://reviews.apache.org/r/62546/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[venkat sairam]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/#review186130
---


Ship it!




Ship It!

- venkat sairam


On Sept. 25, 2017, 3:02 p.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62546/
> ---
> 
> (Updated Sept. 25, 2017, 3:02 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22052
> https://issues.apache.org/jira/browse/AMBARI-22052
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Zeppelin's conf HDFS directory owner is wrong. However, on restart of 
> Zeppelin server it takes the correct value
> 
> ```
> [zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
> Found 4 items
> drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
> drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 
> /user/zeppelin/notebook
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
> ```
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  382566eff7 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> f379f40534 
> 
> 
> Diff: https://reviews.apache.org/r/62546/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[Prabhjyot Singh]

> On Sept. 25, 2017, 8:34 p.m., venkat sairam wrote:
> > ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
> > Lines 361 (patched)
> > 
> >
> > Can we use owner for the same?
> > Or will there be seperate user and owner?

Owner was already there at line no 360 (but is not working) hence adding `user` 
as well.


- Prabhjyot


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/#review186125
---


On Sept. 25, 2017, 8:32 p.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62546/
> ---
> 
> (Updated Sept. 25, 2017, 8:32 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22052
> https://issues.apache.org/jira/browse/AMBARI-22052
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Zeppelin's conf HDFS directory owner is wrong. However, on restart of 
> Zeppelin server it takes the correct value
> 
> ```
> [zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
> Found 4 items
> drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
> drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 
> /user/zeppelin/notebook
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
> ```
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  382566eff7 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> f379f40534 
> 
> 
> Diff: https://reviews.apache.org/r/62546/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>

Re: Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[venkat sairam]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/#review186125
---




ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
Lines 361 (patched)


Can we use owner for the same?
Or will there be seperate user and owner?


- venkat sairam


On Sept. 25, 2017, 3:02 p.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62546/
> ---
> 
> (Updated Sept. 25, 2017, 3:02 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22052
> https://issues.apache.org/jira/browse/AMBARI-22052
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Zeppelin's conf HDFS directory owner is wrong. However, on restart of 
> Zeppelin server it takes the correct value
> 
> ```
> [zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
> Found 4 items
> drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
> drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 
> /user/zeppelin/notebook
> drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
> ```
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  382566eff7 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> f379f40534 
> 
> 
> Diff: https://reviews.apache.org/r/62546/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>

Review Request 62546: Zeppelin's conf HDFS directory owner is wrong

[Prabhjyot Singh]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62546/
---

Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit 
Choudhary, Sumit Mohanty, and venkat sairam.


Bugs: AMBARI-22052
https://issues.apache.org/jira/browse/AMBARI-22052


Repository: ambari


Description
---

Zeppelin's conf HDFS directory owner is wrong. However, on restart of Zeppelin 
server it takes the correct value

```
[zeppelin@test-clu-1 ~]$ hdfs dfs -ls /user/zeppelin
Found 4 items
drwx--   - zeppelin hdfs  0 2017-09-25 14:12 /user/zeppelin/.Trash
drwxr-xr-x   - hdfs hdfs  0 2017-09-25 14:33 /user/zeppelin/conf
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 14:08 /user/zeppelin/notebook
drwxr-xr-x   - zeppelin hdfs  0 2017-09-25 13:13 /user/zeppelin/test
```


Diffs
-

  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
 382566eff7 
  ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
f379f40534 


Diff: https://reviews.apache.org/r/62546/diff/1/


Testing
---

Manually


Thanks,

Prabhjyot Singh