Project

General

Profile

Actions

Idea #8002

closed

[Deployment] python-pyvcf backport is unnecessary

Added by Brett Smith over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Deployment
Target version:
Story points:
0.5

Description

I suggest we stop building it and revoke the previously-published packages. Nothing declares it as a dependency:

(arv)locke % git grep -i pyvcf .
docker/compute/Dockerfile:RUN apt-get install -qy supervisor python-pip python-pyvcf python-gflags python-google-api-python-client python-virtualenv libattr1-dev libfuse-dev python-dev python-llfuse crunchstat python-arvados-fuse cron dnsmasq
docker/shell/Dockerfile:    python-pip python-pyvcf python-gflags python-google-api-python-client \

(We'll need to remove these installs first.) And no Python code imports the module:

(arv)locke % find -name setup.py -print0 | xargs -0 dirname | xargs git grep -l vcf crunch_scripts
crunch_scripts/GATK2-VariantFiltration
crunch_scripts/GATK2-bqsr
crunch_scripts/GATK2-merge-call
crunch_scripts/GATK2-realign
sdk/python/tests/data/1000G_ref_manifest
sdk/python/tests/data/jlake_manifest

The results are too much to paste here but the Crunch scripts all have the string 'vcf' in filenames, not from any module import. The Dockerfile suggests we were previously installing this on compute+shell nodes as a convenience for bioinformatics work, but the way we use Docker makes that moot now.


Subtasks 1 (0 open1 closed)

Task #8759: Review 8002-no-pyvcf-wipResolvedBrett Smith03/18/2016Actions
Actions

Also available in: Atom PDF