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

felixwang9817 · web-flow · commit a2dc0d0410eb · 2022-09-21T20:35:42.000-07:00
* Build dockerfile correctly

Signed-off-by: Felix Wang &lt;wangfelix98@gmail.com&gt;

* Update

Signed-off-by: Felix Wang &lt;wangfelix98@gmail.com&gt;

* Fix

Signed-off-by: Felix Wang &lt;wangfelix98@gmail.com&gt;

Signed-off-by: Felix Wang &lt;wangfelix98@gmail.com&gt;
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
