mirror of
https://github.com/kyverno/kyverno.git
synced 2025-03-09 17:37:12 +00:00
* feat: mutate existing, replace GR by UR in webhook server (#3601) * add attributes for post mutation Signed-off-by: ShutingZhao <shuting@nirmata.com> * add UR informer to webhook server Signed-off-by: ShutingZhao <shuting@nirmata.com> * - replace gr with ur in the webhook server; - create ur for mutateExsiting policies Signed-off-by: ShutingZhao <shuting@nirmata.com> * replace gr by ur across entire packages Signed-off-by: ShutingZhao <shuting@nirmata.com> * add YAMLs Signed-off-by: ShutingZhao <shuting@nirmata.com> * update api docs & fix unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * add UR deletion handler Signed-off-by: ShutingZhao <shuting@nirmata.com> * add api docs for v1beta1 Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix clientset method Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix v1beta1 client registration Signed-off-by: ShutingZhao <shuting@nirmata.com> * feat: mutate existing - generates UR for admission requests (#3623) Signed-off-by: ShutingZhao <shuting@nirmata.com> * replace with UR in policy controller generate rules (#3635) Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com> * - enable mutate engine to process mutateExisting rules; - add unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * implemented ur background reconciliation for mutateExisting policies Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix webhook update error Signed-off-by: ShutingZhao <shuting@nirmata.com> * temporary comment out new unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * feat: mutate existing, replace GR by UR in webhook server (#3601) * add attributes for post mutation Signed-off-by: ShutingZhao <shuting@nirmata.com> * add UR informer to webhook server Signed-off-by: ShutingZhao <shuting@nirmata.com> * - replace gr with ur in the webhook server; - create ur for mutateExsiting policies Signed-off-by: ShutingZhao <shuting@nirmata.com> * replace gr by ur across entire packages Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix missing policy.kyverno.io/policy-name label (#3599) Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com> * refactor cli code from pkg to cmd (#3591) * refactor cli code from pkg to cmd Signed-off-by: Mritunjay Sharma <mritunjaysharma394@gmail.com> * fixes in imports Signed-off-by: Mritunjay Sharma <mritunjaysharma394@gmail.com> * fixes tests Signed-off-by: Mritunjay Sharma <mritunjaysharma394@gmail.com> * fixed conflicts Signed-off-by: Mritunjay Sharma <mritunjaysharma394@gmail.com> * moved non-commands to utils Signed-off-by: Mritunjay Sharma <mritunjaysharma394@gmail.com> Co-authored-by: Vyankatesh Kudtarkar <vyankateshkd@gmail.com> * add YAMLs Signed-off-by: ShutingZhao <shuting@nirmata.com> * update api docs & fix unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * add UR deletion handler Signed-off-by: ShutingZhao <shuting@nirmata.com> * add api docs for v1beta1 Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix clientset method Signed-off-by: ShutingZhao <shuting@nirmata.com> * add-kms-libraries for cosign (#3603) * add-kms-libraries Signed-off-by: anushkamittal20 <anumittal4641@gmail.com> * Shifted providers to cosign package Signed-off-by: anushkamittal20 <anumittal4641@gmail.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * Add support for custom image extractors (#3596) Signed-off-by: Sambhav Kothari <skothari44@bloomberg.net> * Update vulnerable dependencies (#3577) Signed-off-by: Shubham Gupta <shubham.gupta2956@gmail.com> Co-authored-by: Jim Bugwadia <jim@nirmata.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix v1beta1 client registration Signed-off-by: ShutingZhao <shuting@nirmata.com> * feat: mutate existing - generates UR for admission requests (#3623) Signed-off-by: ShutingZhao <shuting@nirmata.com> * updating version in Chart.yaml (#3618) * updatimg version in Chart.yaml Signed-off-by: Prateeknandle <prateeknandle@gmail.com> * changes from, make gen-helm Signed-off-by: Prateeknandle <prateeknandle@gmail.com> Co-authored-by: Vyankatesh Kudtarkar <vyankateshkd@gmail.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * Allow kyverno-policies to have preconditions defined (#3606) * Allow kyverno-policies to have preconditions defined Signed-off-by: Trey Dockendorf <tdockendorf@osc.edu> * Fix docs Signed-off-by: Trey Dockendorf <tdockendorf@osc.edu> Signed-off-by: ShutingZhao <shuting@nirmata.com> * replace with UR in policy controller generate rules (#3635) Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * - enable mutate engine to process mutateExisting rules; - add unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * implemented ur background reconciliation for mutateExisting policies Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix webhook update error Signed-off-by: ShutingZhao <shuting@nirmata.com> * temporary comment out new unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * Image verify attestors (#3614) * fix logs Signed-off-by: Jim Bugwadia <jim@nirmata.com> * fix logs Signed-off-by: Jim Bugwadia <jim@nirmata.com> * support multiple attestors Signed-off-by: Jim Bugwadia <jim@nirmata.com> * rm CLI tests (not currently supported) Signed-off-by: Jim Bugwadia <jim@nirmata.com> * apply attestor repo Signed-off-by: Jim Bugwadia <jim@nirmata.com> * fix linter issues Signed-off-by: Jim Bugwadia <jim@nirmata.com> * fix entryError assignment Signed-off-by: Jim Bugwadia <jim@nirmata.com> * fix tests Signed-off-by: Jim Bugwadia <jim@nirmata.com> * format Signed-off-by: Jim Bugwadia <jim@nirmata.com> * add intermediary certs Signed-off-by: Jim Bugwadia <jim@nirmata.com> * Allow defining imagePullSecrets (#3633) * Allow defining imagePullSecrets Signed-off-by: Trey Dockendorf <tdockendorf@osc.edu> * Use dict for imagePullSecrets Signed-off-by: Trey Dockendorf <tdockendorf@osc.edu> * Simplify how imagePullSecrets is defined Signed-off-by: Trey Dockendorf <tdockendorf@osc.edu> Signed-off-by: ShutingZhao <shuting@nirmata.com> * Fix race condition in pCache (#3632) * fix race condition in pCache Signed-off-by: ShutingZhao <shuting@nirmata.com> * refact: remove unused Run function from generate (#3638) Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com> * Remove helm mode setting (#3628) Signed-off-by: Charles-Edouard Brétéché <charled.breteche@gmail.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * refactor: image utils (#3630) Signed-off-by: Charles-Edouard Brétéché <charled.breteche@gmail.com> Signed-off-by: ShutingZhao <shuting@nirmata.com> * -resolve lift comments; -fix informer sync issue Signed-off-by: ShutingZhao <shuting@nirmata.com> * refact the update request cleanup controller Signed-off-by: prateekpandey14 <prateek.pandey@nirmata.com> * - fix delete request for mutateExisting; - fix context variable substitution; - improve logging Signed-off-by: ShutingZhao <shuting@nirmata.com> * - enable events; - add last applied annotation Signed-off-by: ShutingZhao <shuting@nirmata.com> * enable mutate existing on policy creation Signed-off-by: ShutingZhao <shuting@nirmata.com> * update autogen code Signed-off-by: ShutingZhao <shuting@nirmata.com> * merge main Signed-off-by: ShutingZhao <shuting@nirmata.com> * add unit tests Signed-off-by: ShutingZhao <shuting@nirmata.com> * address list comments Signed-off-by: ShutingZhao <shuting@nirmata.com> * update api docs Signed-off-by: ShutingZhao <shuting@nirmata.com> * fix "Implicit memory aliasing in for loop" Signed-off-by: ShutingZhao <shuting@nirmata.com> * remove unused definitions Signed-off-by: ShutingZhao <shuting@nirmata.com> * update api docs Signed-off-by: ShutingZhao <shuting@nirmata.com> Co-authored-by: Prateek Pandey <prateek.pandey@nirmata.com> Co-authored-by: Mritunjay Kumar Sharma <mritunjaysharma394@gmail.com> Co-authored-by: Vyankatesh Kudtarkar <vyankateshkd@gmail.com> Co-authored-by: Anushka Mittal <55237170+anushkamittal20@users.noreply.github.com> Co-authored-by: Sambhav Kothari <sambhavs.email@gmail.com> Co-authored-by: Shubham Gupta <shubham.gupta2956@gmail.com> Co-authored-by: Jim Bugwadia <jim@nirmata.com> Co-authored-by: Prateek Nandle <56027872+Prateeknandle@users.noreply.github.com> Co-authored-by: treydock <tdockendorf@osc.edu> Co-authored-by: Charles-Edouard Brétéché <charled.breteche@gmail.com>
224 lines
7 KiB
Go
224 lines
7 KiB
Go
package mutate
|
|
|
|
import (
|
|
"encoding/json"
|
|
"testing"
|
|
|
|
"github.com/kyverno/kyverno/pkg/engine/context"
|
|
"k8s.io/apiextensions-apiserver/pkg/apis/apiextensions"
|
|
|
|
types "github.com/kyverno/kyverno/api/kyverno/v1"
|
|
"github.com/kyverno/kyverno/pkg/engine/response"
|
|
|
|
"gotest.tools/assert"
|
|
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
|
|
"sigs.k8s.io/controller-runtime/pkg/log"
|
|
|
|
"github.com/kyverno/kyverno/pkg/engine/utils"
|
|
)
|
|
|
|
// jsonPatch is used to build test patches
|
|
type jsonPatch struct {
|
|
Path string `json:"path,omitempty" yaml:"path,omitempty"`
|
|
Operation string `json:"op,omitempty" yaml:"op,omitempty"`
|
|
Value apiextensions.JSON `json:"value,omitempty" yaml:"value,omitempty"`
|
|
}
|
|
|
|
const endpointsDocument string = `{
|
|
"kind": "Endpoints",
|
|
"apiVersion": "v1",
|
|
"metadata": {
|
|
"name": "my-endpoint-service",
|
|
"labels": {
|
|
"originalLabel": "isHere"
|
|
}
|
|
},
|
|
"subsets": [
|
|
{
|
|
"addresses": [
|
|
{
|
|
"ip": "1.2.3.4"
|
|
}
|
|
],
|
|
"ports": [
|
|
{
|
|
"port": 9376
|
|
}
|
|
]
|
|
}
|
|
]
|
|
}`
|
|
|
|
func applyPatches(rule *types.Rule, resource unstructured.Unstructured) (*response.RuleResponse, unstructured.Unstructured) {
|
|
mutateResp := Mutate(rule, context.NewContext(), resource, log.Log)
|
|
|
|
if mutateResp.Status != response.RuleStatusPass {
|
|
return &response.RuleResponse{
|
|
Type: response.Mutation,
|
|
Status: mutateResp.Status,
|
|
Message: mutateResp.Message,
|
|
}, resource
|
|
}
|
|
|
|
return &response.RuleResponse{
|
|
Type: response.Mutation,
|
|
Status: response.RuleStatusPass,
|
|
Patches: mutateResp.Patches,
|
|
}, mutateResp.PatchedResource
|
|
}
|
|
|
|
func TestProcessPatches_EmptyPatches(t *testing.T) {
|
|
var emptyRule = &types.Rule{Name: "emptyRule"}
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
|
|
rr, _ := applyPatches(emptyRule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusError)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func makeAddIsMutatedLabelPatch() jsonPatch {
|
|
return jsonPatch{
|
|
Path: "/metadata/labels/is-mutated",
|
|
Operation: "add",
|
|
Value: "true",
|
|
}
|
|
}
|
|
|
|
func makeRuleWithPatch(t *testing.T, patch jsonPatch) *types.Rule {
|
|
patches := []jsonPatch{patch}
|
|
return makeRuleWithPatches(t, patches)
|
|
}
|
|
|
|
func makeRuleWithPatches(t *testing.T, patches []jsonPatch) *types.Rule {
|
|
jsonPatches, err := json.Marshal(patches)
|
|
if err != nil {
|
|
t.Errorf("failed to marshal patch: %v", err)
|
|
}
|
|
|
|
mutation := types.Mutation{
|
|
PatchesJSON6902: string(jsonPatches),
|
|
}
|
|
return &types.Rule{
|
|
Mutation: mutation,
|
|
}
|
|
}
|
|
|
|
func TestProcessPatches_EmptyDocument(t *testing.T) {
|
|
rule := makeRuleWithPatch(t, makeAddIsMutatedLabelPatch())
|
|
rr, _ := applyPatches(rule, unstructured.Unstructured{})
|
|
assert.Equal(t, rr.Status, response.RuleStatusFail)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func TestProcessPatches_AllEmpty(t *testing.T) {
|
|
emptyRule := &types.Rule{}
|
|
rr, _ := applyPatches(emptyRule, unstructured.Unstructured{})
|
|
assert.Equal(t, rr.Status, response.RuleStatusError)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func TestProcessPatches_AddPathDoesntExist(t *testing.T) {
|
|
patch := makeAddIsMutatedLabelPatch()
|
|
patch.Path = "/metadata/additional/is-mutated"
|
|
rule := makeRuleWithPatch(t, patch)
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusSkip)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func TestProcessPatches_RemovePathDoesntExist(t *testing.T) {
|
|
patch := jsonPatch{Path: "/metadata/labels/is-mutated", Operation: "remove"}
|
|
rule := makeRuleWithPatch(t, patch)
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusSkip)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func TestProcessPatches_AddAndRemovePathsDontExist_EmptyResult(t *testing.T) {
|
|
patch1 := jsonPatch{Path: "/metadata/labels/is-mutated", Operation: "remove"}
|
|
patch2 := jsonPatch{Path: "/spec/labels/label3", Operation: "add", Value: "label3Value"}
|
|
rule := makeRuleWithPatches(t, []jsonPatch{patch1, patch2})
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusPass)
|
|
assert.Equal(t, len(rr.Patches), 1)
|
|
}
|
|
|
|
func TestProcessPatches_AddAndRemovePathsDontExist_ContinueOnError_NotEmptyResult(t *testing.T) {
|
|
patch1 := jsonPatch{Path: "/metadata/labels/is-mutated", Operation: "remove"}
|
|
patch2 := jsonPatch{Path: "/spec/labels/label2", Operation: "remove", Value: "label2Value"}
|
|
patch3 := jsonPatch{Path: "/metadata/labels/label3", Operation: "add", Value: "label3Value"}
|
|
rule := makeRuleWithPatches(t, []jsonPatch{patch1, patch2, patch3})
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusPass)
|
|
assert.Assert(t, len(rr.Patches) != 0)
|
|
assertEqStringAndData(t, `{"path":"/metadata/labels/label3","op":"add","value":"label3Value"}`, rr.Patches[0])
|
|
}
|
|
|
|
func TestProcessPatches_RemovePathDoesntExist_EmptyResult(t *testing.T) {
|
|
patch := jsonPatch{Path: "/metadata/labels/is-mutated", Operation: "remove"}
|
|
rule := makeRuleWithPatch(t, patch)
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusSkip)
|
|
assert.Assert(t, len(rr.Patches) == 0)
|
|
}
|
|
|
|
func TestProcessPatches_RemovePathDoesntExist_NotEmptyResult(t *testing.T) {
|
|
patch1 := jsonPatch{Path: "/metadata/labels/is-mutated", Operation: "remove"}
|
|
patch2 := jsonPatch{Path: "/metadata/labels/label2", Operation: "add", Value: "label2Value"}
|
|
rule := makeRuleWithPatches(t, []jsonPatch{patch1, patch2})
|
|
resourceUnstructured, err := utils.ConvertToUnstructured([]byte(endpointsDocument))
|
|
if err != nil {
|
|
t.Error(err)
|
|
}
|
|
rr, _ := applyPatches(rule, *resourceUnstructured)
|
|
assert.Equal(t, rr.Status, response.RuleStatusPass)
|
|
assert.Assert(t, len(rr.Patches) == 1)
|
|
assertEqStringAndData(t, `{"path":"/metadata/labels/label2","op":"add","value":"label2Value"}`, rr.Patches[0])
|
|
}
|
|
|
|
func assertEqDataImpl(t *testing.T, expected, actual []byte, formatModifier string) {
|
|
if len(expected) != len(actual) {
|
|
t.Errorf("len(expected) != len(actual): %d != %d\n1:"+formatModifier+"\n2:"+formatModifier, len(expected), len(actual), expected, actual)
|
|
return
|
|
}
|
|
|
|
for idx, val := range actual {
|
|
if val != expected[idx] {
|
|
t.Errorf("Slices not equal at index %d:\n1:"+formatModifier+"\n2:"+formatModifier, idx, expected, actual)
|
|
}
|
|
}
|
|
}
|
|
|
|
func assertEqStringAndData(t *testing.T, str string, data []byte) {
|
|
var p1 jsonPatch
|
|
json.Unmarshal([]byte(str), &p1)
|
|
|
|
var p2 jsonPatch
|
|
json.Unmarshal([]byte(data), &p2)
|
|
|
|
assert.Equal(t, p1, p2)
|
|
}
|