Skip to content

Commit

Permalink
Merge pull request #401 from rsvoboda/fix.k8s.name.checks
Browse files Browse the repository at this point in the history
Kubernetes resource name no longer depends on container image name
  • Loading branch information
mjurc authored Jun 21, 2023
2 parents 2ab9278 + f3fe64b commit a52bf1f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
quarkus.container-image.group=qe
quarkus.container-image.name=demo
quarkus.container-image.tag=1.0.0
quarkus.application.name=non-application endpoints
quarkus.http.root-path=/api
quarkus.http.non-application-root-path=/q
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,10 @@ public void testOpenShiftYamlHealthProbesWithCustomImageName() throws FileNotFou

// finalMap.entrySet().forEach(System.out::println);

assertEquals("demo", finalMap.get("name"), "Deployment name is not expected 'demo'");
assertEquals("non-application endpoints", finalMap.get("name"),
"Deployment name is different than expected 'non-application endpoints'");
assertEquals("qe/demo:1.0.0", finalMap.get("image"),
"Deployment image is different than expected 'qe/demo:1.0.0'");
assertTrue(finalMap.containsKey("livenessProbe"), "livenessProbe is not defined");
assertTrue(finalMap.containsKey("readinessProbe"), "readinessProbe is not defined");
}
Expand Down

0 comments on commit a52bf1f

Please sign in to comment.