deprecated karmada.io/override and karmada.io/cluster-override label

Signed-off-by: RainbowMango <renhongcai@huawei.com>
This commit is contained in:
RainbowMango 2021-03-29 19:51:14 +08:00 committed by Hongcai Ren
parent f1777392f9
commit 4bba4df183
2 changed files with 6 additions and 27 deletions

View File

@ -24,20 +24,6 @@ const (
// WorkNameLabel is added to objects to specify associated Work's name. // WorkNameLabel is added to objects to specify associated Work's name.
WorkNameLabel = "work.karmada.io/name" WorkNameLabel = "work.karmada.io/name"
// OverrideClaimKey will set in Work resource, indicates that
// the resource is overridden by override policies
OverrideClaimKey = "karmada.io/overridden-by"
// AppliedOverrideKey is the key of a OverridePolicy list.
// It is used to annotates what override policies have been applied for a specific manifest.
// The value is a comma-separated list of override policy names, the namespace is omitted.
AppliedOverrideKey = "karmada.io/override"
// AppliedClusterOverrideKey is the key of a ClusterOverridePolicy list.
// it is used to annotates what cluster override policies have been applied for a specific manifest.
// The value is a comma-separated list of cluster override policy names.
AppliedClusterOverrideKey = "karmada.io/cluster-override"
) )
// Define annotations used by karmada system. // Define annotations used by karmada system.

View File

@ -3,7 +3,6 @@ package overridemanager
import ( import (
"context" "context"
"encoding/json" "encoding/json"
"strings"
"github.com/evanphx/json-patch/v5" "github.com/evanphx/json-patch/v5"
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured" "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
@ -94,19 +93,16 @@ func (o *overrideManagerImpl) applyClusterOverrides(rawObj *unstructured.Unstruc
return nil, nil return nil, nil
} }
var appliedList []string appliedList := &AppliedOverrides{}
appliedOverrides := &AppliedOverrides{}
for _, p := range matchedPolicies { for _, p := range matchedPolicies {
if err := applyJSONPatch(rawObj, parseJSONPatch(p.Spec.Overriders.Plaintext)); err != nil { if err := applyJSONPatch(rawObj, parseJSONPatch(p.Spec.Overriders.Plaintext)); err != nil {
return nil, err return nil, err
} }
klog.V(2).Infof("Applied cluster overrides(%s) for %s/%s", p.Name, rawObj.GetNamespace(), rawObj.GetName()) klog.V(2).Infof("Applied cluster overrides(%s) for %s/%s", p.Name, rawObj.GetNamespace(), rawObj.GetName())
appliedList = append(appliedList, p.Name) appliedList.Add(p.Name, p.Spec.Overriders)
appliedOverrides.Add(p.Name, p.Spec.Overriders)
} }
util.MergeAnnotation(rawObj, util.AppliedClusterOverrideKey, strings.Join(appliedList, ","))
return appliedOverrides, nil return appliedList, nil
} }
// applyNamespacedOverrides will apply overrides according to OverridePolicy instructions. // applyNamespacedOverrides will apply overrides according to OverridePolicy instructions.
@ -128,19 +124,16 @@ func (o *overrideManagerImpl) applyNamespacedOverrides(rawObj *unstructured.Unst
return nil, nil return nil, nil
} }
var appliedList []string appliedList := &AppliedOverrides{}
appliedOverrides := &AppliedOverrides{}
for _, p := range matchedPolicies { for _, p := range matchedPolicies {
if err := applyJSONPatch(rawObj, parseJSONPatch(p.Spec.Overriders.Plaintext)); err != nil { if err := applyJSONPatch(rawObj, parseJSONPatch(p.Spec.Overriders.Plaintext)); err != nil {
return nil, err return nil, err
} }
klog.V(2).Infof("Applied overrides(%s/%s) for %s/%s", p.Namespace, p.Name, rawObj.GetNamespace(), rawObj.GetName()) klog.V(2).Infof("Applied overrides(%s/%s) for %s/%s", p.Namespace, p.Name, rawObj.GetNamespace(), rawObj.GetName())
appliedList = append(appliedList, p.Name) appliedList.Add(p.Name, p.Spec.Overriders)
appliedOverrides.Add(p.Name, p.Spec.Overriders)
} }
util.MergeAnnotation(rawObj, util.AppliedOverrideKey, strings.Join(appliedList, ","))
return appliedOverrides, nil return appliedList, nil
} }
func (o *overrideManagerImpl) getMatchedClusterOverridePolicy(policies []policyv1alpha1.ClusterOverridePolicy, resource *unstructured.Unstructured, cluster *clusterv1alpha1.Cluster) []policyv1alpha1.ClusterOverridePolicy { func (o *overrideManagerImpl) getMatchedClusterOverridePolicy(policies []policyv1alpha1.ClusterOverridePolicy, resource *unstructured.Unstructured, cluster *clusterv1alpha1.Cluster) []policyv1alpha1.ClusterOverridePolicy {