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 [email protected] or file a JIRA ticket
with INFRA.
---
