next

Signed-off-by: Andreas Karis <[email protected]>

Author: andreaskaris

apis/v1alpha1/bpf_application_state_types.go

@@ -99,13 +99,14 @@ type BpfApplicationStateStatus struct {
 	// as a lightweight sequence number to verify that the API server is serving
 	// the most recent version of the object before beginning a new Reconcile
 	// operation.
-	// +kubebuilder:validation:Minimum=0
+	// +kubebuilder:validation:Minimum=1
 	// +optional
-	UpdateCount int64 `json:"updateCount"`
+	UpdateCount int64 `json:"updateCount,omitempty"`
 	// node is the name of the Kubernetes node for this BpfApplicationState.
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=253
 	// +optional
-	Node string `json:"node"`
+	Node string `json:"node,omitempty"`
 	// appLoadStatus reflects the status of loading the eBPF application on the
 	// given node.
 	//
@@ -128,7 +129,7 @@ type BpfApplicationStateStatus struct {
 	// UnloadError is returned if one or more programs encountered an error when
 	// being unloaded.
 	// +optional
-	AppLoadStatus AppLoadStatus `json:"appLoadStatus"`
+	AppLoadStatus AppLoadStatus `json:"appLoadStatus,omitempty"`
 	// programs is a list of eBPF programs contained in the parent BpfApplication
 	// instance. Each entry in the list contains the derived program attributes as
 	// well as the attach status for each program on the given Kubernetes node.

apis/v1alpha1/bpf_application_types.go

@@ -36,7 +36,7 @@ type BpfApplicationProgram struct {
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
-	Name string `json:"name"`
+	Name string `json:"name,omitempty"`
 
 	// type is a required field used to specify the type of the eBPF program.
 	//
@@ -73,7 +73,7 @@ type BpfApplicationProgram struct {
 	// +unionDiscriminator
 	// +required
 	// +kubebuilder:validation:Enum:="XDP";"TC";"TCX";"UProbe";"URetProbe"
-	Type EBPFProgType `json:"type"`
+	Type EBPFProgType `json:"type,omitempty"`
 
 	// xdp is an optional field, but required when the type field is set to XDP.
 	// xdp defines the desired state of the application's XDP programs. XDP program
@@ -163,7 +163,7 @@ type BpfApplicationSpec struct {
 	// programs in the list to be reloaded, which could be temporarily service
 	// effecting. For this reason, modifying the list is currently not allowed.
 	// +required
-	// +kubebuilder:validation:MinItems:=1
+	// +kubebuilder:validation:MinItems=1
 	// +kubebuilder:validation:MaxItems=1023
 	// +listType=atomic
 	Programs []BpfApplicationProgram `json:"programs,omitempty"`
@@ -194,7 +194,7 @@ type BpfApplication struct {
 
 	// spec defines the desired state of the BpfApplication.
 	// +required
-	Spec BpfApplicationSpec `json:"spec,omitempty"`
+	Spec BpfApplicationSpec `json:"spec,omitzero"`
 	// status reflects the observed state of the BpfApplication.
 	// +optional
 	Status BpfAppStatus `json:"status,omitempty"`

apis/v1alpha1/cluster_bpf_application_state_types.go

@@ -71,7 +71,7 @@ type ClBpfApplicationProgramState struct {
 	// +unionDiscriminator
 	// +required
 	// +kubebuilder:validation:Enum:="FEntry";"FExit";"KProbe";"KRetProbe";"TC";"TCX";"TracePoint";"UProbe";"URetProbe";"XDP"
-	Type EBPFProgType `json:"type"`
+	Type EBPFProgType `json:"type,omitempty"`
 
 	// xdp contains the attachment data for an XDP program when type is set to XDP.
 	// +unionMember
@@ -150,12 +150,14 @@ type ClBpfApplicationStateStatus struct {
 	// as a lightweight sequence number to verify that the API server is serving
 	// the most recent version of the object before beginning a new Reconcile
 	// operation.
+	// +kubebuilder:validation:Minimum=1
 	// +optional
-	UpdateCount int64 `json:"updateCount"`
+	UpdateCount int64 `json:"updateCount,omitempty"`
 	// node is the name of the Kubernetes node for this ClusterBpfApplicationState.
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=253
 	// +optional
-	Node string `json:"node"`
+	Node string `json:"node,omitempty"`
 	// appLoadStatus reflects the status of loading the eBPF application on the
 	// given node.
 	//
@@ -176,7 +178,7 @@ type ClBpfApplicationStateStatus struct {
 	// UnloadError is returned if one or more programs encountered an error when
 	// being unloaded.
 	// +optional
-	AppLoadStatus AppLoadStatus `json:"appLoadStatus"`
+	AppLoadStatus AppLoadStatus `json:"appLoadStatus,omitempty"`
 	// programs is a list of eBPF programs contained in the parent
 	// ClusterBpfApplication instance. Each entry in the list contains the derived
 	// program attributes as well as the attach status for each program on the

apis/v1alpha1/cluster_bpf_application_types.go

@@ -82,7 +82,7 @@ type ClBpfApplicationProgram struct {
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
-	Name string `json:"name"`
+	Name string `json:"name,omitempty"`
 
 	// type is a required field used to specify the type of the eBPF program.
 	//
@@ -143,7 +143,7 @@ type ClBpfApplicationProgram struct {
 	// +unionDiscriminator
 	// +required
 	// +kubebuilder:validation:Enum:="XDP";"TC";"TCX";"FEntry";"FExit";"KProbe";"KRetProbe";"UProbe";"URetProbe";"TracePoint"
-	Type EBPFProgType `json:"type"`
+	Type EBPFProgType `json:"type,omitempty"`
 
 	// xdp is an optional field, but required when the type field is set to XDP.
 	// xdp defines the desired state of the application's XDP programs. XDP program
@@ -325,7 +325,7 @@ type ClusterBpfApplication struct {
 
 	// spec defines the desired state of the BpfApplication.
 	// +required
-	Spec ClBpfApplicationSpec `json:"spec,omitempty"`
+	Spec ClBpfApplicationSpec `json:"spec,omitzero"`
 	// status reflects the observed state of the BpfApplication.
 	// +optional
 	Status BpfAppStatus `json:"status,omitempty"`

apis/v1alpha1/cluster_fentry_program_types.go

@@ -40,7 +40,7 @@ type ClFentryLoadInfo struct {
 	// function to attach the FEntry program. function must not be an empty string,
 	// must not exceed 64 characters in length, must start with alpha characters
 	// and must only contain alphanumeric characters.
-	// +required
+	// +optional
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
@@ -58,7 +58,7 @@ type ClFentryAttachInfo struct {
 	// attempt to be attached. To detach the FEntry program, remove the link entry.
 	// +required
 	// +kubebuilder:validation:Enum=Attach;
-	Mode AttachTypeAttach `json:"mode"`
+	Mode AttachTypeAttach `json:"mode,omitempty"`
 }
 
 type ClFentryProgramInfoState struct {

apis/v1alpha1/cluster_fexit_program_types.go

@@ -41,7 +41,7 @@ type ClFexitLoadInfo struct {
 	// must be between 1 and 64 characters in length, must start with alpha
 	// characters and must only contain alphanumeric characters and underscores.
 	// The pattern ^[a-zA-Z][a-zA-Z0-9_]+. is enforced.
-	// +required
+	// +optional
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
@@ -53,7 +53,7 @@ type ClFexitAttachInfo struct {
 	// attempt to be attached. To detach the FExit program, remove the link entry.
 	// +required
 	// +kubebuilder:validation:Enum=Attach;
-	Mode AttachTypeAttach `json:"mode"`
+	Mode AttachTypeAttach `json:"mode,omitempty"`
 }
 
 type ClFexitProgramInfoState struct {

apis/v1alpha1/cluster_kprobe_program_types.go

@@ -77,7 +77,7 @@ type ClKprobeAttachInfoState struct {
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
-	Function string `json:"function"`
+	Function string `json:"function,omitempty"`
 
 	// offset is the provisioned offset, whose value is added to the address of the
 	// attachment point function.

apis/v1alpha1/cluster_kretprobe_program_types.go

@@ -43,7 +43,7 @@ type ClKretprobeAttachInfo struct {
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
-	Function string `json:"function"`
+	Function string `json:"function,omitempty"`
 }
 
 type ClKretprobeProgramInfoState struct {
@@ -68,5 +68,5 @@ type ClKretprobeAttachInfoState struct {
 	// +kubebuilder:validation:Pattern="^[a-zA-Z][a-zA-Z0-9_]+."
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=64
-	Function string `json:"function"`
+	Function string `json:"function,omitempty"`
 }

apis/v1alpha1/cluster_tc_program_types.go

@@ -46,7 +46,7 @@ type ClTcAttachInfo struct {
 	// by providing a list of interface names, enabling auto discovery, or setting
 	// the primaryNodeInterface flag, but only one option is allowed.
 	// +required
-	InterfaceSelector InterfaceSelector `json:"interfaceSelector"`
+	InterfaceSelector InterfaceSelector `json:"interfaceSelector,omitzero"`
 
 	// networkNamespaces is an optional field that identifies the set of network
 	// namespaces in which to attach the eBPF program. If networkNamespaces is not
@@ -114,20 +114,22 @@ type ClTcAttachInfoState struct {
 	// interfaceName is the name of the interface the TC program should be
 	// attached.
 	// +required
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=63
-	InterfaceName string `json:"interfaceName"`
+	InterfaceName string `json:"interfaceName,omitempty"`
 
 	// netnsPath is the optional path to the network namespace inside of which the
 	// TC program should be attached.
 	// +optional
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=1023
 	NetnsPath string `json:"netnsPath,omitempty"`
 
 	// direction is the provisioned direction of traffic, Ingress or Egress, the TC
 	// program should be attached for a given network device.
 	// +required
 	// +kubebuilder:validation:Enum=Ingress;Egress
-	Direction TCDirectionType `json:"direction"`
+	Direction TCDirectionType `json:"direction,omitempty"`
 
 	// priority is the provisioned priority of the TC program in relation to other
 	// programs of the same type with the same attach point. It is a value from 0
@@ -143,5 +145,5 @@ type ClTcAttachInfoState struct {
 	// +required
 	// +kubebuilder:validation:MaxItems=1023
 	// +listType=atomic
-	ProceedOn []TcProceedOnValue `json:"proceedOn"`
+	ProceedOn []TcProceedOnValue `json:"proceedOn,omitempty"`
 }

apis/v1alpha1/cluster_tcx_program_types.go

@@ -44,7 +44,7 @@ type ClTcxAttachInfo struct {
 	// by providing a list of interface names, enabling auto discovery, or setting
 	// the primaryNodeInterface flag, but only one option is allowed.
 	// +required
-	InterfaceSelector InterfaceSelector `json:"interfaceSelector"`
+	InterfaceSelector InterfaceSelector `json:"interfaceSelector,omitzero"`
 
 	// networkNamespaces is an optional field that identifies the set of network
 	// namespaces in which to attach the eBPF program. If networkNamespaces is not
@@ -63,7 +63,7 @@ type ClTcxAttachInfo struct {
 	// transmitted by the interface.
 	// +required
 	// +kubebuilder:validation:Enum=Ingress;Egress
-	Direction TCDirectionType `json:"direction"`
+	Direction TCDirectionType `json:"direction,omitempty"`
 
 	// priority is an optional field and determines the execution order of the TCX
 	// program relative to other TCX programs attached to the same attachment
@@ -96,21 +96,23 @@ type ClTcxAttachInfoState struct {
 	// interfaceName is the name of the interface the TCX program should be
 	// attached. interfaceName must not exceed 63 characters in length.
 	// +required
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=63
-	InterfaceName string `json:"interfaceName"`
+	InterfaceName string `json:"interfaceName,omitempty"`
 
 	// netnsPath is the optional path to the network namespace inside of which the
 	// TCX program should be attached. If not provided, the default network
 	// namespace is used. netnsPath must not exceed 1023 characters in length.
 	// +optional
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=1023
 	NetnsPath string `json:"netnsPath,omitempty"`
 
 	// direction is the provisioned direction of traffic, Ingress or Egress, the TC
 	// program should be attached for a given network device.
 	// +required
 	// +kubebuilder:validation:Enum=Ingress;Egress
-	Direction TCDirectionType `json:"direction"`
+	Direction TCDirectionType `json:"direction,omitempty"`
 
 	// priority is the provisioned priority of the TCX program in relation to other
 	// programs of the same type with the same attach point. It is a value from 0