Ensure all GitLab unit tests succeed in FIPS mode
Overview
As part of FIPS compliance, we would like to see all Unit and Integration tests executed and pass in FIPS mode.
More context can be found here: omnibus-gitlab#6845 (comment 968192969)
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Maintainer
Based on the comment here, @stanhu has already started working on this as part of !88470 (closed).
@stanhu I am assigning this issue to you but please feel free to reassign as you see fit.
- Vincy Wilson mentioned in issue omnibus-gitlab#6845 (closed)
mentioned in issue omnibus-gitlab#6845 (closed)
- Vincy Wilson assigned to @stanhu
assigned to @stanhu
- Author Maintainer
@cdu1 Could you please assign it to the right epic? I assume this would fit into Eng Catch all epic - &5104 (closed) but not 100% sure. Thank you!
Collapse replies @vincywilson Please check out this comment https://gitlab.com/groups/gitlab-org/-/epics/8182#note_972983788. This task can be postponed to FedRAMP and it's tracked by https://gitlab.com/gitlab-org/gitaly/-/issues/4269.
- Maintainer
@vincywilson, please can you add a type label to this issue to help with issue discovery in issue reports.
- 🤖 GitLab Bot 🤖 added auto updated label
added auto updated label
- Stan Hu mentioned in merge request !88469 (merged)
mentioned in merge request !88469 (merged)
- Stan Hu mentioned in merge request !91138 (merged)
mentioned in merge request !91138 (merged)
- Stan Hu mentioned in merge request !91302 (merged)
mentioned in merge request !91302 (merged)
- Stan Hu mentioned in merge request !91619 (merged)
mentioned in merge request !91619 (merged)
- Stan Hu mentioned in merge request !91625 (merged)
mentioned in merge request !91625 (merged)
- Stan Hu mentioned in merge request !91627 (merged)
mentioned in merge request !91627 (merged)
There's a lot of noise in the failures in !88470 (closed), but the test failure in https://gitlab.com/gitlab-org/gitlab/-/jobs/2669241719 revealed that there are errors whenever the MD5 fingerprint is needed but not present.
Here are a few fixes to address this:
- Stan Hu mentioned in merge request !92131 (merged)
mentioned in merge request !92131 (merged)
- Stan Hu mentioned in merge request !92803 (merged)
mentioned in merge request !92803 (merged)
- Stan Hu mentioned in merge request !92809 (merged)
mentioned in merge request !92809 (merged)
- Stan Hu mentioned in merge request !92816 (merged)
mentioned in merge request !92816 (merged)
- Stan Hu mentioned in commit shamvi050801/gitlab@1907c7bd
mentioned in commit shamvi050801/gitlab@1907c7bd
- Stan Hu closed
closed