Skip to content

Commit 70ead50

Browse files
Check if TLS field in Ingress spec is set (kubernetes#1231) (kubernetes#1234)
Co-authored-by: Mikael Johansson <[email protected]>
1 parent 103f44d commit 70ead50

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/ingress/controller/controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,7 @@ func (c *Controller) deleteIngress(ing *nwv1beta1.Ingress) error {
515515
lbName := utils.GetResourceName(ing.Namespace, ing.Name, c.config.ClusterName)
516516

517517
// Delete Barbican secrets
518-
if c.osClient.Barbican != nil {
518+
if c.osClient.Barbican != nil && ing.Spec.TLS != nil {
519519
nameFilter := fmt.Sprintf("kube_ingress_%s_%s_%s", c.config.ClusterName, ing.Namespace, ing.Name)
520520
if err := openstackutil.DeleteSecrets(c.osClient.Barbican, nameFilter); err != nil {
521521
return fmt.Errorf("failed to remove Barbican secrets: %v", err)

0 commit comments

Comments
 (0)