Skip to content

Commit 02a8c4e

Browse files
committed
resolved nits from pr
1 parent bae503d commit 02a8c4e

File tree

4 files changed

+4
-6
lines changed

4 files changed

+4
-6
lines changed

crd/multitenancy/api/v1alpha1/multitenantpodnetworkconfig.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ type MultitenantPodNetworkConfigStatus struct {
8585
// InterfaceInfos describes all of the network container goal state for this Pod
8686
// +kubebuilder:validation:Optional
8787
InterfaceInfos []InterfaceInfo `json:"interfaceInfos,omitempty"`
88-
// DefaultDenyAcl bool indicates whether default deny policy will be present on the pods upon pod creation
88+
// DefaultDenyACL bool indicates whether default deny policy will be present on the pods upon pod creation
89+
// +kubebuilder:validation:Optional
8990
DefaultDenyACL bool `json:"defaultDenyACL"`
9091
}
9192

crd/multitenancy/api/v1alpha1/podnetworkinstance.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,9 @@ type PodNetworkInstanceSpec struct {
5656
// optional for now in case orchestrator uses the deprecated fields
5757
// +kubebuilder:validation:Optional
5858
PodNetworkConfigs []PodNetworkConfig `json:"podNetworkConfigs"`
59-
// DefaultDenyAcl bool indicates whether default deny policy will be present on the pods upon pod creation
59+
// DefaultDenyACL bool indicates whether default deny policy will be present on the pods upon pod creation
6060
// +kubebuilder:default=false
61+
// +kubebuilder:validation:Optional
6162
DefaultDenyACL bool `json:"defaultDenyACL"`
6263
}
6364

crd/multitenancy/manifests/multitenancy.acn.azure.com_multitenantpodnetworkconfigs.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,6 @@ spec:
123123
primaryIP:
124124
description: Deprecated - use InterfaceInfos
125125
type: string
126-
required:
127-
- defaultDenyACL
128126
type: object
129127
type: object
130128
served: true

crd/multitenancy/manifests/multitenancy.acn.azure.com_podnetworkinstances.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,6 @@ spec:
8585
podnetwork:
8686
description: Deprecated - use PodNetworks
8787
type: string
88-
required:
89-
- defaultDenyACL
9088
type: object
9189
status:
9290
description: PodNetworkInstanceStatus defines the observed state of PodNetworkInstance

0 commit comments

Comments
 (0)