forked from knative/serving
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add patch to make e2e tls tests work on OCP
- Loading branch information
Showing
1 changed file
with
15 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
diff --git a/test/e2e/clusterlocaldomaintls/cluster_local_domain_tls_test.go b/test/e2e/clusterlocaldomaintls/cluster_local_domain_tls_test.go | ||
--- a/test/e2e/clusterlocaldomaintls/cluster_local_domain_tls_test.go (revision a35c0c4839e309e10f23c6d6c0fd72b44c491c7f) | ||
+++ b/test/e2e/clusterlocaldomaintls/cluster_local_domain_tls_test.go (revision eebdb4a9e34f66dbaece8f636578989d5748ae0f) | ||
@@ -129,8 +129,9 @@ | ||
t.Fatalf("Internal URL scheme of service %v was not https", names.Service) | ||
} | ||
|
||
- if externalURL.Scheme != "http" { | ||
- t.Fatalf("External URL scheme of service %v was not http", names.Service) | ||
+ // On OpenShift this is always https | ||
+ if externalURL.Scheme != "https" { | ||
+ t.Fatalf("External URL scheme of service %v was not https", names.Service) | ||
} | ||
|
||
// Check normal access on external domain |