[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user phillipkent commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-206907782 @bhaisaab : Thanks! --- 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

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-cloudmonkey/pull/11 --- 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

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-206823599 @phillipkent I hit this issue today, while passing an argument. this worked for me. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-171620358 I've tried the update network API with ACS 4.5.2 and 4.7.0 and it works for me, so it could be a 4.3.2 related limitation. In that case, we can have

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user phillipkent commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-171242939 @bhaisaab I tested a clean virtualenv install of Cloudmonkey 5.3.2 (from PyPy) and also a clone of the latest Cloudmonkey from github (Python version

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170516429 @phillipkent I understand, but looks like it still works without your fix. I was able to update network display test with a * in it: Update

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user phillipkent commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170278540 @bhaisaab The commands that I tested were to set or modify the displaytext/displayname for a network or virtual machine, such as " update network

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170056433 LGTM --- 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

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170059404 @phillipkent can you share examples of API/cloudmonkey commands where we would like to use * in the args? --- If your project is set up for it, you

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

GitHub user phillipkent opened a pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11 Update requester.py urllib.quote_plus() requires the option safe="*" (that is, the * character is not encoded) to be consistent with CloudStack's internal encoder/decoder.