fairing icon indicating copy to clipboard operation
fairing copied to clipboard

Add requiements.txt installation to append builder

Open rpasricha opened this issue 6 years ago • 5 comments

Currently docker and cluster builders pip install the user's requirements.txt file if provided, but not the append builder.

rpasricha avatar Mar 01 '19 01:03 rpasricha

Here's two solutions:

  1. create a fake virtualenv, run pip install within new virtualenv, and package that up.
  2. ignore requirements.txt and package up existing virtualenv

r2d4 avatar Mar 01 '19 18:03 r2d4

create a fake virtualenv, run pip install within new virtualenv, and package that up.

This might be running under OS/env that might work with the base container right?

ignore requirements.txt and package up existing virtualenv

Same as above. Actually, I remember you bringing catching this issue in one of our meetings.

karthikv2k avatar Mar 01 '19 23:03 karthikv2k

@r2d4 for the solutions, did you imply to package the virtualenv in to the base image?

panbalag avatar May 16 '20 02:05 panbalag

Issue-Label Bot is automatically applying the labels:

Label Probability
feature 0.90

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! Links: app homepage, dashboard and code for this bot.

issue-label-bot[bot] avatar May 16 '20 02:05 issue-label-bot[bot]

@r2d4 is there any plans of fixing the append builder code to install packages from requirements.txt?

panbalag avatar May 16 '20 04:05 panbalag