Skip to content

Add FIPS support with Go native crypto and unique operator labels - #317

Open
JasonPowr wants to merge 1 commit into
mainfrom
fips-work
Open

Add FIPS support with Go native crypto and unique operator labels#317
JasonPowr wants to merge 1 commit into
mainfrom
fips-work

Conversation

@JasonPowr

@JasonPowr JasonPowr commented Aug 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Switch the admission-webhook-controller binary from OpenSSL-based FIPS (CGO_ENABLED=1, GOEXPERIMENT=strictfipsruntime) to Go 1.24+ native FIPS (GOFIPS140=v1.0.0, CGO_ENABLED=0, -tags no_openssl), and set GODEBUG=fips140=only on FIPS-enabled clusters at runtime
  • Replace the default Operator SDK control-plane: controller-manager label with control-plane: policy-controller-operator across all manifests to prevent webhook/metrics routing collisions with other operators in the same namespace
  • Add a dedicated FIPS e2e test suite (test/e2e_fips/) that verifies kernel FIPS mode, GODEBUG env propagation, and the GOFIPS140 build marker in running binaries
  • Integrate FIPS detection into the existing e2e and upgrade suites so they automatically configure GODEBUG=fips140=only when running on FIPS clusters
  • Mark the operator bundle as fips-compliant: "true" and enable the fips-check Tekton parameter in CI pipelines
  • Fix flaky e2e test failure cause

@codecov-commenter

codecov-commenter commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 18.33%. Comparing base (d3c42ca) to head (9c36079).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #317   +/-   ##
=======================================
  Coverage   18.33%   18.33%           
=======================================
  Files           3        3           
  Lines          60       60           
=======================================
  Hits           11       11           
  Misses         49       49           
Flag Coverage Δ
unit 18.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@JasonPowr
JasonPowr force-pushed the fips-work branch 4 times, most recently from a4fc6de to 0a07010 Compare August 13, 2026 14:57
@JasonPowr

Copy link
Copy Markdown
Member Author

/retest

@JasonPowr
JasonPowr force-pushed the fips-work branch 4 times, most recently from eb42fe9 to 111a966 Compare August 14, 2026 09:42
@JasonPowr JasonPowr changed the title feat: support fips mode, add fips e2e Add FIPS support with Go native crypto and unique operator labels Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants