Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: modify the ingress resource #1156

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

MariamFahmy98
Copy link
Contributor

Related Issue(s)

N/A

Description

This PR fixes the test other-cel/require-ingress-https as it was failing because of the following error:

 l.go:52: | 11:40:28 | require-ingress-https | step-02   | CREATE    | WARN  | networking.k8s.io/v1/Ingress @ chainsaw-moral-parrot/goodingress01
        === ERROR
        Ingress in version "v1" cannot be handled as a Ingress: strict decoding error: unknown field "spec.rules[0].https"

Checklist

  • I have read the policy contribution guidelines.
  • I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
  • I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

Signed-off-by: Mariam Fahmy <[email protected]>
Signed-off-by: Mariam Fahmy <[email protected]>
@eddycharly eddycharly enabled auto-merge (squash) September 9, 2024 14:02
@eddycharly
Copy link
Member

/cherry-pick release-1.13

@eddycharly eddycharly merged commit 4707b3d into kyverno:main Sep 9, 2024
276 checks passed
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Sep 9, 2024
* fix: modify the ingress resource

Signed-off-by: Mariam Fahmy <[email protected]>

* fix resources

Signed-off-by: Mariam Fahmy <[email protected]>

* fix tekton test

Signed-off-by: Mariam Fahmy <[email protected]>

---------

Signed-off-by: Mariam Fahmy <[email protected]>
@MariamFahmy98 MariamFahmy98 deleted the fix-ingress-test branch September 9, 2024 14:23
eddycharly pushed a commit that referenced this pull request Sep 9, 2024
* fix: modify the ingress resource



* fix resources



* fix tekton test



---------

Signed-off-by: Mariam Fahmy <[email protected]>
Co-authored-by: Mariam Fahmy <[email protected]>
@vishal-chdhry
Copy link
Member

/cherry-pick release-1.12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants