Skip to content

Conversation

@andriisoldatenko
Copy link
Contributor

@andriisoldatenko andriisoldatenko commented Nov 27, 2025

Description

This PR resolves https://dt-rnd.atlassian.net/browse/DAQ-11573

Note

- fixed how we run e2e tests using ocp fips cluster and fips image (we broke it for a while and run normal image using FIPS cluster) fixed by @avorima in another PR

  • Added log line FIPS 140-2 Mode Enabled: true/false when we start operator/webhook to easily find which one we use currently

Warning

In testing phase, potentially postponed until Go 1.26 release.

How can this be tested?

run e2e tests with/ using FIPS and non FIPS image.

TAG=snapshot-build-operator-fips-fips make test/e2e/standard

@andriisoldatenko andriisoldatenko changed the title add fips enabled to logs Build operator image from single source of truth using FIPS-140-3 Nov 27, 2025
Copy link
Contributor

@avorima avorima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can consolidate a bit here. Special make targets for fips can be removed in favor of making the mode fully dependent on the env/args. The same goes for workflows. fips-ci and fips-release are very similar to their non-fips counterparts

@andriisoldatenko
Copy link
Contributor Author

I think we can consolidate a bit here. Special make targets for fips can be removed in favor of making the mode fully dependent on the env/args. The same goes for workflows. fips-ci and fips-release are very similar to their non-fips counterparts

potentially yes, but it requires more work and testing, okey let me do it :)

@andriisoldatenko andriisoldatenko force-pushed the build-operator-fips branch 2 times, most recently from 9a1e1c2 to ded6429 Compare December 9, 2025 12:38
@andriisoldatenko andriisoldatenko changed the title Build operator image from single source of truth using FIPS-140-3 [WIP] Build operator image from single source of truth using FIPS-140-3 Dec 10, 2025
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.

5 participants