-
Notifications
You must be signed in to change notification settings - Fork 60
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
Specifying multiple platforms #65
Comments
It is not possible right now with the conda-mirror CLI. To have this functionality internally we wrap the conda-mirror cli with this: for platform in linux-64 win-64 osx-64;
do
source "/opt/continuum/anaconda/bin/activate" "/opt/envs/conda-mirror" && \
conda-mirror \
--config /opt/tools/conda-mirror-config.yaml \
--upstream-channel https://conda.anaconda.org/conda-forge \
--target-directory /opt/data/conda/www/upstream-conda-forge \
--num-threads 7 \
--platform $platform -vv 2>> /var/log/condaserver/mirror_conda_conda-forge.log
done |
Are PRs welcome for this sort of functionality? |
Short answer, yes. Long answer, I'm at a fork in the road for how to maintain conda-mirror long term. I think it needs some re-architecting. There are a handful of people that I know of that use conda-mirror in different ways. All want conda-mirror to support their use case (#48: rapid validation, #64: incremental initial sync, skipping validation: #62, and this issue). I totally understand the desire here, but this code base has already grown the concurrent validation appendage and other unwieldy appendages are looming on the horizon. The way that we use conda-mirror internally is to use this open source version as the starting point and fold in additional functionality like hipchat integration and sending metrics to graphite, to call out a couple. I think that this is a model that will make conda-mirror generically usable for most people while minimizing the maintenance burden of the open-source version. As such, I've been wanting to find some time for a few months now to implement v2 of the mirror strictly as:
I think this let's our internal implementation remain just that: An internal, mostly gross wrapping of conda-mirror that solves our internal needs while keeping the annoying parts of interacting with anaconda.org out in the open. I think that other people that use conda-mirror should adopt the same philosophy: Use conda-mirror for interacting with anaconda.org and make any local optimizations that are necessary for your specific infrastructure. Now, with that said, it is unclear if/when I will ever have time to reimplement a simplified version of conda-mirror as I just described above 😢. So, in the mean time, you're welcome to submit a PR that adds something like |
Is it possible (or could it be possible) to specify multiple platforms at once?
The text was updated successfully, but these errors were encountered: