[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1037#issuecomment-216195896 @atrbgithub please rebase against latest master and push -f, update on status of your PR --- If your project is set up for it, you can reply to this email and have y

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-12-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1037#issuecomment-162995054 Hi @atrbgithub Yes, the info clarifies feel things, but unfortunately I'm also busy and do not have time to write a test to cover it. I also have s

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-12-08 Thread atrbgithub
Github user atrbgithub commented on the pull request: https://github.com/apache/cloudstack/pull/1037#issuecomment-162829832 Hi @wilderrodrigues To test this you would need to have 2 or more cloudstack management servers operating in a cluster with Xen hosts. Before applying t

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-12-08 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1037#issuecomment-162824106 Ping @bhaisaab @atrbgithub If I get some details on how to test - steps - I can give it a try. Cheers, Wilder --- If your project is

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-23 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1037#issuecomment-158875003 LGTM, anyone tested this yet? --- 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

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-158860647 Anyone tested this patch yet? --- 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 n

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-05 Thread atrbgithub
Github user atrbgithub commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-154042688 Fix for master branch has been proposed here https://github.com/apache/cloudstack/pull/1037 --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-05 Thread atrbgithub
GitHub user atrbgithub opened a pull request: https://github.com/apache/cloudstack/pull/1037 Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i… …n clustered management server environment This pull request relates to the following Jira bug report:

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-10-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-150836810 Hi @atrbgithub Thanks for your response. We indeed need to reimplement your fix on current master. Once that is in place and tested, we merge it on master (that

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-10-23 Thread atrbgithub
Github user atrbgithub commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-150595815 Hi Remi, I've forced-pushed the commit (amended the commit message and re-pushed) and it looks like the tests are running once more. I had a look at making a simi

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-10-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-150550247 Hi @atrbgithub thanks for the patch! Can you supply this patch also to master? We can only merge it, if it enters master first (will change soon, but right now t

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-10-12 Thread atrbgithub
GitHub user atrbgithub opened a pull request: https://github.com/apache/cloudstack/pull/925 Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i… …n clustered management server environment This pull request relates to the following Jira bug report: h