Skip to content
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

Jump to bottom

Reconcile RouteAdvertisements in cluster manager #4691

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Reconcile RouteAdvertisements in cluster manager #4691

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
8a2c7a7
Generate RouteAdvertisements CRD
jcaamano Jun 25, 2024
368c337
Add route advertisements CLI flag
jcaamano Jul 3, 2024
65ddfc7
Add RouteAdvertisements to watch factory
jcaamano Jul 3, 2024
b05aad7
Deploy FFR-k8s in kind
jcaamano Jul 3, 2024
3aa2272
Mock NAD informer instead of handling nil in controller
jcaamano Aug 14, 2024
e51dcc2
Add VRFs as reconcilable NetInfo
jcaamano Jul 16, 2024
f4347ed
Network controllers reconcile VRF changes
jcaamano Jul 16, 2024
fdf0b59
Provide NAD controller access to the default network controller
jcaamano Jul 16, 2024
03d1889
Add node/RA informer access from node watch factory
jcaamano Jul 16, 2024
26be58c
Define route advertisements annotation
jcaamano Jul 16, 2024
482559b
Get route advertisement information in NAD controller
jcaamano Jul 16, 2024
c9bb04f
Bump to k8s 1.31
jcaamano Aug 21, 2024
4bc1ff3
Add FRRConfiguration to factory
jcaamano Aug 8, 2024
e6563df
Reconcile RouteAdvertisements from cluster manager
jcaamano Aug 14, 2024
754d12f
Use RA status condition when reconciling networks
jcaamano Aug 29, 2024
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
Prev Previous commit
Next Next commit
Define route advertisements annotation 
This annotation will be set by a future cluster manager controller on
the NADs and will list the names of route advertisements that apply to
the given NAD. This will ease processing time of other zone/node
controllers that need to track which route advertisements apply to a
network avoiding them from processing all route advertisements on each
of their reconciliation loops.

Note that this will happen for the default network as well. For that
probably a dummy NAD on ovn-kubernetes namespace is the best option.

Signed-off-by: Jaime Caamaño Ruiz <jcaamano@redhat.com>
  • Loading branch information
@jcaamano
jcaamano committed Sep 17, 2024
commit 26be58cd6e8e84377df980da1101a780fb8b1f9c
5 changes: 3 additions & 2 deletions go-controller/pkg/types/const.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,9 @@ const (
// OVN-K8S annotation & taint constants
OvnK8sPrefix = "k8s.ovn.org"
// Deprecated: we used to set topology version as an annotation on the node. We don't do this anymore.
OvnK8sTopoAnno = OvnK8sPrefix + "/" + "topology-version"
OvnK8sSmallMTUTaintKey = OvnK8sPrefix + "/" + "mtu-too-small"
OvnK8sTopoAnno = OvnK8sPrefix + "/" + "topology-version"
OvnK8sSmallMTUTaintKey = OvnK8sPrefix + "/" + "mtu-too-small"
OvnRouteAdvertisementsKey = OvnK8sPrefix + "/route-advertisements"

// name of the configmap used to synchronize status (e.g. watch for topology changes)
OvnK8sStatusCMName = "control-plane-status"
Expand Down