-
Notifications
You must be signed in to change notification settings - Fork 300
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
Update packages with high CVE issues #192
Conversation
Looks like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is going to require some code changes for API movement; however skimming through the new vendor there's nothing that jumps out at me as being ripe for regressions.
Looks like there's still some test failures to dig into related to seccomp profiles. |
This looks like an error with the integration action in general. The same failure is happening on all of the other open PRs and master 👀 |
Ah, good catch. I think we will have to follow-up on that later than. If the failures are not new, LGTM 👍 |
It looks like master previously passed and rerunning it now fails due to this issue. So this is a new issue with integration. I'm going to hold off on merging this and take a look at the problem now and I will open a fix in another PR |
c8fc4fa
to
fe87571
Compare
This updates packages to fix the CVEs reported in #191
Running the scanner with these changes shows the reported CVEs are fixed.