Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

Commit 21b3b39

Browse files
authored
Merge pull request #102 from nikhiljindal/fixSSLBug
Fixing a bug in deleting SSL certs
2 parents 86a0da1 + f249f49 commit 21b3b39

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

app/kubemci/pkg/gcp/loadbalancer/loadbalancersyncer.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -224,14 +224,14 @@ func (l *LoadBalancerSyncer) DeleteLoadBalancer(ing *v1beta1.Ingress) error {
224224
// Aggregate errors and return all at the end.
225225
err = multierror.Append(err, frErr)
226226
}
227-
if scErr := l.scs.DeleteSSLCert(); scErr != nil {
228-
// Aggregate errors and return all at the end.
229-
err = multierror.Append(err, scErr)
230-
}
231227
if tpErr := l.tps.DeleteTargetProxies(); tpErr != nil {
232228
// Aggregate errors and return all at the end.
233229
err = multierror.Append(err, tpErr)
234230
}
231+
if scErr := l.scs.DeleteSSLCert(); scErr != nil {
232+
// Aggregate errors and return all at the end.
233+
err = multierror.Append(err, scErr)
234+
}
235235
if umErr := l.ums.DeleteURLMap(); umErr != nil {
236236
// Aggregate errors and return all at the end.
237237
err = multierror.Append(err, umErr)

0 commit comments

Comments
 (0)