Skip to content

[ST] Fix FIPS check in case that ConfigMap is missing#12546

Open
im-konge wants to merge 1 commit intostrimzi:mainfrom
im-konge:fix-fips-check
Open

[ST] Fix FIPS check in case that ConfigMap is missing#12546
im-konge wants to merge 1 commit intostrimzi:mainfrom
im-konge:fix-fips-check

Conversation

@im-konge
Copy link
Member

Type of change

  • Test fix

Description

This small PR fixes the check if the cluster where we are running our STs is FIPS - in case that the ConfigMap is missing, the check will throw NPE.
The fix should ensure that the false is returned in case that the ConfigMap is missing, otherwise the data are taking from CM.

Checklist

  • Make sure all tests pass

Signed-off-by: Lukas Kral <lukywill16@gmail.com>
@im-konge im-konge self-assigned this Mar 20, 2026
@im-konge im-konge added this to the 1.0.0 milestone Mar 20, 2026
@im-konge im-konge requested a review from a team March 20, 2026 16:49
@im-konge
Copy link
Member Author

/gha run pipeline=acceptance

@github-actions
Copy link

github-actions bot commented Mar 20, 2026

⏳ System test verification started: link

The following 2 job(s) will be executed:

  • acceptance-amd64 (oracle-vm-4cpu-16gb-x86-64)
  • acceptance-arm64 (oracle-vm-4cpu-16gb-arm64)

Tests will start after successful build completion.

@codecov
Copy link

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.98%. Comparing base (eccdd77) to head (8131608).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #12546      +/-   ##
============================================
- Coverage     75.00%   74.98%   -0.03%     
+ Complexity     6435     6430       -5     
============================================
  Files           373      373              
  Lines         24881    24881              
  Branches       3205     3205              
============================================
- Hits          18663    18656       -7     
- Misses         4903     4907       +4     
- Partials       1315     1318       +3     

see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

🎉 System test verification passed: link

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.

1 participant