[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113882368 LGTM waiting for travis to agree --- 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

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/482 --- 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: This is a quick fix to attempt to resolve...

2015-06-20 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113852244 I'd say this now downloads around 99% of project dependencies before build starts Mostly just maven-remote-resources-plugin stuff is still downloaded, plus

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113644292 It seems latest fix uncovered tests that were failing but actually reporting success! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113522017 Are you done with this @rsafonseca ? As in; should I merge? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113487647 7 successes in a row is looking better :) Will do a couple more minor changes to force some more runs.. will rebase at the end when it feels like travis is

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113532978 @DaanHoogland LGTM, altough travis has been very stable today :) Lastest commit is also useful, since the after_failure.sh script was doing nothing at all..

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113577142 Perfect, last travis failure was just what i was referring to in the last comment, after_failure.sh printed full marvin logs, we can properly troubleshoot that

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113585258 It seems like this test sometimes fails because the SSVM takes a while to startup.. the test/integration/smoke/test_secondary_storage.py:159 asserts that

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113614309 Ever since i first saw this failure I wondered... why is this considered a success?!? === TestName: test_02_sys_template_ready | Status : EXCEPTION ===

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-19 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113610266 Mentioned issue fixed in https://github.com/apache/cloudstack/pull/493/ :D --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-18 Thread rsafonseca
GitHub user rsafonseca opened a pull request: https://github.com/apache/cloudstack/pull/482 This is a quick fix to attempt to resolve most of the travis failures Most of the failures have been due to transient network failures, that are causing dependency artifact downloads to fail

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-18 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113284857 Travis failure on 2c0d07e was due to second phase not running after i moved the script to tools/travis xerxesImpl is downloaded on that phase Issue is

[GitHub] cloudstack pull request: This is a quick fix to attempt to resolve...

2015-06-18 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/482#issuecomment-113321916 Travis is still not looking very well :( will resume soon --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub