conda-incubator / conda-store

Data science environments, for collaboration. ✨
https://conda.store
BSD 3-Clause "New" or "Revised" License
144 stars 46 forks source link

[Bug] using diferent conda channels leads to issue with conda-store database #275

Closed viniciusdc closed 1 year ago

viniciusdc commented 2 years ago

When attempting to create an R environment the following error shows up during the build.

Traceback (most recent call last):
  File "/opt/conda-store-server/conda_store_server/build.py", line 178, in build_conda_environment
    set_build_completed(conda_store, build, output.encode("utf-8"), packages)
  File "/opt/conda-store-server/conda_store_server/build.py", line 47, in set_build_completed
    raise ValueError(
ValueError: channel url=https://repo.anaconda.com/pkgs/r not recognized in conda-store channel database

Conda-Store Server version (based on docker image): image: quansight/conda-store-server:v0.3.10

There seems to be a check happening in conda_store_server/build.py that does not account for external channels (?)

The environment yaml:

channels:
  - conda-forge
dependencies:
  - r-base
  - r-irkernel
  - r-ggplot2
  - r-dplyr
  - r-esquisse
  - r-Shiny
  - r-lubridate
  - r-knitr
  - r-mlr
  - r-quanteda
  - r-Caret
  - r-Rmarkdown
  - r-Leaflet
  - r-Janitor
  - r-Corrplot
  - r-Lattice
  - r-GBM
  - r-Prophet
  - r-Randomforest
  - r-C50
  - r-Ggvis
  - r-Plotly
  - r-Rbokeh
  - r-Broom
  - r-StringR
  - r-Magrittr
  - r-Rvest
  - r-Future
  - r-RMySQL
  - r-RSQLite
  - r-Glmnet
  - r-Text2Vec
  - r-SnowballC
  - r-Quantmod
  - r-Rstan
  - r-Swirl
  - r-DBI
  - r-Odbc
  - r-Haven
  - r-Xlconnect
  - r-Xlsx
  - r-Tidyr
  - r-Stringr
  - r-Survival
  - r-Multcomp
  - r-Lme4
  - r-Zoo
  - r-Xts
  - r-Quantmod
  - r-Rcpp
  - r-Devtools
  - r-Testthat
  - r-Roxygen2
  - r-Dalex
  - r-DataExplorer
  - r-Sparklyr
  - r-drake
  - r-tidyverse
name: rkernel
prefix: null
costrouc commented 2 years ago

@viniciusdc right now I have conda-store setup such that there are a list of "approved" conda channels that can be used. I will likely need to make this not so strict.

kcpevey commented 1 year ago

@costrouc can you provide an update on this issue?

trallard commented 1 year ago

Related to #357 - need @costrouc to circle back on these issues

pierrotsmnrd commented 1 year ago

I've tried the given yaml. The env doesn’t build but there’s no more error from conda store, it’s “just” an R env that doesn’t build. I consider this fixed.