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 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 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 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 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 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 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 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 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 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 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 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 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 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
14 matches
Mail list logo