Skip to content

fix(cpo): diff SG permissions before AuthorizeSecurityGroupIngress - #8869

Closed
sdminonne wants to merge 1 commit into
openshift:mainfrom
sdminonne:OCPBUGS-93738
Closed

sdminonne wants to merge 1 commit into
openshift:mainfrom
sdminonne:OCPBUGS-93738

fix(cpo): diff SG permissions before AuthorizeSecurityGroupIngress

c15b7fc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 30, 2026 in 1s

43.19% (+0.00%) compared to bb1af6d

View this Pull Request on Codecov

43.19% (+0.00%) compared to bb1af6d

Details

Codecov Report

❌ Patch coverage is 55.76923% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.19%. Comparing base (bb1af6d) to head (c15b7fc).
⚠️ Report is 730 commits behind head on main.

Files with missing lines Patch % Lines
...ostedcontrolplane/hostedcontrolplane_controller.go 0.00% 13 Missing ⚠️
support/awsutil/sg.go 76.31% 6 Missing and 3 partials ⚠️
...ollers/awsprivatelink/awsprivatelink_controller.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8869   +/-   ##
=======================================
  Coverage   43.19%   43.19%           
=======================================
  Files         767      767           
  Lines       94914    94931   +17     
=======================================
+ Hits        40998    41007    +9     
- Misses      51052    51059    +7     
- Partials     2864     2865    +1     
Files with missing lines Coverage Δ
...ollers/awsprivatelink/awsprivatelink_controller.go 40.69% <0.00%> (-0.96%) ⬇️
support/awsutil/sg.go 10.06% <76.31%> (+10.06%) ⬆️
...ostedcontrolplane/hostedcontrolplane_controller.go 45.64% <0.00%> (-0.10%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.