Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-219637824
@rhtyd
>please rebase against latest master and push -f.
Done.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-216189486
@priyankparihar 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 ha
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-214208033
@priyankparihar Are you saying that you have tested the fix in all
scenarios with different HVs (XS, KVM, Vmware)? Also if HV doesn't allow
certain operations it
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-214146326
@koushik-das
Currently UI does not allow migration between different versions, and
changing UI for this change is not elegant. Purpose of doing this chang
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212846127
@sateesh-chodapuneedi @koushik-das I will get back to you soon.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user sateesh-chodapuneedi commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212845624
>Purpose of doing this change is if some user intentionally want to do
migration from lower to higher then he can do it via API(from UI he can not do
th
Github user sateesh-chodapuneedi commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212842765
>>Currently UI does not allow migration between different versions, and
changing UI for this change is not elegant.
@priyankparihar
Why allow t
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212832800
@priyankparihar This version check removal is at a generic place and impact
all HVs. Since you have only verified for XS 6.2 -> 6.5, there are concerns as
to wha
Github user rajesh-battala commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212777172
Looks good to me.
---
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 ha
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164395806
>>"So a migration from 6.2 to 6.5 is allowed but not the other way around."
@agneya2001 Currently UI does not allow migration between different
versions,
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164381436
@runseb completely agree. But, what kind of proof should i present ?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user agneya2001 commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164376871
@priyankparihar As per XenMotion docs: The target host must have the same
or a more recent version of XenServer installed as the source host. So a
migration from
Github user runseb commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164369338
-1
Just on the fact that "it runs on my machine" is the single most horrible
reason to justify working software.
---
If your project is set up for it,
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164354949
@DaanHoogland I have tested it on my machines. what kind of proof do you
need ?
---
If your project is set up for it, you can reply to this email and have
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-164294423
a test setup for this is not trivial, I can live with it but would like to
see some proof before merging.
---
If your project is set up for it, you can reply t
Github user priyankparihar commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-163542539
@DaanHoogland I did this change because UI does not allow migration
between different versions of hyper-visors of but sometimes user wants to do
migration f
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-162455918
Does what it says but probably more. Do we not mind any other version?
@priyankparihar can you show how ACS doesn't allow invalid migrations, "Storage
XenMotion
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-162346266
Pinging @DaanHoogland to review. Will run some tests.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-140650617
LGTM cc @abhinandanprateek
---
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
GitHub user priyankparihar opened a pull request:
https://github.com/apache/cloudstack/pull/815
CLOUDSTACK-8841: Storage XenMotion from XS 6.2 to XS 6.5 fails.
Removed Host version check in API. Because
Case 1:(Lower to Higher Version)
Storage XenMotion from lower version t
20 matches
Mail list logo
