Skip to content

Commit

Permalink
Merge pull request #847 from jcmoraisjr/jm-fix-ing-update
Browse files Browse the repository at this point in the history
Fix ingress update to an existing backend
  • Loading branch information
jcmoraisjr authored Aug 30, 2021
2 parents 1cfd69b + 3a70ea3 commit 58eac35
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/converters/ingress/ingress.go
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,7 @@ func (c *converter) syncPartial() {
//
// All state change works removing hosts and backs objects in an old state and
// resyncing ingress objects to recreate hosts and backs in a new state. This
// works very well, except with new ingress objects that references hosts or
// works very well, except with ingress objects that starts to reference hosts or
// backs that already exist - all the tracking starts from the ingress parsing.
//
// trackAddedIngress does the same tracking the sync ingress already do, but
Expand All @@ -377,7 +377,7 @@ func (c *converter) syncPartial() {
// here and removed before parse the added ingress which will readd such hosts
// and backs
func (c *converter) trackAddedIngress() {
for _, ing := range c.changed.IngressesAdd {
for _, ing := range append(c.changed.IngressesAdd, c.changed.IngressesUpd...) {
name := ing.Namespace + "/" + ing.Name
if ing.Spec.DefaultBackend != nil {
backend := c.findBackend(ing.Namespace, ing.Spec.DefaultBackend)
Expand Down

0 comments on commit 58eac35

Please sign in to comment.