Github user ustcweizhou commented on the pull request:

    https://github.com/apache/cloudstack/pull/1027#issuecomment-153681436
  
    @wido @sspans 
    To be honest, this code was written by me about two years ago , which is 
not used for now. I created the PR because you guys can judge this code are 
useful or not. I have no comments if we revert some of the commits (or even 
all).
    1. for the default port, if most of you agree, I will revert the related 
commits.
    2. I agree. I did not remove it in the commits. However, I change the 
'vmName'.vport as the first virtio port, because cloud-early-config get the 
publickey and boot args from /dev/vport0p1.
    3. rom my point of view, python is easy for testing, the data send/receive 
are already implemented. If you want to replace it with Java, that's very good. 
for now, python works fine in my testing, I even use it for guest agent command 
testing. btw, do you want replace patachviasocket.pl with java ?  I replace it 
with the python script in the PR.



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to