[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

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

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164271512 @bhaisaab I've already build the RPM's! We use them when running integration tests, so nothing broke in that area. I'll build the DEB packages now. ---

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164270975 I think we might need to test if package building still works with the changes, i.e. is rpm/deb packaging are using or referencing it. Possible test would be to bu

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164256015 @borisroman I must learn to be less brittish ;) LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164255977 @DaanHoogland That's an LGTM :+1: right? :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164208189 OK, that makes sense. let's do it. if we need it there is always git --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164205140 Hi @DaanHoogland In late 2012 it was replaced with cloudmonkey. It's just some code that was left behind. It's not touched since halfway through 2012.

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164204591 I'm hesitant, is really nothing usefull or being used in there? I would like to know why this became out of use before disgarding it. --- If your project is s

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164170827 LGTM based on the above tests. Makes no sense to run the same tests again :-) --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1228#issuecomment-164162326 *Environment* - 1 KVM host on CentOS 7.1 - 1 Management Server on CentOS 7.1 - Agent + Common RPMs built from source *Integration test suit

[GitHub] cloudstack pull request: Removed cloud-cli folder and contents, as...

GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1228 Removed cloud-cli folder and contents, as it is not maintained or used anymore. Remove legacy code. You can merge this pull request into a Git repository by running: $ git pull https:/