From b1789ab7cd0774c6b717f51971fc0a4b1fd08a10 Mon Sep 17 00:00:00 2001 From: Sergio Franco Date: Wed, 29 Nov 2023 10:29:31 +0000 Subject: [PATCH] Addressing suggestions --- .gitignore | 1 - rfcs/0004-policy-sync-v1.md | 12 +++++++++++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index 485dee64..e69de29b 100644 --- a/.gitignore +++ b/.gitignore @@ -1 +0,0 @@ -.idea diff --git a/rfcs/0004-policy-sync-v1.md b/rfcs/0004-policy-sync-v1.md index 4f5b4ff8..b0f5cf4f 100644 --- a/rfcs/0004-policy-sync-v1.md +++ b/rfcs/0004-policy-sync-v1.md @@ -71,6 +71,13 @@ by placing the policy in the spoke clusters, with the following mutations: * The `TargetRef` of the policy is changed to reference the downstream Gateway * The `kuadrant.io/policy-synced` annotation is set +The upstream policy is annotated with a reference to the name and namespace +of the downstream policies: + ```yaml + annotations: + "kuadrant.io/policies-synced": "[{\"cluster\": \"...\", \"name\": \"...\", \"namespace\": \"...\"}]" + ``` + # Reference-level explanation [reference-level-explanation]: #reference-level-explanation @@ -114,7 +121,10 @@ The controller will set the `kuadrant.io/policy-synced` annotation on the policy when placing it in the spoke cluster. The Kuadrant operator will be aware of the presence of this annotation, and, in case -of conflicts, override Policies that contain this annotation. +of conflicts, override Policies that contain this annotation. When a policy is +overriden due to conflicts, the `Enforced` status will be set to `False`, with +the reason being `Overriden` and a human readable message explaining the reason +why the policy was overriden. See [Policy Status RFC](./0004-policy-status.md) # Drawbacks [drawbacks]: #drawbacks