Skip to content

fix: image verify cache test #8462

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

Merged
merged 7 commits into from
Sep 20, 2023
Merged

fix: image verify cache test #8462

merged 7 commits into from
Sep 20, 2023

Conversation

vishal-chdhry
Copy link
Member

Explanation

This PR fixes image verify cache tests

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
@eddycharly
Copy link
Member

Needs label/milestone

eddycharly
eddycharly previously approved these changes Sep 20, 2023
Copy link
Member

@eddycharly eddycharly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤷

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Merging #8462 (42a0a29) into main (4f0e378) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #8462   +/-   ##
=======================================
  Coverage   35.54%   35.54%           
=======================================
  Files         309      309           
  Lines       24996    24996           
=======================================
  Hits         8885     8885           
  Misses      15220    15220           
  Partials      891      891           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
Copy link
Member

@eddycharly eddycharly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's give this a chance 🤞

@eddycharly
Copy link
Member

/cherry-pick release-1.11

@eddycharly eddycharly merged commit fd01e50 into kyverno:main Sep 20, 2023
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Sep 20, 2023
* fix: image verify cache test

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>

* feat: print err message

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>

* feat: clear mock

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>

* feat: defer clear mock

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>

---------

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
Co-authored-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@eddycharly eddycharly added the cherry-pick-completed The PR was cherry-picked (or merged) to required release branches label Sep 20, 2023
eddycharly added a commit that referenced this pull request Sep 20, 2023
* fix: image verify cache test



* feat: print err message



* feat: clear mock



* feat: defer clear mock



---------

Signed-off-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
Co-authored-by: Vishal Choudhary <sendtovishalchoudhary@gmail.com>
Co-authored-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-completed The PR was cherry-picked (or merged) to required release branches cherry-pick-required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants