Skip to content

Commit 0f24f89

Browse files
authored
Implement ProxySettingsPolicy (#4286)
Problem: We need to implement the ProxySettingsPolicy API Solution: Implement ProxySettingsPolicy Manual Testing: Tested using the example files provided which will be the basis of the documentation: Steps (using the provided examples): Deploy an application which returns a large payload (coffee) and one which does not (tea) Apply the PSP for the Gateway which will support the smaller payload application Verify tea works but coffee does not (response too large error in NGINX logs) Apply the PSP for the coffee route which increases the proxy buffer size Verify coffee now works (note: return output to file if testing this as it fills the terminal)
1 parent 8181386 commit 0f24f89

File tree

39 files changed

+1405
-18
lines changed

39 files changed

+1405
-18
lines changed

.yamllint.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ rules:
2929
check-multi-line-strings: true
3030
ignore: |
3131
operators/**/*
32+
examples/proxy-settings-policy/app.yaml
3233
key-duplicates: enable
3334
key-ordering: disable
3435
line-length:

charts/nginx-gateway-fabric/templates/clusterrole.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,7 @@ rules:
129129
- clientsettingspolicies
130130
- observabilitypolicies
131131
- upstreamsettingspolicies
132+
- proxysettingspolicies
132133
{{- if .Values.nginxGateway.snippetsFilters.enable }}
133134
- snippetsfilters
134135
{{- end }}
@@ -142,6 +143,7 @@ rules:
142143
- clientsettingspolicies/status
143144
- observabilitypolicies/status
144145
- upstreamsettingspolicies/status
146+
- proxysettingspolicies/status
145147
{{- if .Values.nginxGateway.snippetsFilters.enable }}
146148
- snippetsfilters/status
147149
{{- end }}

deploy/azure/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

deploy/default/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

deploy/experimental-nginx-plus/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,7 @@ rules:
171171
- clientsettingspolicies
172172
- observabilitypolicies
173173
- upstreamsettingspolicies
174+
- proxysettingspolicies
174175
verbs:
175176
- list
176177
- watch
@@ -181,6 +182,7 @@ rules:
181182
- clientsettingspolicies/status
182183
- observabilitypolicies/status
183184
- upstreamsettingspolicies/status
185+
- proxysettingspolicies/status
184186
verbs:
185187
- update
186188
- apiGroups:

deploy/experimental/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,7 @@ rules:
171171
- clientsettingspolicies
172172
- observabilitypolicies
173173
- upstreamsettingspolicies
174+
- proxysettingspolicies
174175
verbs:
175176
- list
176177
- watch
@@ -181,6 +182,7 @@ rules:
181182
- clientsettingspolicies/status
182183
- observabilitypolicies/status
183184
- upstreamsettingspolicies/status
185+
- proxysettingspolicies/status
184186
verbs:
185187
- update
186188
- apiGroups:

deploy/inference-nginx-plus/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

deploy/inference/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

deploy/nginx-plus/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

deploy/nodeport/deploy.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ rules:
169169
- clientsettingspolicies
170170
- observabilitypolicies
171171
- upstreamsettingspolicies
172+
- proxysettingspolicies
172173
verbs:
173174
- list
174175
- watch
@@ -179,6 +180,7 @@ rules:
179180
- clientsettingspolicies/status
180181
- observabilitypolicies/status
181182
- upstreamsettingspolicies/status
183+
- proxysettingspolicies/status
182184
verbs:
183185
- update
184186
- apiGroups:

0 commit comments

Comments
 (0)