[ci] fixed conda #2016
Merged
[ci] fixed conda #2016
Conversation
This file contains 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
StrikerRUS
commented
Feb 16, 2019
| - script: | | ||
| conda update -q -y conda | ||
| conda create -q -y -n %CONDA_ENV% python=%PYTHON_VERSION% matplotlib nose numpy pandas psutil pytest python-graphviz scikit-learn scipy | ||
| cmd /c "activate %CONDA_ENV% & powershell -ExecutionPolicy Bypass -File %BUILD_SOURCESDIRECTORY%/.ci/test_windows.ps1" |
There was a problem hiding this comment.
Unfortunately, despite the fact that conda>=4.6 supports environment managing in PowerShell, I can't get it work neither on my home machine, nor at CI side. So, I included this hack.
I've tested notebook with error, example with error, and, of course, impassable test. All cases expectedly fail a CI job, so, I hope that LastExitCode is handled properly here.
guolinke
approved these changes
Feb 18, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Fixed #2010. Refer to ContinuumIO/anaconda-issues#10629.
Multiple minor additions of
-qand-yoptions forcondaandwgetcommands.The main idea of this PR is about that
CondaEnvironmenttask on Azure Pipelines has been deprecated recently. Now users should managecondaenvironments by their own scripts. As it was discovered from the conversation in ContinuumIO/anaconda-issues#10629, Azure Pipelines used their own "activation" ofcondaenvironments which is not correct and doesn't work with the latestconda's Python patch.Also, we cannot rely on the preinstalled
condaon Linux and macOS because it's installed in the folder which requires to execute everyconda-command withsudo, what is strongly discouraged bycondaitself (refer to our ordeals in #1849). So, this PR addscondainstallation into$AGENT_HOMEDIRECTORY/minicondafolder and overwrites environmental variableCONDA.