Allow setting additional listen addresses (-a options). The operator …

…still ensures that Varnish listens on port 6081 since the service expects that port. But all other -a options will be respected and appended. (#76) Signed-off-by: Tomash Sidei <tomash.sidei@ibm.com>
IBM · Aug 17, 2022 · b37c3db · b37c3db
1 parent deb1751
commit b37c3db
Show file tree

Hide file tree

Showing 5 changed files with 35 additions and 5 deletions.
diff --git a/api/v1alpha1/varnishcluster_webhook.go b/api/v1alpha1/varnishcluster_webhook.go
@@ -24,7 +24,6 @@ func SetWebhookLogger(l *logger.Logger) {
 var (
 	varnishArgsKeyRegexp  = regexp.MustCompile(`^-\w$`)
 	disallowedVarnishArgs = map[string]bool{
-		"-a": true,
 		"-f": true,
 		"-F": true,
 		"-n": true,

diff --git a/api/v1alpha1/varnishcluster_webhook_test.go b/api/v1alpha1/varnishcluster_webhook_test.go
@@ -21,6 +21,17 @@ func TestValidatingWebhook(t *testing.T) {
 			},
 			valid: true,
 		},
+		{
+			name: "Additional listen addresses should be allowed",
+			vc: &VarnishCluster{
+				Spec: VarnishClusterSpec{
+					Varnish: &VarnishClusterVarnish{
+						Args: []string{"-a", ":8080,PROXY"},
+					},
+				},
+			},
+			valid: true,
+		},
 		{
 			name: "Invalid values",
 			vc: &VarnishCluster{
@@ -59,7 +70,7 @@ func TestValidatingWebhook(t *testing.T) {
 			vc: &VarnishCluster{
 				Spec: VarnishClusterSpec{
 					Varnish: &VarnishClusterVarnish{
-						Args: []string{"-a", "-f", "-F", "-n", "-S"},
+						Args: []string{"-f", "-F", "-n", "-S"},
 					},
 				},
 			},

diff --git a/api/v1alpha1/zz_generated.deepcopy.go b/api/v1alpha1/zz_generated.deepcopy.go
diff --git a/pkg/varnishcluster/controller/varnishcluster_varnish_args.go b/pkg/varnishcluster/controller/varnishcluster_varnish_args.go
@@ -20,7 +20,6 @@ var (
 func getSanitizedVarnishArgs(spec *vcapi.VarnishClusterSpec) []string {
 	varnishArgsOverrides := [][]string{
 		{"-F"},
-		{"-a", fmt.Sprintf("0.0.0.0:%d", vcapi.VarnishPort)},
 		{"-S", "/etc/varnish-secret/secret"},
 		{"-b", "127.0.0.1:0"}, //start a varnishd without predefined backend. It has to be overridden by settings from ConfigMap
 		{"-T", fmt.Sprintf("0.0.0.0:%d", vcapi.VarnishAdminPort)},
@@ -52,6 +51,7 @@ func getSanitizedVarnishArgs(spec *vcapi.VarnishClusterSpec) []string {
 	}
 
 	varnishArgs := append(parsedArgs, varnishArgsOverrides...)
+	varnishArgs = append(varnishArgs, []string{"-a", fmt.Sprintf("0.0.0.0:%d", vcapi.VarnishPort)})
 
 	// sort the arguments so they won't appear in different order in different reconcile loops and trigger redeployment
 	sort.SliceStable(varnishArgs, func(i, j int) bool {

diff --git a/pkg/varnishcluster/controller/varnishcluster_varnish_args_test.go b/pkg/varnishcluster/controller/varnishcluster_varnish_args_test.go
@@ -75,7 +75,7 @@ func TestGetSanitizedVarnishArgs(t *testing.T) {
 			spec: &v1alpha1.VarnishClusterSpec{
 				VCL: vclConfigMap,
 				Varnish: &v1alpha1.VarnishClusterVarnish{
-					Args: []string{"-S", "/etc/varnish/newsecret", "-T", "127.0.0.1:4235", "-a", "0.0.0.0:3425", "-b", "127.0.0.1:3456"},
+					Args: []string{"-S", "/etc/varnish/newsecret", "-T", "127.0.0.1:4235", "-b", "127.0.0.1:3456"},
 				},
 			},
 			expectedResult: []string{
@@ -122,6 +122,26 @@ func TestGetSanitizedVarnishArgs(t *testing.T) {
 				"-p", "default_ttl=3600",
 			},
 		},
+		{
+			name: "additional -a arguments can be specified",
+			spec: &v1alpha1.VarnishClusterSpec{
+				VCL: vclConfigMap,
+				Varnish: &v1alpha1.VarnishClusterVarnish{
+					Args: []string{"-a", ":8080,PROXY", "-a", ":4392", "-p", "default_grace=3600", "-p", "default_ttl=3600"},
+				},
+			},
+			expectedResult: []string{
+				"-F",
+				"-S", "/etc/varnish-secret/secret",
+				"-T", fmt.Sprintf("0.0.0.0:%d", v1alpha1.VarnishAdminPort),
+				"-a", fmt.Sprintf("0.0.0.0:%d", v1alpha1.VarnishPort),
+				"-a", ":4392",
+				"-a", ":8080,PROXY",
+				"-b", "127.0.0.1:0",
+				"-p", "default_grace=3600",
+				"-p", "default_ttl=3600",
+			},
+		},
 	}
 
 	for _, c := range cases {