fix: Pass region to S3 client only if set (Java) (#4151)

* pass region to s3 client only if set Signed-off-by: tokoko <togurg14@freeuni.edu.ge> * java ci changes Signed-off-by: tokoko <togurg14@freeuni.edu.ge> --------- Signed-off-by: tokoko <togurg14@freeuni.edu.ge>
feast-dev · May 2, 2024 · b8087f7 · b8087f7
1 parent b0c5ffc
commit b8087f7
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 5 deletions.
diff --git a/.github/workflows/java_master_only.yml b/.github/workflows/java_master_only.yml
@@ -124,6 +124,7 @@ jobs:
         with:
           path: ${{ steps.uv-cache.outputs.dir }}
           key: ${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-uv-${{ hashFiles(format('**/py{0}-ci-requirements.txt', env.PYTHON)) }}
+
       - name: Install Python dependencies
         run: make install-python-ci-dependencies-uv
       - uses: actions/cache@v4

diff --git a/java/serving/src/main/java/feast/serving/service/config/RegistryConfigModule.java b/java/serving/src/main/java/feast/serving/service/config/RegistryConfigModule.java
@@ -41,9 +41,14 @@ Storage googleStorage(ApplicationProperties applicationProperties) {
 
   @Provides
   public AmazonS3 awsStorage(ApplicationProperties applicationProperties) {
-    return AmazonS3ClientBuilder.standard()
-        .withRegion(applicationProperties.getFeast().getAwsRegion())
-        .build();
+    AmazonS3ClientBuilder builder = AmazonS3ClientBuilder.standard();
+    String region = applicationProperties.getFeast().getAwsRegion();
+
+    if (region != null) {
+      builder = builder.withRegion(region);
+    }
+
+    return builder.build();
   }
 
   @Provides

diff --git a/java/serving/src/test/resources/docker-compose/feast10/Dockerfile b/java/serving/src/test/resources/docker-compose/feast10/Dockerfile
@@ -1,13 +1,16 @@
 FROM python:3.11
 
 WORKDIR /app
-COPY java/serving/src/test/resources/docker-compose/feast10/ .
 COPY sdk/python /mnt/feast/sdk/python
 COPY protos /mnt/feast/protos
 COPY setup.py /mnt/feast/setup.py
 COPY pyproject.toml /mnt/feast/pyproject.toml
 COPY README.md /mnt/feast/README.md
-RUN cd /mnt/feast && SETUPTOOLS_SCM_PRETEND_VERSION="0.1.0" pip install .[grpcio,redis]
+COPY Makefile /mnt/feast/Makefile
+ENV SETUPTOOLS_SCM_PRETEND_VERSION=0.1.0
+RUN pip install uv
+RUN cd /mnt/feast && uv pip install --system .[grpcio,redis]
+COPY java/serving/src/test/resources/docker-compose/feast10/ .
 EXPOSE 8080
 
 CMD ["./entrypoint.sh"]