Skip to content

Commit

Permalink
Fix ovn_controller tls args
Browse files Browse the repository at this point in the history
TLS support in ovn was originally added with [1], but tls args
got messed in [2], the issue got visible after
tls enabled by default[3], this patch fixes it.

[1] openstack-k8s-operators#234
[2] openstack-k8s-operators#233
[3] openstack-k8s-operators/openstack-operator#727

Related-Issue: OSPRH-2191
Depends-On: openstack-k8s-operators/openstack-operator#745
  • Loading branch information
karelyatin committed Apr 10, 2024
1 parent 4d443b8 commit 2ca4015
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/ovncontroller/daemonset.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ package ovncontroller

import (
"fmt"
"strings"

"github.com/openstack-k8s-operators/lib-common/modules/common/env"
"github.com/openstack-k8s-operators/lib-common/modules/common/tls"
Expand Down Expand Up @@ -72,7 +73,7 @@ func CreateOVNDaemonSet(
{
Name: "ovn-controller",
Command: []string{"/bin/bash", "-c"},
Args: args,
Args: []string{strings.Join(args, " ")},
Lifecycle: &corev1.Lifecycle{
PreStop: &corev1.LifecycleHandler{
Exec: &corev1.ExecAction{
Expand Down

0 comments on commit 2ca4015

Please sign in to comment.