[GitHub] cloudstack pull request: [BLOCKER] Combined PRs that fix VR issues

2015-09-26 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/887#issuecomment-143409077 Update: we found a bug introduced in PR #784 (that we merge here) that needs resolving. Working on that with @wilderrodrigues as we speak so don't merge yet. -

[GitHub] cloudstack pull request: [BLOCKER] Combined PRs that fix VR issues

2015-09-26 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/887#issuecomment-143409187 This PR also fixes CLOUDSTACK-8878 (https://issues.apache.org/jira/browse/CLOUDSTACK-8878): No route to host on VPC router after stop/start/reboot --- If your

Re: Blameless post mortem

2015-09-26 Thread sebgoa
Remi, thanks for the detailed post-mortem, it's a good read and great learning. I hope everyone reads it. The one thing to emphasize is that we now have a very visible way to get code into master, we have folks investing time to provide review (great), we need the submitters to make due diligenc

[GitHub] cloudstack pull request: CLOUDSTACK-8913: Search box in Templates ...

2015-09-26 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/891 CLOUDSTACK-8913: Search box in Templates tab out of alignment Increased the margin and padding to accomodate all the boxes inside toolbar. You can merge this pull request into a Git repos

[GitHub] cloudstack pull request: CLOUDSTACK-8913: Search box in Templates ...

2015-09-26 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/891#issuecomment-143419034 Sorry Its Decreased not increased. --- 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 pr

[BLOCKER] CLOUDSTACK-8848

2015-09-26 Thread Rene Moser
I discovered the race condition bug related to CLOUDSTACK-8848 while testing in our lab and daan started a PR https://github.com/apache/cloudstack/pull/829 for discussion. But it turned out to be a dead end discussion. Daan and I started a debug session on Friday a week ago and we discovered t

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-26 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-143442228 We don't need new method isPowerStateUpToDate(long instanceId) as we are already performing the same operations in _instanceDao.updatePowerState(entry.getKey(), h

Re: [BLOCKER] CLOUDSTACK-8848

2015-09-26 Thread Rajani Karuturi
Thanks for the update rene. Unfortunately I do not have a vmware setup to test. I will do a code review tomorrow and run some tests on xen to make sure its not breaking anything else On శని., సెప్టెంబర్ 26, 2015 at 17:39 అపరాహ్న, Rene Moser < m...@renemoser.net> wrote: I discovered the race cond

Re: Need Info on configuring cloudstack HA

2015-09-26 Thread Nux!
Hm, this kind of sucks, did not know this detail that the hypervisor needs to come back and report (though I understand why this happens). If the hypervisor machine crashes, then it could need repairing for some time.. it's not good that the VM will be down during this period. Any work around to

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2015-09-26 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-143475690 @SudharmaJain The title and the content of your PR do not match. You seem to re-implement the XenServer 6.0.2 resource? If that is what you want, make a PR just

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2015-09-26 Thread SudharmaJain
Github user SudharmaJain commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-143478261 @remibergsma I didn't checked the PR #861. I was looking to fix the issue with xenserver 6.0.2 and higher. For this I have added a logrotate script and a Patch

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-26 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-143480210 LGTM :+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 this feature e

[GitHub] cloudstack pull request: [BLOCKER] Combined PRs that fix VR issues

2015-09-26 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/887#issuecomment-143524179 @wilderrodrigues and I spent yesterday testing and fixing small issues. I just (force) pushed the commits we that we tested against over night. The follo

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-26 Thread remibergsma
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