[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-8999: Don't overr...

2015-10-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/991#issuecomment-151784718 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 enabled

[GitHub] cloudstack pull request: CLOUDSTACk-9002: VM deployment is success...

2015-10-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/995#issuecomment-151784319 Seems good to me, but while we are working on this. Any way we can Unit Test this? Looking at the code it's possible. If you could just write a test to validate

[GitHub] cloudstack pull request: CLOUDSTACK-8991 - IP address is not remov...

2015-10-28 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/989#issuecomment-151782444 Ping @remibergsma @borisroman @karuturi @DaanHoogland @miguelaferreira @michaelandersen Test results: * Hardware required: false * Manag

[GitHub] cloudstack pull request: Add all tests in /test/integration/smoke ...

2015-10-28 Thread runseb
GitHub user runseb opened a pull request: https://github.com/apache/cloudstack/pull/997 Add all tests in /test/integration/smoke to Travis run and nove nicir… Adds all tests in smoke folder to Travis. Moves nicira test to /test/integration/plugins You can merge this pull requ

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9000: logrotate cloudsta...

2015-10-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/993#issuecomment-151778493 Why only RPM and not for DEB? Keep in mind though that with systemd this is no longer valid --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/992#issuecomment-151778449 Why only RPM and not for DEB? --- 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 h

[GitHub] cloudstack pull request: Add all tests in /test/integration/smoke ...

2015-10-28 Thread runseb
Github user runseb closed the pull request at: https://github.com/apache/cloudstack/pull/920 --- 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: [4.6] CLOUDSTACK-8994 Add logging for the...

2015-10-28 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/983#issuecomment-151776994 How does this impact logrotating? --- 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 d

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-8994 Add logging for the...

2015-10-28 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/983#issuecomment-151773344 @borisroman I started seeing below error messages on console and VR doesnt start. Is it related to this change? ``` [ 480.108811] "echo 0 > /proc/

[GitHub] cloudstack pull request: CLOUDSTACK-8993: DHCP fails with "no addr...

2015-10-28 Thread serbaut
Github user serbaut commented on the pull request: https://github.com/apache/cloudstack/pull/981#issuecomment-151768463 In our environment it was really easy to reproduce. * Deploy instance 1, got IP 10.7.32.101 * Delete and expunge instance 1 * Deploy instance 2, got IP 100

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-28 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151759873 @K0zka I also cleaned the folder /var/cache/cloudstack/management/work/, it does not fix the issue. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-28 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-151757451 @K0zka (1) not working: ``` [root@weizhou master]# curl -v 10.11.115.10:8080/client/ -o /dev/null * About to connect() to 10.11.115.10 port 8

[GitHub] cloudstack pull request: CLOUDSTACK-8993: DHCP fails with "no addr...

2015-10-28 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/981#issuecomment-151755364 Hi @serbaut , Could you write an integration test with marvin to cover this change? If you don't have a test environment or are not familiar with Mar

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...

2015-10-28 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/979#issuecomment-151755463 Ping @remibergsma @DaanHoogland Any time to test this one today? Cheers, Wilder --- If your project is set up for it, you can reply to

Re: Build failure for plugin ovm3

2015-10-28 Thread Daan Hoogland
Sateesh, I really don't know what the root cause of your problem is. Downgrading is worth a try but no guarantees. I don't see anything in your reported platform that rings alarms. On Wed, Oct 28, 2015 at 2:03 AM, Sateesh Chodapuneedi < sateesh.chodapune...@citrix.com> wrote: > Hi Daan, > > Than

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-8984: VPC Network offeri...

2015-10-28 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/996 [4.6] CLOUDSTACK-8984: VPC Network offerings tab missing from UI This is a regression from commit af2f21894ce061faadc8cec29b901719303a29dc added vpcofferings to the select list

[GitHub] cloudstack pull request: CLOUDSTACk-9002: VM deployment is success...

2015-10-28 Thread kansal
GitHub user kansal opened a pull request: https://github.com/apache/cloudstack/pull/995 CLOUDSTACk-9002: VM deployment is successful even when dhcp entry com… …mand fails - Fixed Reason: The return value of the call to accept() function in the applyRules() function of B

<    1   2