[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-14 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-140009394 @remibergsma That sounds good. --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-14 Thread sureshanaparti
Github user sureshanaparti commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/806#discussion_r39372846 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareClient.java --- @@ -152,13 +152,26 @@ public void connect(String url, String

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-13 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-139962348 @remibergsma sure, but can do this next week (lack infra right now) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-12 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-139787067 @sureshanaparti Nice work! The PR title confused me a bit though. What about: CLOUDSTACK-8820: Support for VMware vCenter 6 data center @bhaisaab

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread sureshanaparti
GitHub user sureshanaparti opened a pull request: https://github.com/apache/cloudstack/pull/806 CLOUDSTACK-8820: Showing error when try to add vCenter 6 CLOUDSTACK-8820: Showing error when try to add advance zone using VMware ESXi 6.0 host Summary: In vCenter 6.0, response

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-139521285 Great PR @sureshanaparti I was going to work on supporting this as ESXi6 works well on top of KVM; I can test this for you next week; LGTM on code change (not

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/806#discussion_r39265018 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareClient.java --- @@ -152,13 +152,26 @@ public void connect(String url, String

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/806#discussion_r39265110 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareClient.java --- @@ -577,41 +590,48 @@ public ManagedObjectReference

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/806#discussion_r39265179 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareClient.java --- @@ -577,41 +590,48 @@ public ManagedObjectReference

[GitHub] cloudstack pull request: CLOUDSTACK-8820: Showing error when try t...

2015-09-11 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/806#issuecomment-139532380 change makes sense but I have some comments on the coding style. --- If your project is set up for it, you can reply to this email and have your reply appear