Bind webhook controller to a non priviliged port

Running a server on 443 with a non priliged user would fail to start it. Fixes #719
tektoncd · Apr 4, 2019 · d719494 · d719494
1 parent a669012
commit d719494
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cmd/webhook/main.go b/cmd/webhook/main.go
@@ -74,7 +74,7 @@ func main() {
 		ServiceName:    "tekton-pipelines-webhook",
 		DeploymentName: "tekton-pipelines-webhook",
 		Namespace:      system.GetNamespace(),
-		Port:           443,
+		Port:           8443,
 		SecretName:     "webhook-certs",
 		WebhookName:    "webhook.tekton.dev",
 	}

diff --git a/config/400-webhook-service.yaml b/config/400-webhook-service.yaml
@@ -22,6 +22,6 @@ metadata:
 spec:
   ports:
     - port: 443
-      targetPort: 443
+      targetPort: 8443
   selector:
-    app: tekton-pipelines-webhook
+    app: tekton-pipelines-webhook