[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-20 Thread borisroman
Github user borisroman closed the pull request at: https://github.com/apache/cloudstack/pull/1238 --- 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 featur

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-20 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-173186580 Closing! --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-17 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-172384379 LGTM based on the files, no tests were made. What is happening with Jenkins? I noticed that many PRs are flagged with error because Jenkins did not work. I

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-171251445 @borisroman can you post any results if you have them? If you care you can force push to have jenkins validate the PR but as far as I can see no added value to

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-171215623 LGTM , I'll look at jenkins again. --- 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 proj

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-12 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-171046691 Nice @borisroman, can you send it to test it again? It seems that Jenkins is not working... --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2015-12-13 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1238 CLOUDSTACK-9160: Remove unused folder(s)/file(s); engine/api/src/org/… …apache/engine/subsystem/api/storage/disktype Empty files. Moved DiskFormat.java to engine/api/src/org/apa