Skip to content

CI/CLN: update travis #36514

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

Merged
merged 23 commits into from
Sep 24, 2020
Merged

CI/CLN: update travis #36514

merged 23 commits into from
Sep 24, 2020

Conversation

fangchenli
Copy link
Member

@fangchenli fangchenli commented Sep 21, 2020

update and cleanup Travis

closes #36601

.travis.yml Outdated
@@ -42,7 +44,9 @@ matrix:

- arch: arm64
env:
- JOB="3.7, arm64" PYTEST_WORKERS=8 ENV_FILE="ci/deps/travis-37-arm64.yaml" PATTERN="(not slow and not network and not clipboard and not arm_slow)"
- JOB="3.7, arm64" ENV_FILE="ci/deps/travis-37-arm64.yaml" PATTERN="(not slow and not network and not clipboard and not arm_slow)"
Copy link
Contributor

Choose a reason for hiding this comment

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

if this works great
but the worker config for arm is odd on travis

Copy link
Member Author

Choose a reason for hiding this comment

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

Good to know. I'll play with it for a bit.

Copy link
Member Author

Choose a reason for hiding this comment

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

Using auto would freeze the CI. I also tested this on a Raspberry Pi. Only the test_fast.sh script works on the Pi. Even though it's a 32bit arm system. But this could be universal for arm. Surprisingly, it happened on master today https://travis-ci.org/github/pandas-dev/pandas/jobs/729807957.

Comment on lines -74 to -79
# Because travis runs on Google Cloud and has a /etc/boto.cfg,
# it breaks moto import, see:
# https://github.com/spulec/moto/issues/1771
# https://github.com/boto/boto/issues/3741
# This overrides travis and tells it to look nowhere.
- export BOTO_CONFIG=/dev/null
Copy link
Member Author

Choose a reason for hiding this comment

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

ccache: true
directories:
- $HOME/.cache # cython cache
- $HOME/.ccache # compiler cache
Copy link
Member Author

Choose a reason for hiding this comment

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

It's redundant.

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

this looks fine

try rebasing and we'll see if travis fixes itself

@jreback jreback added the CI Continuous Integration label Sep 24, 2020
@jreback jreback added this to the 1.2 milestone Sep 24, 2020
@jbrockmendel
Copy link
Member

try rebasing and we'll see if travis fixes itself

This particular Travis failure has been affecting everything all day.

@jreback
Copy link
Contributor

jreback commented Sep 24, 2020

any idea where this is coming from?

@fangchenli
Copy link
Member Author

fangchenli commented Sep 24, 2020

any idea where this is coming from?

I set the number of workers to 1, and it works now. It's almost identical to what I saw on Raspberry Pi. I think the root cause is pytest-xdist. It probably has a problem specific to ARM architecture.

@jreback
Copy link
Contributor

jreback commented Sep 24, 2020

any idea where this is coming from?

I set the number of workers to 1, and it works now. It's almost identical to what I saw on Raspberry Pi. I think the root cause is pytest-xdist. It probably has a problem specific to ARM architecture.

oh wow, ok! ping on green.

@jreback jreback merged commit 27aae22 into pandas-dev:master Sep 24, 2020
@jreback
Copy link
Contributor

jreback commented Sep 24, 2020

thanks @fangchenli

@fangchenli fangchenli deleted the clean-ci branch October 3, 2020 05:41
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: move arm64 build to allow_failures for now
3 participants