Skip to content

Commit f491f1b

Browse files
committed
Some minor rewording of the priority conflict
This makes it more clear that admins should make different priorties for conflicts.
1 parent e333b5e commit f491f1b

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

apis/v1alpha1/adminnetworkpolicy_types.go

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -56,16 +56,20 @@ type AdminNetworkPolicyStatus struct {
5656

5757
// AdminNetworkPolicySpec defines the desired state of AdminNetworkPolicy.
5858
type AdminNetworkPolicySpec struct {
59-
// Priority is a value from 0 to 1000. Policies with lower priority values have
60-
// higher precedence, and are checked before policies with higher priority values.
61-
// All AdminNetworkPolicy rules have higher precedence than NetworkPolicy or
59+
// Priority is a value from 0 to 1000. Policies with lower numeric
60+
// priority values have higher precedence, and are checked before
61+
// policies with higher priority values. All AdminNetworkPolicy rules
62+
// have higher precedence than NetworkPolicy or
6263
// BaselineAdminNetworkPolicy rules.
63-
// If two (or more) policies with the same priority could both match a connection,
64-
// then the implementation can apply any of the matching policies to the
65-
// connection, and there is no way for the user to reliably determine which one it
66-
// will choose. Administrators must be careful about assigning the priorities for
67-
// policies with rules that will match many connections, and ensure that policies
68-
// have unique priority values in cases where ambiguity would be unacceptable.
64+
//
65+
// If two (or more) policies with equal priority match the same
66+
// traffic, then any one of the policies will be applied to the
67+
// traffic. There is no way for the user to reliably determine the
68+
// policy that will be applied.
69+
//
70+
// Administrators should use different priorities numbers if
71+
// non-deterministic matching between overlapping policies is a
72+
// problem.
6973
//
7074
// +kubebuilder:validation:Minimum=0
7175
// +kubebuilder:validation:Maximum=1000

0 commit comments

Comments
 (0)