-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add Operator FIPS Compliance checks as required tasks #96
base: main
Are you sure you want to change the base?
feat: add Operator FIPS Compliance checks as required tasks #96
Conversation
To be merged after konflux-ci/build-definitions#1741. thank you! :) |
@arewm @simonbaird FYI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason why the effective on date is set for January 24th? That will be less than a month for users to react to the change (assuming that many will not be back until January).
This needs to be preceded by a mass update of users' .tekton/ files in git by sending PRs and MRs to add the task. |
38e70e2
to
a71be1a
Compare
CVP-4373. This commit adds fbc-fips-check as a required task to the FBC pipeline and fips-operator-bundle-check as a required task in the container pipeline. The commit also adds their repective TA versions. Signed-off-by: Yashvardhan Nanavati <[email protected]>
a71be1a
to
1b329ec
Compare
The date has been revised to March 1, 2025. |
with fips-operator-bundle-check and fips-operator-bundle-check-ta being added as required tasks, they may fail even if the operator bundle is FIPS compliant for a specific OCP version. This is because on a bundle level, there's no way to determine the target OCP version for it and hence, we cannot apply the appropriate exception list during the check. This can only be determined when dealing with an FBC fragment. Hence, this commit adds the operator bundle check as informative. Signed-off-by: Yashvardhan Nanavati <[email protected]>
I was thinking this one (and #95) could be merged before that, and then the mass update, which would need to be done in advance of the effective on date, i.e. March 1st. See also EC-1063 and CVP-4376. |
CVP-4373. This commit adds fbc-fips-check as a required task to the FBC pipeline and fips-operator-bundle-check as a required task in the container pipeline. The commit also adds their repective TA versions.