You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Chip Childers <ch...@apache.org> on 2013/06/26 20:15:52 UTC
Re: Review Request 12106: Add Hyperv-V support to ListHypervisorsCmd API call
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/
-----------------------------------------------------------
(Updated June 26, 2013, 6:15 p.m.)
Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
Changes
-------
Adding the cloudstack group.
Repository: cloudstack-git
Description
-------
Add Hyperv-V support to ListHypervisorsCmd API call
Diffs
-----
agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
Diff: https://reviews.apache.org/r/12106/diff/
Testing
-------
Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
Thanks,
Donal Lafferty
Re: Review Request 12106: Removed Dead Code from Management Server Hyper-V
2012 Support
Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/#review22605
-----------------------------------------------------------
commit 4361418e415b81567423367c0bd3aff263cc15c8
Author: Donal Lafferty <do...@citrix.com>
Date: Thu Jun 27 11:46:02 2013 -0700
Closing the review
- Prasanna Santhanam
On June 27, 2013, 8:56 a.m., Donal Lafferty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12106/
> -----------------------------------------------------------
>
> (Updated June 27, 2013, 8:56 a.m.)
>
>
> Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Update ImageFormat enum to include VHDX format introduced with Hyper-V Server 2012.
> Remove existing Hyper-V plugin, because it does not work and is dead code.
> Remove references to existing Hyper-V plugin from config files.
> Remove Hypervisor.HypervisorType.Hyperv special cases from manager code that are unused or unsupported.
> Specifically, there is no CIFS secondary storage class "CifsSecondaryStorageResource". Also, the Hyper-V plugin's ServerResource is contacted by the management server and not the other way around.
> Add Hyperv-V support to ListHypervisorsCmd API call
>
>
> Diffs
> -----
>
> agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
> api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
> client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
> core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
> engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
> plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
> server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
> server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
> server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
> server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
> server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
>
> Diff: https://reviews.apache.org/r/12106/diff/
>
>
> Testing
> -------
>
> Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
>
>
> Thanks,
>
> Donal Lafferty
>
>
Re: Review Request 12106: Removed Dead Code from Management Server Hyper-V
2012 Support
Posted by edison su <ed...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/#review22480
-----------------------------------------------------------
Ship it!
Ship It!
- edison su
On June 27, 2013, 8:56 a.m., Donal Lafferty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12106/
> -----------------------------------------------------------
>
> (Updated June 27, 2013, 8:56 a.m.)
>
>
> Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Update ImageFormat enum to include VHDX format introduced with Hyper-V Server 2012.
> Remove existing Hyper-V plugin, because it does not work and is dead code.
> Remove references to existing Hyper-V plugin from config files.
> Remove Hypervisor.HypervisorType.Hyperv special cases from manager code that are unused or unsupported.
> Specifically, there is no CIFS secondary storage class "CifsSecondaryStorageResource". Also, the Hyper-V plugin's ServerResource is contacted by the management server and not the other way around.
> Add Hyperv-V support to ListHypervisorsCmd API call
>
>
> Diffs
> -----
>
> agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
> api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
> client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
> core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
> engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
> plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
> server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
> server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
> server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
> server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
> server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
>
> Diff: https://reviews.apache.org/r/12106/diff/
>
>
> Testing
> -------
>
> Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
>
>
> Thanks,
>
> Donal Lafferty
>
>
Re: Review Request 12106: Removed Dead Code from Management Server Hyper-V
2012 Support
Posted by Donal Lafferty <do...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/
-----------------------------------------------------------
(Updated June 27, 2013, 8:56 a.m.)
Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
Changes
-------
Move Review requests 12105, 12103, 12085, because this submission duplicates the diffs in those review requests.
Summary (updated)
-----------------
Removed Dead Code from Management Server Hyper-V 2012 Support
Repository: cloudstack-git
Description (updated)
-------
Update ImageFormat enum to include VHDX format introduced with Hyper-V Server 2012.
Remove existing Hyper-V plugin, because it does not work and is dead code.
Remove references to existing Hyper-V plugin from config files.
Remove Hypervisor.HypervisorType.Hyperv special cases from manager code that are unused or unsupported.
Specifically, there is no CIFS secondary storage class "CifsSecondaryStorageResource". Also, the Hyper-V plugin's ServerResource is contacted by the management server and not the other way around.
Add Hyperv-V support to ListHypervisorsCmd API call
Diffs
-----
agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
Diff: https://reviews.apache.org/r/12106/diff/
Testing
-------
Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
Thanks,
Donal Lafferty
Re: Review Request 12106: Add Hyperv-V support to ListHypervisorsCmd API call
Posted by Donal Lafferty <do...@citrix.com>.
> On June 26, 2013, 6:20 p.m., David Nalley wrote:
> > Despite different descriptions, 12105 and 12106 appear to be the same patch.
They're different. 12106's patch was generated by post-review. By default, post-review included patches submitted in 12106, 12105, 12103, and 12085, because their changes were not added to Master.
I can moved the comments from 12105, 12103, and 12085 to 12106, and close the earlier Review Board submissions.
- Donal
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/#review22411
-----------------------------------------------------------
On June 26, 2013, 6:15 p.m., Donal Lafferty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12106/
> -----------------------------------------------------------
>
> (Updated June 26, 2013, 6:15 p.m.)
>
>
> Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Add Hyperv-V support to ListHypervisorsCmd API call
>
>
> Diffs
> -----
>
> agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
> api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
> client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
> core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
> engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
> plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
> server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
> server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
> server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
> server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
> server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
>
> Diff: https://reviews.apache.org/r/12106/diff/
>
>
> Testing
> -------
>
> Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
>
>
> Thanks,
>
> Donal Lafferty
>
>
Re: Review Request 12106: Add Hyperv-V support to ListHypervisorsCmd API call
Posted by David Nalley <da...@gnsa.us>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12106/#review22411
-----------------------------------------------------------
Despite different descriptions, 12105 and 12106 appear to be the same patch.
- David Nalley
On June 26, 2013, 6:15 p.m., Donal Lafferty wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12106/
> -----------------------------------------------------------
>
> (Updated June 26, 2013, 6:15 p.m.)
>
>
> Review request for cloudstack, Alex Huang, Chip Childers, and edison su.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Add Hyperv-V support to ListHypervisorsCmd API call
>
>
> Diffs
> -----
>
> agent/src/com/cloud/agent/VmmAgentShell.java 190d1168284243f9e860677a03311da9db31c745
> api/src/com/cloud/storage/Storage.java 9a50ffa786c9a1b516baa7b8bce04de7d3c7b2bc
> client/tomcatconf/applicationContext.xml.in ac1f3e46fdb43b4a5e38f9f2b3c498a2c258a0aa
> core/src/com/cloud/hypervisor/hyperv/resource/HypervResource.java 725f0cc1ae214447f20f5cd51c40e7e9128f0506
> engine/storage/integration-test/test/resource/component.xml 5ba87e8ebe9a682b320c60caf0e0057d2eb92027
> plugins/network-elements/dns-notifier/resources/components-example.xml 717c5e063fe0271eeb1143215a493c6342c5811e
> server/src/com/cloud/configuration/Config.java ba508495c350714fff90da0923b046783d42b1c3
> server/src/com/cloud/hypervisor/guru/HypervGuru.java 630080e21cbb6a501d67bda41179feb278b3d79c
> server/src/com/cloud/hypervisor/hyperv/HypervServerDiscoverer.java 06658b7f3e2ea5e80edf012dcaf29d980758084d
> server/src/com/cloud/resource/ResourceManagerImpl.java fe91cb337d0f5901012ca45e131d9cb2f7c54cf2
> server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 954c7e970f02e7ba0eae6e8e1616d9a08c9168b0
>
> Diff: https://reviews.apache.org/r/12106/diff/
>
>
> Testing
> -------
>
> Compiled using mvn clean install, used in integration test on cshv3 branch from github mirror.
>
>
> Thanks,
>
> Donal Lafferty
>
>