Skip to content

Bug 2062842: [4.8z] After reboot egress node, lr-policy-list was not correct, some duplicate records or missed internal IPs #1009

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
openshift-merge-robot merged 10 commits into from
Apr 8, 2022
Merged

Bug 2062842: [4.8z] After reboot egress node, lr-policy-list was not correct, some duplicate records or missed internal IPs #1009

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
5e27257
Change assignment set sequence for egress node ADD
alexanderConstantinescu Jun 16, 2021
94090d4
Avoid erroring if master cannot find all egress labels
alexanderConstantinescu Jun 16, 2021
c154e22
Egress IP with ECMP routing
alexanderConstantinescu Dec 8, 2020
dc423eb
Update e2e to verify egress IP with ECMP
alexanderConstantinescu Jun 25, 2021
f727978
Clean legacy logical router policies for egress IP on upgrade
alexanderConstantinescu Mar 22, 2021
57bb0de
Enable info logging for successful assignment of egress IP
flavio-fernandes Mar 1, 2022
4a73da1
Fix cache building used for removing stale egress IPs
flavio-fernandes Mar 1, 2022
9732a40
Egress IP: Remove legacy logical router policies in bulk
flavio-fernandes Mar 17, 2022
537487f
EgressIp sync needs to account for stale nexthops
flavio-fernandes Mar 12, 2022
fba695a
CARRY: fix idempotently updating egress IP route policies
flavio-fernandes Mar 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion go-controller/pkg/kube/kube.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ func (k *Kube) UpdateEgressFirewall(egressfirewall *egressfirewall.EgressFirewal

// UpdateEgressIP updates the EgressIP with the provided EgressIP data
func (k *Kube) UpdateEgressIP(eIP *egressipv1.EgressIP) error {
klog.Infof("Updating status on EgressIP %s", eIP.Name)
klog.Infof("Updating status on EgressIP %s status %v", eIP.Name, eIP.Status)
_, err := k.EIPClient.K8sV1().EgressIPs().Update(context.TODO(), eIP, metav1.UpdateOptions{})
return err
}
Expand Down
Loading