-
Notifications
You must be signed in to change notification settings - Fork 40
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[bgp] Add new CRD to manage FRRConfiguration
Secondary network interfaces on pods need be announced in a BGP environment. This can be done by creating FRRConfiguration, per default in the metallb namespace. This PR introduce a new CRD which, if an instance got created, the controller watches pods * which have the NAD annotation on it * the NAD has an IPAM configured For each of them a FRRConfiguration gets created. The metallbs k8s service FRRConfiguration of that worker node is taken as the base to create this configuration. Known issue: Since there are then two FRRConfiguration, which hold same configs, like timeouts. It is not possible to update thise. The FRRConfiguration webhook will block those. A possible way to change it, would be stop the infra-operator controller-manager, delete the pod FRRConfigurations, do the change that it gets reflected in the metallb LB FRRConfiguration, then enable the controller that the pod FRRConfiguration get re-created. Jira: OSPRH-8680 Signed-off-by: Martin Schuppert <[email protected]>
- Loading branch information
Showing
19 changed files
with
1,433 additions
and
58 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
155 changes: 155 additions & 0 deletions
155
apis/bases/network.openstack.org_bgpconfigurations.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,155 @@ | ||
| --- | ||
| apiVersion: apiextensions.k8s.io/v1 | ||
| kind: CustomResourceDefinition | ||
| metadata: | ||
| annotations: | ||
| controller-gen.kubebuilder.io/version: v0.11.1 | ||
| creationTimestamp: null | ||
| name: bgpconfigurations.network.openstack.org | ||
| spec: | ||
| group: network.openstack.org | ||
| names: | ||
| kind: BGPConfiguration | ||
| listKind: BGPConfigurationList | ||
| plural: bgpconfigurations | ||
| singular: bgpconfiguration | ||
| scope: Namespaced | ||
| versions: | ||
| - name: v1beta1 | ||
| schema: | ||
| openAPIV3Schema: | ||
| description: BGPConfiguration is the Schema for the bgpconfigurations API | ||
| properties: | ||
| apiVersion: | ||
| description: 'APIVersion defines the versioned schema of this representation | ||
| of an object. Servers should convert recognized schemas to the latest | ||
| internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources' | ||
| type: string | ||
| kind: | ||
| description: 'Kind is a string value representing the REST resource this | ||
| object represents. Servers may infer this from the endpoint the client | ||
| submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds' | ||
| type: string | ||
| metadata: | ||
| type: object | ||
| spec: | ||
| description: BGPConfigurationSpec defines the desired state of BGPConfiguration | ||
| properties: | ||
| frrConfigurationNamespace: | ||
| default: metallb-system | ||
| description: FRRConfigurationNamespace - namespace where to create | ||
| the FRRConfiguration. Defaults to metallb-system. | ||
| type: string | ||
| frrNodeConfigurationSelector: | ||
| description: 'FRRNodeConfigurationSelector - per default the FRRConfiguration | ||
| per node within the FRRConfigurationNamespace gets queried using | ||
| the FRRConfiguration.spec.NodeSelector `kubernetes.io/hostname: | ||
| worker-0`. In case a more specific' | ||
| items: | ||
| description: FRRNodeConfigurationSelectorType - | ||
| properties: | ||
| frrConfigurationNamespace: | ||
| description: NodeName - name of the node object as seen by | ||
| running the `oc get nodes` command | ||
| type: string | ||
| nodeSelector: | ||
| description: NodeSelector to identify the correct FRRConfiguration | ||
| from spec.nodeSelector | ||
| properties: | ||
| matchExpressions: | ||
| description: matchExpressions is a list of label selector | ||
| requirements. The requirements are ANDed. | ||
| items: | ||
| description: A label selector requirement is a selector | ||
| that contains values, a key, and an operator that relates | ||
| the key and values. | ||
| properties: | ||
| key: | ||
| description: key is the label key that the selector | ||
| applies to. | ||
| type: string | ||
| operator: | ||
| description: operator represents a key's relationship | ||
| to a set of values. Valid operators are In, NotIn, | ||
| Exists and DoesNotExist. | ||
| type: string | ||
| values: | ||
| description: values is an array of string values. | ||
| If the operator is In or NotIn, the values array | ||
| must be non-empty. If the operator is Exists or | ||
| DoesNotExist, the values array must be empty. This | ||
| array is replaced during a strategic merge patch. | ||
| items: | ||
| type: string | ||
| type: array | ||
| required: | ||
| - key | ||
| - operator | ||
| type: object | ||
| type: array | ||
| matchLabels: | ||
| additionalProperties: | ||
| type: string | ||
| description: matchLabels is a map of {key,value} pairs. | ||
| A single {key,value} in the matchLabels map is equivalent | ||
| to an element of matchExpressions, whose key field is | ||
| "key", the operator is "In", and the values array contains | ||
| only "value". The requirements are ANDed. | ||
| type: object | ||
| type: object | ||
| x-kubernetes-map-type: atomic | ||
| type: object | ||
| type: array | ||
| type: object | ||
| status: | ||
| description: BGPConfigurationStatus defines the observed state of BGPConfiguration | ||
| properties: | ||
| conditions: | ||
| description: Conditions | ||
| items: | ||
| description: Condition defines an observation of a API resource | ||
| operational state. | ||
| properties: | ||
| lastTransitionTime: | ||
| description: Last time the condition transitioned from one status | ||
| to another. This should be when the underlying condition changed. | ||
| If that is not known, then using the time when the API field | ||
| changed is acceptable. | ||
| format: date-time | ||
| type: string | ||
| message: | ||
| description: A human readable message indicating details about | ||
| the transition. | ||
| type: string | ||
| reason: | ||
| description: The reason for the condition's last transition | ||
| in CamelCase. | ||
| type: string | ||
| severity: | ||
| description: Severity provides a classification of Reason code, | ||
| so the current situation is immediately understandable and | ||
| could act accordingly. It is meant for situations where Status=False | ||
| and it should be indicated if it is just informational, warning | ||
| (next reconciliation might fix it) or an error (e.g. DB create | ||
| issue and no actions to automatically resolve the issue can/should | ||
| be done). For conditions where Status=Unknown or Status=True | ||
| the Severity should be SeverityNone. | ||
| type: string | ||
| status: | ||
| description: Status of the condition, one of True, False, Unknown. | ||
| type: string | ||
| type: | ||
| description: Type of condition in CamelCase. | ||
| type: string | ||
| required: | ||
| - lastTransitionTime | ||
| - status | ||
| - type | ||
| type: object | ||
| type: array | ||
| type: object | ||
| type: object | ||
| served: true | ||
| storage: true | ||
| subresources: | ||
| status: {} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.