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.

Sign up for GitHub

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

Jump to bottom

CircleCI tests with prerelease (nightlies) are failing due to conda timeout #424

Closed
tswast opened this issue Nov 11, 2021 · 1 comment · Fixed by #446
Closed

CircleCI tests with prerelease (nightlies) are failing due to conda timeout #424

tswast opened this issue Nov 11, 2021 · 1 comment · Fixed by #446
Assignees
@tswast
Labels
api: bigquery Issues related to the googleapis/python-bigquery-pandas API. type: process A process-related concern. May include testing, release, or the like.

Comments

@tswast
Copy link
Collaborator

tswast commented Nov 11, 2021 •
edited
Loading

Stuck on "solving environment"

+++ export CONDA_PREFIX=/opt/conda/envs/test-environment
+++ CONDA_PREFIX=/opt/conda/envs/test-environment
+++ export CONDA_SHLVL=1
+++ CONDA_SHLVL=1
+++ export CONDA_DEFAULT_ENV=test-environment
+++ CONDA_DEFAULT_ENV=test-environment
+++ export CONDA_PROMPT_MODIFIER=
+++ CONDA_PROMPT_MODIFIER=
+++ export CONDA_EXE=/opt/conda/bin/conda
+++ CONDA_EXE=/opt/conda/bin/conda
+++ export _CE_M=
+++ _CE_M=
+++ export _CE_CONDA=
+++ _CE_CONDA=
+++ export CONDA_PYTHON_EXE=/opt/conda/bin/python
+++ CONDA_PYTHON_EXE=/opt/conda/bin/python
++ __conda_hashr
++ '[' -n '' ']'
++ '[' -n '' ']'
++ hash -r
+ REQ=ci/requirements-3.9-NIGHTLY
+ conda install -q --file ci/requirements-3.9-NIGHTLY.conda
+ local cmd=install
+ case "$cmd" in
+ __conda_exe install -q --file ci/requirements-3.9-NIGHTLY.conda
+ __add_sys_prefix_to_path
+ '[' -n '' ']'
++ dirname /opt/conda/bin/conda
+ SYSP=/opt/conda/bin
++ dirname /opt/conda/bin
+ SYSP=/opt/conda
+ '[' -n '' ']'
+ PATH=/opt/conda/bin:/opt/conda/envs/test-environment/bin:/opt/conda/condabin:/opt/conda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+ export PATH
+ /opt/conda/bin/conda install -q --file ci/requirements-3.9-NIGHTLY.conda
Collecting package metadata (current_repodata.json): ...working... done
Solving environment: ...working... failed with initial frozen solve. Retrying with flexible solve.
Solving environment: ...working... failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): ...working... done
Solving environment: ...working... /opt/conda/etc/profile.d/conda.sh: line 35:   799 Killed                  "$CONDA_EXE" $_CE_M $_CE_CONDA "$@"
+ return

Exited with code exit status 137
CircleCI received exit code 137

https://app.circleci.com/pipelines/github/googleapis/python-bigquery-pandas/150/workflows/f1bfd01f-dd32-42bc-90d4-4a11c26046ec/jobs/1108

Maybe switch to mamba?
@tswast tswast added the type: process A process-related concern. May include testing, release, or the like. label Nov 11, 2021
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery-pandas API. label Nov 11, 2021
@tswast tswast mentioned this issue Nov 22, 2021
Merged
4 tasks
@tswast
Copy link
Collaborator Author

tswast commented Dec 1, 2021

Alternatively, we could just depend on the pip prerelease session we just added and remove the CircleCI build. We'll still need some conda builds that (1) test with pinned dependencies and (2) test with latest available dependencies

@tswast tswast mentioned this issue Dec 8, 2021
Merged
4 tasks
@tswast tswast self-assigned this Dec 8, 2021
gcf-merge-on-green bot pushed a commit that referenced this issue Dec 9, 2021
@tswast
test: remove redundant NIGHTLY run from CircleCI (#446)
c0f9da0 
Eventually we will want to migrate these to Kokoro (or even GitHub actions using https://github.com/google-github-actions/auth) but just remove the broken session for now. The prerelease run is redundant with the session we recently added, so replace it with a session using the latest released pandas via conda.

The reason we're not removing the conda sessions completely is that pandas recommends installation via conda/mamba. Since pandas-gbq is an optional dependency of pandas, we need to ensure this package is also always installable via conda/mamba.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-bigquery-pandas/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes #424 🦕
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tswast tswast

Labels
api: bigquery Issues related to the googleapis/python-bigquery-pandas API. type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: remove redundant NIGHTLY run from CircleCI tswast/python-bigquery-pandas
1 participant
@tswast