quick fix: remove protected image repo spec test

After cnti-testcatalog#2114 was merged, an issue was revealed with image size - protected repo spec test. It was discovered only after merging. Until this is fixed, disabling this variant of the image size spec test. Refs: cnti-testcatalog#2114 cnti-testcatalog#2136 Signed-off-by: Martin Matyas <[email protected]>
martin-mat · Aug 16, 2024 · d66109f · d66109f
1 parent 3006e37
commit d66109f
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/spec/workload/microservice_spec.cr b/spec/workload/microservice_spec.cr
@@ -149,13 +149,14 @@ describe "Microservice" do
 
   it "'reasonable_image_size' should pass if image is smaller than 5gb, when using a protected image", tags: ["reasonable_image_size"]  do
     # If github secrets are loaded correctly - use protected coredns
-    if ENV["PROTECTED_DOCKERHUB_USERNAME"]? && ENV["PROTECTED_DOCKERHUB_PASSWORD"]? && ENV["PROTECTED_DOCKERHUB_EMAIL"]? && ENV["PROTECTED_IMAGE_REPO"]? &&
-       !ENV["PROTECTED_DOCKERHUB_USERNAME"].to_s.empty? && !ENV["PROTECTED_DOCKERHUB_PASSWORD"].to_s.empty? &&
-       !ENV["PROTECTED_DOCKERHUB_EMAIL"].to_s.empty? && !ENV["PROTECTED_IMAGE_REPO"].to_s.empty?
-     cnf = "./sample-cnfs/sample_coredns_protected"
-   else
-     cnf = "./sample-cnfs/sample-coredns-cnf"
-   end
+    # TODO: fix protected image repo
+    #if ENV["PROTECTED_DOCKERHUB_USERNAME"]? && ENV["PROTECTED_DOCKERHUB_PASSWORD"]? && ENV["PROTECTED_DOCKERHUB_EMAIL"]? && ENV["PROTECTED_IMAGE_REPO"]? &&
+    #   !ENV["PROTECTED_DOCKERHUB_USERNAME"].to_s.empty? && !ENV["PROTECTED_DOCKERHUB_PASSWORD"].to_s.empty? &&
+    #   !ENV["PROTECTED_DOCKERHUB_EMAIL"].to_s.empty? && !ENV["PROTECTED_IMAGE_REPO"].to_s.empty?
+    # cnf = "./sample-cnfs/sample_coredns_protected"
+    #else
+    cnf = "./sample-cnfs/sample-coredns-cnf"
+    #end
     ShellCmd.cnf_setup("cnf-path=#{cnf}")
     result = ShellCmd.run_testsuite("reasonable_image_size verbose")
     result[:status].success?.should be_true