Skip to content
This repository has been archived by the owner on Dec 16, 2024. It is now read-only.

Add PolicyAffected status condition for gateway #324

Closed
maleck13 opened this issue Jul 11, 2023 · 0 comments · Fixed by #436
Closed

Add PolicyAffected status condition for gateway #324

maleck13 opened this issue Jul 11, 2023 · 0 comments · Fixed by #436
Assignees
@maksymvavilov
Labels
mvp refined

Comments

@maleck13
Copy link
Contributor

maleck13 commented Jul 11, 2023 •
edited
Loading

With DNSPolicy targeting gateways, it is import we update the gateway status with a condition to let the owner know the gateway is affected by a given DNSPolicy resource. This will also bring us into line with the latest proposed changes around policy discovery

https://github.com/kubernetes-sigs/gateway-api/pull/2128/files#diff-afe84021d0647e83f420f99f5d18b392abe5ec82d68f03156c7534de9f19a30aR888-R899

https://gateway-api.sigs.k8s.io/geps/gep-713/#conditions
@maleck13 maleck13 converted this from a draft issue Jul 11, 2023
@maleck13 maleck13 added the mvp label Jul 11, 2023
@maleck13 maleck13 changed the title add PolicyAffected status condition for gateway Add PolicyAffected status condition for gateway Aug 9, 2023
@maksymvavilov maksymvavilov self-assigned this Aug 10, 2023
@maksymvavilov maksymvavilov moved this from Todo to In Progress in Multicluster Gateway Controller Aug 10, 2023
@mikenairn mikenairn mentioned this issue Aug 15, 2023
Merged
maksymvavilov added a commit to maksymvavilov/multicluster-gateway-controller that referenced this issue Aug 17, 2023
@maksymvavilov
KuadrantGH-324 pollicyaffected condition for gateway
f540886
@maksymvavilov maksymvavilov linked a pull request Aug 17, 2023 that will close this issue
GH-324 pollicyaffected condition for gateway #433
Closed
maksymvavilov added a commit to maksymvavilov/multicluster-gateway-controller that referenced this issue Aug 17, 2023
@maksymvavilov
KuadrantGH-324 pollicyaffected condition for gateway
729a026
maksymvavilov added a commit to maksymvavilov/multicluster-gateway-controller that referenced this issue Aug 17, 2023
@maksymvavilov
KuadrantGH-324 pollicyaffected condition for gateway
026b971
maksymvavilov added a commit to maksymvavilov/multicluster-gateway-controller that referenced this issue Aug 17, 2023
@maksymvavilov
KuadrantGH-324 pollicyaffected condition for gateway
c877a64
@maleck13 maleck13 moved this from In Progress to Review in Multicluster Gateway Controller Aug 21, 2023
@maleck13 maleck13 mentioned this issue Aug 17, 2023
Closed
maleck13 pushed a commit that referenced this issue Aug 21, 2023
@maksymvavilov @maleck13
GH-324 pollicyaffected condition for gateway
5828f4b
@maleck13 maleck13 mentioned this issue Aug 21, 2023
Merged
maleck13 pushed a commit that referenced this issue Aug 22, 2023
@maksymvavilov @maleck13
GH-324 pollicyaffected condition for gateway
15ac42e 
updates to original PR

add status check integration test
maleck13 pushed a commit that referenced this issue Aug 22, 2023
@maksymvavilov @maleck13
GH-324 pollicyaffected condition for gateway
cf56892 
updates to original PR

add status check integration test

add namespace to policy affected conditon update tests
openshift-merge-robot added a commit that referenced this issue Aug 22, 2023
@openshift-merge-robot
Merge pull request #436 from Kuadrant/GH-324-policy-affected-condition
2c58229 
Gh 324 policy affected condition
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@maksymvavilov maksymvavilov

Labels
mvp refined
Projects
No open projects
Multicluster Gateway Controller
Status: Done
Milestone
No milestone
2 participants
@maleck13 @maksymvavilov