HTTP 500 trying to mkdir as 'admin' user which is not hdfs user

Review Request #2059 — Created May 2, 2012 and submitted

Information
romain
old-hue-rw
Reviewers
hue
bcwalrus 
commit db0e6089450b35880bf1e60c4ef972db1ee4948e
Author: Romain Rigaux <romain@cloudera.com>
Date:   Wed May 2 18:08:50 2012 -0700

    HTTP 500 trying to mkdir as "admin" user which is not "hdfs" user
    
    Add a note in the error message when the user is a Hue admin but
    not a HDFS root.
    Remove the notice from chown in order to be consistent with all
    the other actions.

:100644 100644 515b469... 6df6836... M	apps/filebrowser/src/filebrowser/templates/chown.mako
:100755 100755 fe4e91d... 10e54d8... M	apps/filebrowser/src/filebrowser/views.py
Ran tests
Visual testing, now we get:

"Cannot perform operation. Note: you are a Hue admin but not a HDFS superuser (which is "hdfs").

AccessControlException: Permission denied (error 403)"
bcwalrus
bcwalrus 
  1. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    




    
    Would keep it. Do you not find this message helpful?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      For me this message if also valid for any other actions (move, delete, rename...) so why put a note just for chown? (it is not consistent and a bit obvious)
      
 
      

      2. 



    2. 
 
 
      
 
      
  
 
      
 
      
  
      You don't need to be a cluster superuser to move/delete/rename. You just need permission on the directory. But chown _really_ needs root access.
      
 
      

      3. 



    3. 
 
 
      
 
      
  
 
      
 
      
  
      Ok I did not know about this, I am going to read/test about that.
      
 
      

      4. 


 
    


    

 
    

    3. 




  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




romain




   

   

  

 



 















 





 

  

   




romain




   

   





  

 



 

  


   

   

    
 
romain


    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    




    
    Tab removed
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




romain




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 







 


 










  

 









     

    

   

  

  

   
   Loading...