rusackas commented on PR #35190:
URL: https://github.com/apache/superset/pull/35190#issuecomment-3325034194

   Some questions that came up during group review:
   
   1) Are we SURE this is comprehensive? Maybe we should add a disclaimer that 
this is an evolving set of best practices
   
   2) Deployment setups vary wildly... some of these steps may apply to some 
deployments and not others, and other deployments may have safety checks that 
are not included here. 
   
   3) Should this even be a doc this long, or should we take all the 
sections/details you've written, and add details to the existing sections, and 
use the "checklist" as the main entry here to link to all of those detailed 
pages/instructions. 
   
   4) Should we add a step to delete the default admin user?
   
   5) For consideration, we could enhance the installation/start scripts to be 
a more interactive CLI asking about these options, and warning about more of 
them at the end with "Warning: X / Y / Z. Proceed? (y/N)"
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to