-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
BUG: Travis building on container-based infrastructure #12946
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
Closed
Closed
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
f48827d
Use sudo: required to stop travis building on container-based infrast…
nparley 3744114
Merge remote-tracking branch 'upstream/master' into travis
nparley 712e592
First go to try locale settings on travis
nparley 01bfe3f
lanuage-pack-zh is not allowed try -chan
nparley 65707fc
Fix spelling
nparley 70c429b
Test zh_CN.UTF-8 as that's allowed to be installed
nparley 09457f3
Remove more sudo
nparley ee23be8
Merge remote-tracking branch 'upstream/master' into travis
nparley 483c989
Delete sudo lines from script files
nparley 91cc6b3
Fix indent bug and add comments to builds which are also in allow_fai…
nparley 2b191bc
Add LOCALE to print versions
nparley a7ccaee
Print version info during test script
nparley 5eca9ce
Test blosc memory Backtrace
nparley 70f583d
Revert back previous change to fix in another PR
nparley 8aa1307
Merge remote-tracking branch 'upstream/master' into travis
nparley 6b03de1
Merge remote-tracking branch 'upstream/master' into travis
nparley 815444f
Experiment adding miniconda to cache
nparley 3b191dc
Fix missing $
nparley e9874ec
Experiment with caching
nparley 25aea25
Change cache name
nparley 30c09f4
Don't recreate pandas if miniconda is from cache
nparley 6bc70e0
Bug fix
nparley d9f0eb3
Try adding ccache to travis
nparley f3adf0a
Bug fix
nparley ecde407
Try caching ccache as well
nparley cf818c1
Bug fix
nparley 5659b0e
Bug fix
nparley 669c115
Test ccache
nparley e813f2d
Code clean up
nparley db0cd90
Test commit
nparley 6201825
bug fix
nparley 9158c0b
Bug fix
nparley eba7f08
Bug fix
nparley e81cd73
Touch msgpack
nparley 6e4d88d
Remove sudo even though file is not being used
nparley ab55c60
Change way version is displayed
nparley b73e7cc
Add use cache
nparley ba16f4a
Remove sudo even though file is not being used
nparley 4d01ef6
Change way version is displayed
nparley 4fc2e49
Merge branch 'travis_cache' into travis
nparley 1c5a687
Merge remote-tracking branch 'upstream/master' into travis
nparley 346ced8
Grep - P is not available on mac OS X
nparley 56cc93c
Add cython files to the cache even if we have not used the cache. The…
nparley ccd1125
Only on back 2 commits not 3 when looking for python file changes. Ma…
nparley 7d17f00
Turn off cache for 35_nslow
nparley 359f00b
Don't hard code in python files. Find them using the pyx files.
nparley 0867cf9
Merge commit 'b06bc7ab10552079b5300f53ae9458b5b1583402' into travis
nparley bcad188
Add check cache to delete all cache if ci changes
nparley f607a6b
Bug fix
nparley f0cba14
Don't use ccache if not using conda cache
nparley 446489c
Bug fix
nparley b602e9a
Merge commit '883df65b095709e2146b1f36c9697b92eb913731' into travis
nparley 62320f6
BUG: fix to_datetime to handle int16 and int8
ravinimmi 5d0e706
BUG: df.pivot_table: margins_name ignored when aggfunc is a list
cmazzullo b1127f1
Check whole PR for python files change if we are in a PR
nparley bac4045
BLD: update appveyor build to use numpy 1.11 for 3.5 build
jreback 021fdbc
Merge commit '20dd17a70063387ccca5f7aca3dbefc904d99457' into travis
nparley 38cee16
DOC: specify correct kernelspec for converting notebooks (#13491)
abc26c9
Display output of git diff HEAD~2 --numstat | grep -E "ci/" and remov…
nparley e256cf4
Merge branch 'upstream' into travis
nparley 58b5564
Try fetch head in pull requests
nparley 8b72f55
Fetch pull request head so we can look back out the last commits to t…
nparley 6c05a13
Merge branch 'upstream' into travis
nparley bbc7b35
Merge branch 'upstream' into travis
nparley 871f52d
Merge branch 'upstream' into travis
nparley File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
#!/bin/bash | ||
|
||
if [ "$TRAVIS_PULL_REQUEST" == "false" ] | ||
then | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. clever! |
||
echo "Not a PR: checking for changes in ci/ from last 2 commits" | ||
git diff HEAD~2 --numstat | grep -E "ci/" | ||
ci_changes=$(git diff HEAD~2 --numstat | grep -E "ci/"| wc -l) | ||
else | ||
echo "PR: checking for changes in ci/ from last 2 commits" | ||
git fetch origin pull/${TRAVIS_PULL_REQUEST}/head:PR_HEAD | ||
git diff PR_HEAD~2 --numstat | grep -E "ci/" | ||
ci_changes=$(git diff PR_HEAD~2 --numstat | grep -E "ci/"| wc -l) | ||
fi | ||
|
||
MINICONDA_DIR="$HOME/miniconda/" | ||
CACHE_DIR="$HOME/.cache/" | ||
CCACHE_DIR="$HOME/.ccache/" | ||
|
||
if [ $ci_changes -ne 0 ] | ||
then | ||
echo "Files have changed in ci/ deleting all caches" | ||
rm -rf "$MINICONDA_DIR" | ||
rm -rf "$CACHE_DIR" | ||
rm -rf "$CCACHE_DIR" | ||
fi |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so some of the builds are showing up above the
allow_failures
section. We want to keep the first 5 builds (osx, 2 x 2.7, 3.4, 3.5) as the required builds, the rest as allow_failures (mainly so things don't take so long, not because they are actually allowed to fail).So you need to repeat anything you added on in the declaration as below; e.g. for the NUMPY_DEV build, you added:
just repeat this in the below section (so they are identical), then it will be fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed this, it was caused by not indenting the new section correcting in the travis file in the allow failures section.