fix: Build dockerfile correctly (feast-dev#3239)

* Build dockerfile correctly Signed-off-by: Felix Wang <[email protected]> * Update Signed-off-by: Felix Wang <[email protected]> * Fix Signed-off-by: Felix Wang <[email protected]> Signed-off-by: Felix Wang <[email protected]>
franciscojavierarceo · Sep 22, 2022 · a2dc0d0 · a2dc0d0
1 parent 2624879
commit a2dc0d0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/sdk/python/feast/infra/feature_servers/multicloud/Dockerfile b/sdk/python/feast/infra/feature_servers/multicloud/Dockerfile
@@ -9,8 +9,8 @@ RUN apt update && \
 
 RUN pip install pip --upgrade
 COPY . .
+RUN pip install ".[aws,gcp,snowflake,redis,go,mysql]"
 
-RUN pip install -r requirements.txt
 RUN apt update
 RUN apt install -y -V ca-certificates lsb-release wget
 RUN wget https://apache.jfrog.io/artifactory/arrow/$(lsb_release --id --short | tr 'A-Z' 'a-z')/apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb

diff --git a/sdk/python/feast/infra/feature_servers/multicloud/Dockerfile.dev b/sdk/python/feast/infra/feature_servers/multicloud/Dockerfile.dev
@@ -9,8 +9,8 @@ RUN apt update && \
 
 RUN pip install pip --upgrade
 COPY . .
+RUN pip install ".[aws,gcp,snowflake,redis,go,mysql]"
 
-RUN pip install -r requirements.txt
 RUN apt update
 RUN apt install -y -V ca-certificates lsb-release wget
 RUN wget https://apache.jfrog.io/artifactory/arrow/$(lsb_release --id --short | tr 'A-Z' 'a-z')/apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb

diff --git a/sdk/python/feast/infra/feature_servers/multicloud/requirements.txt b/sdk/python/feast/infra/feature_servers/multicloud/requirements.txt