[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread likitha
Github user likitha commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117526563 Yes, now in addition to HyperV, even in case of VMware we want CS to chose the host containing the VM as the endpoint when the storage actions is 'MIGRATEVOLUME'.

[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117537697 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 pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117541379 Hi @likitha And what about DeleteVolume? It now supports only VMware, but the MigrateVolume supports both VMware and HyperV. It would be nice to

[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread likitha
Github user likitha commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117559997 Thanks for taking the time to review @wilderrodrigues. I have updated the issue description. As far as 'StorageAction.DELETEVOLUME' is concerned I am not

[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/541 --- 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: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread likitha
Github user likitha commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117515627 @wilderrodrigues , Background - 1. When CS tries to live migrate a volume, it chooses a host endpoint to perform the migration by selecting any host that

[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-07-01 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/541#issuecomment-117517309 Hi @likitha I understood what the issue says, but what I really meant concerns what the code does. The only practical change in the code

[GitHub] cloudstack pull request: CLOUDSTACK-8597. Failed to migrate volume...

2015-06-30 Thread likitha
GitHub user likitha opened a pull request: https://github.com/apache/cloudstack/pull/541 CLOUDSTACK-8597. Failed to migrate volume from zone-wide to cluster-w… …ide storage. While live migrating a volume, CS chooses the endpoint to perform the migration by selecting