From 969350a8065b2c2aaf4c9780e5c007b8193ad689 Mon Sep 17 00:00:00 2001 From: saakhan Date: Fri, 22 Sep 2023 13:49:10 +0530 Subject: [PATCH 1/4] change error messages to debug while retrying partition creation Signed-off-by: saakhan --- .../com/autotune/analyzer/services/CreateExperiment.java | 2 +- .../java/com/autotune/database/dao/ExperimentDAOImpl.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/autotune/analyzer/services/CreateExperiment.java b/src/main/java/com/autotune/analyzer/services/CreateExperiment.java index ed0971dec..7a9d2a7f2 100644 --- a/src/main/java/com/autotune/analyzer/services/CreateExperiment.java +++ b/src/main/java/com/autotune/analyzer/services/CreateExperiment.java @@ -121,7 +121,7 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response) sendErrorResponse(response, null, HttpServletResponse.SC_BAD_REQUEST, addedToDB.getMessage()); } } else { - LOGGER.error("Failed to create experiment: {}", invalidKruizeObject.getValidation_data().getMessage()); + LOGGER.debug("Failed to create experiment: {}", invalidKruizeObject.getValidation_data().getMessage()); sendErrorResponse(response, null, invalidKruizeObject.getValidation_data().getErrorCode(), invalidKruizeObject.getValidation_data().getMessage()); } } diff --git a/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java b/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java index 017b77287..0f24f826c 100644 --- a/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java +++ b/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java @@ -176,13 +176,13 @@ public List addToDBAndFetchFailedResults(List addToDBAndFetchFailedResults(List Date: Tue, 31 Oct 2023 13:36:00 +0530 Subject: [PATCH 2/4] Updated the manifests to use postgres image from quay.io Signed-off-by: Chandrakala Subramanyam --- .../minikube/kruize-crc-minikube.yaml | 2 +- .../openshift/kruize-crc-openshift.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml b/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml index 0c3169d9d..f2f9d148c 100644 --- a/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml +++ b/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml @@ -46,7 +46,7 @@ spec: # runAsUser: 1000 containers: - name: postgres - image: postgres:15.2 + image: quay.io/enterprisedb/postgresql imagePullPolicy: IfNotPresent env: - name: POSTGRES_PASSWORD diff --git a/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml b/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml index 80f1c77bb..7f24152a1 100644 --- a/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml +++ b/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml @@ -124,7 +124,7 @@ spec: serviceAccountName: kruize-sa containers: - name: postgres - image: postgres:15.2 + image: quay.io/enterprisedb/postgresql imagePullPolicy: IfNotPresent env: - name: POSTGRES_PASSWORD From 9b10008fc0afae28f02ce774016cf86563259caf Mon Sep 17 00:00:00 2001 From: Chandrakala Subramanyam Date: Tue, 31 Oct 2023 17:34:53 +0530 Subject: [PATCH 3/4] Updated the tag for the postgres image Signed-off-by: Chandrakala Subramanyam --- .../minikube/kruize-crc-minikube.yaml | 2 +- .../openshift/kruize-crc-openshift.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml b/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml index f2f9d148c..705a3cf94 100644 --- a/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml +++ b/manifests/crc/default-db-included-installation/minikube/kruize-crc-minikube.yaml @@ -46,7 +46,7 @@ spec: # runAsUser: 1000 containers: - name: postgres - image: quay.io/enterprisedb/postgresql + image: quay.io/enterprisedb/postgresql:16-postgis-multilang imagePullPolicy: IfNotPresent env: - name: POSTGRES_PASSWORD diff --git a/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml b/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml index 7f24152a1..279380766 100644 --- a/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml +++ b/manifests/crc/default-db-included-installation/openshift/kruize-crc-openshift.yaml @@ -124,7 +124,7 @@ spec: serviceAccountName: kruize-sa containers: - name: postgres - image: quay.io/enterprisedb/postgresql + image: quay.io/enterprisedb/postgresql:16-postgis-multilang imagePullPolicy: IfNotPresent env: - name: POSTGRES_PASSWORD From 4ffd0fbb4d4ebdc7cf89e67de4cf079bd746fd16 Mon Sep 17 00:00:00 2001 From: saakhan Date: Tue, 31 Oct 2023 18:52:09 +0530 Subject: [PATCH 4/4] rebased and updated code as per review Signed-off-by: saakhan --- src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java b/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java index 1d9a605c7..1ce139751 100644 --- a/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java +++ b/src/main/java/com/autotune/database/dao/ExperimentDAOImpl.java @@ -191,7 +191,7 @@ public List addToDBAndFetchFailedResults(List