Fix sidecars resource requirements properties in Kubernetes

Before these changes, the resource requirements limits/requests were ignored for the sidecars containers. Note that these changes include a workaround for dekorateio/dekorate#1234. Fix #35006
quarkusio · Jul 26, 2023 · 6294fa7 · 6294fa7
1 parent 207fc2c
commit 6294fa7
Show file tree

Hide file tree

Showing 6 changed files with 23 additions and 56 deletions.
diff --git a/...anilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AddSidecarDecorator.java b/...anilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AddSidecarDecorator.java
@@ -1,5 +1,6 @@
 package io.quarkus.kubernetes.deployment;
 
+import io.dekorate.kubernetes.adapter.ContainerAdapter;
 import io.dekorate.kubernetes.config.Container;
 import io.dekorate.kubernetes.decorator.Decorator;
 import io.dekorate.kubernetes.decorator.NamedResourceDecorator;
@@ -8,7 +9,9 @@
 import io.fabric8.kubernetes.api.model.PodSpecBuilder;
 
 /**
- * Copied from dekorate in order to fix some issues
+ * Copied from dekorate in order to fix some issues.
+ * TODO: This decorator should be removed and replaced by the Dekorate AddSidecarDecorator class after
+ * https://github.com/dekorateio/dekorate/pull/1234 is merged and Dekorate is bumped.
  */
 class AddSidecarDecorator extends NamedResourceDecorator<PodSpecBuilder> {
 
@@ -26,7 +29,10 @@ public AddSidecarDecorator(String deployment, Container container) {
     @Override
     public void andThenVisit(PodSpecBuilder podSpec, ObjectMeta resourceMeta) {
         // this was changed to use our patched adapter
-        podSpec.addToContainers(ContainerAdapter.adapt(container));
+        var sidecarContainer = ContainerAdapter.adapt(container);
+        // This is necessary because of the issue that this pull request fixes https://github.com/dekorateio/dekorate/pull/1234
+        sidecarContainer.setWorkingDir(container.getWorkingDir());
+        podSpec.addToContainers(sidecarContainer);
     }
 
     public Class<? extends Decorator>[] after() {

diff --git a/...s/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ContainerAdapter.java b/...s/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ContainerAdapter.java
diff --git a/...est/java/io/quarkus/resteasy/reactive/jackson/deployment/test/CustomObjectMapperTest.java b/...est/java/io/quarkus/resteasy/reactive/jackson/deployment/test/CustomObjectMapperTest.java
@@ -5,6 +5,7 @@
 
 import java.util.Objects;
 
+import jakarta.enterprise.inject.Produces;
 import jakarta.ws.rs.Consumes;
 import jakarta.ws.rs.POST;
 import jakarta.ws.rs.Path;
@@ -61,6 +62,7 @@ public Request(String value) {
             this.value = value;
         }
 
+        @Produces
         public String getValue() {
             return value;
         }

diff --git a/...uarkus/rest/client/reactive/jackson/test/DifferentObjectMapperForClientAndServerTest.java b/...uarkus/rest/client/reactive/jackson/test/DifferentObjectMapperForClientAndServerTest.java
@@ -32,6 +32,7 @@
 import io.quarkus.rest.client.reactive.QuarkusRestClientBuilder;
 import io.quarkus.test.QuarkusUnitTest;
 import io.quarkus.test.common.http.TestHTTPResource;
+import jakarta.ws.rs.ext.Provider;
 
 public class DifferentObjectMapperForClientAndServerTest {
     @RegisterExtension
@@ -147,6 +148,7 @@ public int hashCode() {
         }
     }
 
+    @Provider
     public static class ClientObjectMapperUnwrappingRootElement implements ContextResolver<ObjectMapper> {
         static final AtomicBoolean USED = new AtomicBoolean(false);
 

diff --git a/...-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithSidecarAndJibTest.java b/...-standard-way/src/test/java/io/quarkus/it/kubernetes/KubernetesWithSidecarAndJibTest.java
@@ -10,6 +10,7 @@
 import org.junit.jupiter.api.extension.RegisterExtension;
 
 import io.fabric8.kubernetes.api.model.HasMetadata;
+import io.fabric8.kubernetes.api.model.Quantity;
 import io.fabric8.kubernetes.api.model.apps.Deployment;
 import io.quarkus.builder.Version;
 import io.quarkus.maven.dependency.Dependency;
@@ -90,6 +91,12 @@ private void assertSidecar(io.fabric8.kubernetes.api.model.PodSpec podSpec) {
                         assertThat(e.getName()).isEqualTo("FOO");
                         assertThat(e.getValue()).isEqualTo("bar");
                     });
+                    assertThat(c.getResources()).satisfies(r -> {
+                        assertThat(r.getRequests().get("cpu")).isEqualTo(new Quantity("102m"));
+                        assertThat(r.getRequests().get("memory")).isEqualTo(new Quantity("201Mi"));
+                        assertThat(r.getLimits().get("cpu")).isEqualTo(new Quantity("100m"));
+                        assertThat(r.getLimits().get("memory")).isEqualTo(new Quantity("200Mi"));
+                    });
                 });
     }
 }
diff --git a/...rnetes/quarkus-standard-way/src/test/resources/kubernetes-with-sidecar-and-jib.properties b/...rnetes/quarkus-standard-way/src/test/resources/kubernetes-with-sidecar-and-jib.properties
@@ -9,3 +9,7 @@ quarkus.kubernetes.sidecars.sc.arguments=-l
 quarkus.kubernetes.sidecars.sc.mounts.app-config.path=/deployments/config
 quarkus.kubernetes.sidecars.sc.ports.http.container-port=3000
 quarkus.kubernetes.sidecars.sc.env-vars.foo.value=bar
+quarkus.kubernetes.sidecars.sc.resources.requests.memory=201Mi
+quarkus.kubernetes.sidecars.sc.resources.limits.memory=200Mi
+quarkus.kubernetes.sidecars.sc.resources.requests.cpu=102m
+quarkus.kubernetes.sidecars.sc.resources.limits.cpu=100m