-
Notifications
You must be signed in to change notification settings - Fork 74
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
Set up continuous integration with Travis #20
Set up continuous integration with Travis #20
Conversation
369ffa6
to
92aaa3a
Compare
@hydrosquall yesterday was a big day for bugs squashing, may be if you update to the latest master 1.9 will work for you. |
@elephantum I think I am branched off the latest commit (c2af12e), but still getting this result. I left in some commented-out code for workarounds I tried (mostly setting |
Run as root user due to changes in python logging
92aaa3a
to
f6bdaa7
Compare
@hydrosquall I'll take a look this evening |
01dda23
to
69d2dc3
Compare
WIP: disable python2 pipeline
69d2dc3
to
8548a3d
Compare
@elephantum - I talked about this with my coworker @yegorski, who came up with an alternative to running as |
As a follow-up to epoch8#20, we can reveal the new users whether the master branch passes current tests.
Things to watch out for:
Based on the travis log messages when running in non-detached mode, I believe the issue is that the user inside the container is unable to create new folders on the host machine. (example, search for permission denied).
I tried a couple different options to get around this issue, but this seemed like the cleanest.
Other possible extras:
Do we want to be stricter about what constitutes "valid" input besides returning 200 status code? (A strict text match doesn't make sense since the memory values would be a little different every time). I think there's room to improve, but that this could still help as a first-pass.
Does epoch8 have an organization Travis account? For now I'm running the CI jobs on my personal account, but you can use this same config file to tie the tests into your organization account: https://travis-ci.org/hydrosquall/airflow-exporter