Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add debug information to travis build #1334

Merged
merged 1 commit into from
May 18, 2017
Merged

add debug information to travis build #1334

merged 1 commit into from
May 18, 2017

Conversation

kain88-de
Copy link
Member

The cron job build seems to fail currently. I'm not sure why that is there is also no indication in the travis log. So lets see what information we get if we add more debug info in the logs.

@kain88-de
Copy link
Member Author

there has been a travis update that might have caused this error

https://blog.travis-ci.com/2017-05-04-precise-image-updates

@jbarnoud
Copy link
Contributor

jbarnoud commented May 5, 2017

The develop branch now fails in the same way: the "numpy dev" build fails silently after the conda install. From what I see from the debug lines ou activated, it seems related to the ci-helper calling clear_console.

@kain88-de
Copy link
Member Author

clear_console is not part of the ci-helper scripts. But the scripts seem to exit with 1 for some reason now.

@kain88-de
Copy link
Member Author

also there was no update of the ci-helper scripts for almost a month. This all worked fine until yesterday

@kain88-de
Copy link
Member Author

Great the build stops and fails when the logout script on travis is getting called. So from the ci-helpers side everything is OK.

@kain88-de
Copy link
Member Author

OK removing the logout script lets this test pass again. But since this is a privacy feature maybe we shouldn't do that. I send a support ticket to Travis already earlier today. I'm waiting for a reply and then inform them about what I found out today.

@Lyoness
Copy link

Lyoness commented May 9, 2017

Hello there 👋

The error you are seeing was not caused by #1334 (comment)

as you are on our container-based infrastructure and we made updates only to full VM images.

The last build log url had logs that cut off, but we're looking into them in more depth to see why the logout script made the build fail.

@orbeckst
Copy link
Member

@kain88-de can we remove the cron job from our travis runs for the time being? It's pretty annoying to get the red "tests do not pass" everywhere even though it's not really the code's fault.

@kain88-de
Copy link
Member Author

@orbeckst sure you can deactivate the test for the time being

@kain88-de
Copy link
Member Author

This should be fixed now in upstream

@orbeckst
Copy link
Member

Yes, the cron job build passed (well, installed and did not fail with an error as previously).

Another job stalled so I restarted.

But it seems indeed that the travis cron job failure has been eliminated. Do you want to revert 429795a ?

@orbeckst orbeckst self-requested a review May 18, 2017 01:13
Copy link
Member

@orbeckst orbeckst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good, enable it again.

(You could have also done a git revert 429795a071eddb6cb351c4d2769b9e0537d4175e.)

@orbeckst orbeckst merged commit a59cd63 into develop May 18, 2017
@orbeckst orbeckst deleted the debug-travis branch May 18, 2017 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants