rhtyd commented on a change in pull request #4069:
URL: https://github.com/apache/cloudstack/pull/4069#discussion_r422977620



##########
File path: debian/cloudstack-agent.postinst
##########
@@ -46,6 +46,11 @@ case "$1" in
             fi
         fi
 
+        # Create symbolic link for python
+        if [ ! -f "/usr/bin/python" ]; then
+            ln -s /usr/bin/python2 /usr/bin/python

Review comment:
       Hi @weizhouapache - my colleagues @shwstppr @davidjumani are working to 
support python2 to python3 in #4068, so all scripts that run on management 
server and kvm host towards 4.15/milestone. So this change is not needed, and 
may break things (we probably don't want misleading symlinks that could mean 
python2 or python3). cc @wido @GabrielBrascher 




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

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


Reply via email to