[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/908#issuecomment-145210488 Hi @borisroman Yes, agree on the cleanup, I mentioned in the PR that it was broken. Since I'm working on improving the tests, I will create

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/908#discussion_r41084587 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py --- @@ -96,7 +96,8 @@ def _redundant_on(self): d = s.

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Enable dynamic scaling t...

2015-10-02 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/902#issuecomment-145200154 @sanju1010 this is not the right file - the following file needs to be updated - https://github.com/apache/cloudstack/blob/94f1ebb60af320166206c213e18b97a177bc145a/

[GitHub] cloudstack-cloudmonkey pull request: Import error when running clo...

2015-10-02 Thread syed
GitHub user syed opened a pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/10 Import error when running cloudmonkey The cloudmonkey install from pip does not run and gives an import error because the latest `requests` package dropped the `requests.packages.urllib3

[GitHub] cloudstack pull request: CLOUDSTACK-8902 Restart Network fails in ...

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/898#issuecomment-145157311 Hi @bvbharatk can you please run BVT tests and share output? --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

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

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-145157098 @SudharmaJain Please respond to comments. If you agree, please close this PR. Thanks! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/899#issuecomment-145154881 @yvsubhash Please update the title of the PR to be more descriptive. --- If your project is set up for it, you can reply to this email and have your reply appear

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

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145154565 Hi @DaanHoogland can you please run the same tests as we did on PR #885 and post output? Let me know if you need help. --- If your project is set up for it, you

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread remibergsma
Github user remibergsma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/908#discussion_r4105 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py --- @@ -96,7 +96,8 @@ def _redundant_on(self): d = s.repl

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/908#issuecomment-145079681 @wilderrodrigues I've ran the same tests as you proposed. On Centos KVM hosts. Resulting in success besides the same exceptions. So that's :+1: LGTM Rega

Build failed in Jenkins: build-master-jdk18 #360

2015-10-02 Thread jenkins
See Changes: [Rajani Karuturi] CLOUDSTACK-8808: Successfully registered VHD template is downloaded -- [...truncated 398 lines...] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time el

[GitHub] cloudstack pull request: CLOUDSTACK-8848

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145037606 @DaanHoogland Could you please add a bit more descriptive title? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-10-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/901 --- 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: CLOUDSTACK-8808: Successfully registered ...

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/901#issuecomment-145036915 Thanks @borisroman, merged! --- 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

Re: [VOTE] jenkins jobs removal

2015-10-02 Thread Sebastien Goasguen
+1 for sure on all build-docs jobs > On Oct 2, 2015, at 2:32 PM, Boris Schrijver wrote: > > Hi, > > +1 on all. > > -- > > Met vriendelijke groet / Kind regards, > > Boris Schrijver > > PCextreme B.V. > > http://www.pcextreme.nl/contact > Tel direct: +31 6 33784542 > >> >>On October

Re: [VOTE] jenkins jobs removal

2015-10-02 Thread Boris Schrijver
Hi, +1 on all. -- Met vriendelijke groet / Kind regards, Boris Schrijver PCextreme B.V. http://www.pcextreme.nl/contact Tel direct: +31 6 33784542 > > On October 2, 2015 at 8:47 AM Remi Bergsma > wrote: > > > +1 on 4.3 jobs, although I'd like to hear from Rohit to confirm > >

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-10-02 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/901#issuecomment-145001598 @remibergsma Anwser is sufficient. I'll cleanup after 4.6 is released. Regarding the PR: :+1: LGTM I ran the test_vm_lifecycle which succeeded an

Re: BVT report 10/1

2015-10-02 Thread Wilder Rodrigues
Hi Sebastien, We are still analysing if the root cause is the same. But anyway, it doesn’t work on 4.4.4. Michael is now busy with improving an existing smoke test for internal loan balancing. He just told me that the existing test does not cover connectivity. I had a look at it, and it doesn’

[GitHub] cloudstack pull request: CLOUDSTACK-8848

2015-10-02 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/909 CLOUDSTACK-8848 added a null guard to @resmo's #885 A unit test or two would be nice as well but as this is a blocker I want to get it to review asap. @koushik-das @wilderrodrigues @ans

Re: BVT report 10/1

2015-10-02 Thread Sebastien Goasguen
> On Oct 1, 2015, at 4:47 PM, Wilder Rodrigues > wrote: > > Sebastien, Remi and Raja… > > Michael and I just tested that one on 4.4.4, and it also happens there: > > https://issues.apache.org/jira/browse/CLOUDSTACK-8697 > Thanks for testing this as well as the detailed detective work from M

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/908#issuecomment-144969500 Ping @remibergsma @DaanHoogland @karuturi @miguelaferreira @borisroman @wido Test environment: * One ACS Management Server running on CentO

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-02 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/908 CLOUDSTACK-8915 - Cannot SSH into VMs deployed Redundant VPC routers In order to reproduce the problem, I did the following * Create a Redundant VPC * Add a tier * Add a

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

2015-10-02 Thread Remi Bergsma
Let's merge this as-is for 4.6 and solve a blocker. The improvement can be done later in a separate PR. Sent from my iPhone > On 02 Oct 2015, at 10:29, DaanHoogland wrote: > > Github user DaanHoogland commented on the pull request: > >https://github.com/apache/cloudstack/pull/885#issueco

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

2015-10-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144958879 Had a look, the NPE won't occur within the scope of this fix. I would suggest some changes anyway to prevent future issues as the method isPowerStateUpToDate()

[GitHub] cloudstack pull request: CLOUDSTACK-8848: extra state to handle; n...

2015-10-02 Thread DaanHoogland
Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/829 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-8848: extra state to handle; n...

2015-10-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/829#issuecomment-144949845 This change will have to go in a more solid and complete refactor, probably in smaller steps. The fix it is intended for will go in #885. --- If your project