Skip to content

GH-34296: [C++][CI] Force appveyor builds to use conda-forge and ignore defaults channel#34297

Merged
assignUser merged 1 commit intoapache:mainfrom
westonpace:bugfix/GH-34296--fix-broken-appveyor
Feb 23, 2023
Merged

GH-34296: [C++][CI] Force appveyor builds to use conda-forge and ignore defaults channel#34297
assignUser merged 1 commit intoapache:mainfrom
westonpace:bugfix/GH-34296--fix-broken-appveyor

Conversation

@westonpace
Copy link
Copy Markdown
Member

@westonpace westonpace commented Feb 22, 2023 •
edited by github-actions bot
Loading

For some reason (potentially related to mamba-org/mamba#656 or maybe just the default conda behavior) it appears that specifying -c conda-forge is not sufficient as conda will still check the defaults channel.

Lately it seems the defaults channel has introduced some incompatibility that leads to a deadlock or hang during solve.

This PR disables defaults completely which seems to avoid the hang.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 22, 2023

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 22, 2023

⚠️ GitHub issue #34296 has been automatically assigned in GitHub to PR creator.

assignUser
assignUser approved these changes Feb 23, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@assignUser assignUser left a comment

Choose a reason for hiding this comment

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

Interesting, I do remember a user issue where solving took forever with -c conda-forge.
Should probably be mentioned in the docs?

@assignUser
Copy link
Copy Markdown
Member

assignUser commented Feb 23, 2023

#32512

@assignUser assignUser merged commit e0e740b into apache:main Feb 23, 2023
@assignUser assignUser mentioned this pull request Feb 23, 2023
Merged
@ursabot
Copy link
Copy Markdown

ursabot commented Feb 23, 2023

Benchmark runs are scheduled for baseline = 7f58c1a and contender = e0e740b. e0e740b is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.34% ⬆️0.0%] test-mac-arm
[Finished ⬇️3.32% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.1% ⬆️0.0%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] e0e740bd ec2-t3-xlarge-us-east-2
[Finished] e0e740bd test-mac-arm
[Finished] e0e740bd ursa-i9-9960x
[Finished] e0e740bd ursa-thinkcentre-m75q
[Finished] 7f58c1a8 ec2-t3-xlarge-us-east-2
[Finished] 7f58c1a8 test-mac-arm
[Finished] 7f58c1a8 ursa-i9-9960x
[Finished] 7f58c1a8 ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

thisisnic pushed a commit that referenced this pull request Feb 27, 2023
@assignUser
GH-32512: [Docs][R] Update conda install command (#34298)
52f4fe1 
As shown in #34297 using the cli flag is not enough to avoid influence of the default channel causing extremely long solve times or conflicts.
* Closes: #32512

Authored-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@assignUser assignUser assignUser approved these changes

@kou kou Awaiting requested review from kou kou is a code owner

@raulcd raulcd Awaiting requested review from raulcd raulcd is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[C++] Appveyor hangs on conda install

3 participants

@westonpace @assignUser @ursabot