Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/861
---
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 user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-147113023
I commented on #883 and will merge this in a few days. Let's hope
@SudharmaJain responds.
---
If your project is set up for it, you can reply to this email and
Github user runseb commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-146119100
I agree on principles, but I have not tested any of this. would be good if
@SudharmaJain could confirm that this PR fixes his problem.
---
If your project is set up
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-146079758
@remibergsma +1
---
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 thi
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-145928165
@koushik-das @runseb, no response from @SudharmaJain on this matter. I
propose to merge this PR #861 and close the other (#883). Agree?
---
If your project is s
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-144500362
@koushik-das Thanks!
Another benefit: this is an easier change that doesn't require much testing
(all it does it execute a SQL query at upgrade time).
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-144347634
I have updated #883 with my comments, looking at the changes it shouldn't
be required. Lets wait for @SudharmaJain response
---
If your project is set up for it
Github user runseb commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-144320963
@harikrishna-patnala and @koushik-das I agree with @remibergsma please
advise on how you want to proceed, considering #883 reimplements the 602
resource.
---
If you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-143524258
@harikrishna-patnala Please check PR #883 If this gets in, then I think
your problem is also solved?
Maybe someone can check both PR #883 and #861 and de
Github user harikrishna-patnala commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-142174711
Thanks Koushik, here is the commit which removed the class
XenServer602Resource.
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/861#issuecomment-142173025
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 feature
e
GitHub user harikrishna-patnala opened a pull request:
https://github.com/apache/cloudstack/pull/861
CLOUDSTACK-: Xenserver 6.0.2 host stuck in disconnected state aftâ¦
Xenserver 6.0.2 hosts are stuck in disconnected state after CS upgrade to
4.6. This is because xenserver602re
12 matches
Mail list logo
