Validate policy in cli according to policy schema

[NoSkillGirl]

Related issue

closes #1700

What type of PR is this

/kind bug

Proposed Changes

• Changed policy crd format.
• Using K8s validate function to validate the resource.

Proof Manifests

Use the below policy (addRequest.yaml):

apiVersion: kyverno.io/v1
kind: ClusterPolicy
metadata:
  name: add-requests
spec:
  rules:
  - name: "Set memory and/or cpu requests for all pods in namespaces labeled 'myprivatelabel'"
    match:
      resources:
        kinds:
        - Pod
        namespaceSelector:
          matchExpressions:
            - key: namespace-type
              operator: In
              values:
              - myprivatelabel
    mutate:
      overlay:
        spec:
          containers:
          - (name): "*"
            resources:
              requests:
                cpu: "1000m"

Validate the above policy in kyverno CLI using:
kyverno validate addRequest.yaml

As the rule name is greater than 63 characters, kyverno validation fails with the following error:

Policy add-requests is invalid.
Error: invalid policy.
Cause: [spec.rules.name: Invalid value: "Set memory and/or cpu requests for all pods in namespaces labeled 'myprivatelabel'": spec.rules.name in body should be at most 63 chars long]

Checklist

• I have read the contributing guidelines.
• [] I have added tests that prove my fix is effective or that my feature works.
• [] My PR contains new or altered behavior to Kyverno and
  • [] I have added or changed the documentation myself in an existing PR and the link is:
  • [] I have raised an issue in kyverno/website to track the doc update and the link is:
  • [] I have read the PR documentation guide and followed the process including adding proof manifests to this PR.

[realshuting]

@NoSkillGirl - can you add "proof manifests" to this PR?

[NoSkillGirl]

@NoSkillGirl - can you add "proof manifests" to this PR?

Added.

[realshuting]

@NoSkillGirl - the changes look good, can you also add unit tests to cover this change?

[realshuting]

Looks good!

Forgot to mention this last time, can you address this?

pkg/kyverno/validate/command.go#L199

[golangci] reported by reviewdog 🐶
S1002: should omit comparison to bool constant, can be simplified to `invalidPolicyFound` (gosimple)

Raw Output:
pkg/kyverno/validate/command.go:199:5: S1002: should omit comparison to bool constant, can be simplified to `invalidPolicyFound` (gosimple)
	if invalidPolicyFound == true {

[NoSkillGirl]

Looks good!

Forgot to mention this last time, can you address this?

pkg/kyverno/validate/command.go#L199

[golangci] reported by reviewdog 🐶
S1002: should omit comparison to bool constant, can be simplified to `invalidPolicyFound` (gosimple)

Raw Output:
pkg/kyverno/validate/command.go:199:5: S1002: should omit comparison to bool constant, can be simplified to `invalidPolicyFound` (gosimple)
	if invalidPolicyFound == true {

will add the change