Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Fix security alerts related to isolated-cluster #4472

Closed
wants to merge 1 commit into from
Closed

Conversation

Vandy-P
Copy link
Contributor

@Vandy-P Vandy-P commented Mar 14, 2023

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #4473

Describe testing done for PR

Run trivy scan before and after changes.

Before:
image

After:
image

Release note

NA

Signed-off-by: Vandana Pathak <pathakv@sjc-ubu-eng-127.vmware.com>
@Vandy-P Vandy-P requested a review from a team as a code owner March 14, 2023 09:00
@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #4472 (48446c9) into main (99dd621) will decrease coverage by 0.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4472      +/-   ##
==========================================
- Coverage   49.75%   48.78%   -0.98%     
==========================================
  Files         452      482      +30     
  Lines       45152    47272    +2120     
==========================================
+ Hits        22466    23060     +594     
- Misses      20566    22032    +1466     
- Partials     2120     2180      +60     

see 35 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Vandy-P Vandy-P changed the title [WIP]Fix security alerts related to isolated-cluster Fix security alerts related to isolated-cluster Mar 14, 2023
@sathyanarays sathyanarays self-requested a review March 14, 2023 10:51
@sathyanarays
Copy link
Contributor

/ok-to-merge

@sathyanarays sathyanarays added the ok-to-merge PRs should be labelled with this before merging label Mar 14, 2023
@Vandy-P Vandy-P closed this Mar 15, 2023
@Vandy-P
Copy link
Contributor Author

Vandy-P commented Mar 15, 2023

Taken care as a part of #4470 PR

@Vandy-P Vandy-P deleted the security_fix_1 branch March 15, 2023 06:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix identified security alerts generated for isolated-cluster plugin
3 participants