Skip to content

split FIPS unit tests from fips140=only test#4766

Merged
michel-laterman merged 2 commits intoelastic:mainfrom
michel-laterman:split-fips-test-pipelines
Apr 4, 2025
Merged

split FIPS unit tests from fips140=only test#4766
michel-laterman merged 2 commits intoelastic:mainfrom
michel-laterman:split-fips-test-pipelines

Conversation

@michel-laterman
Copy link
Copy Markdown
Contributor

What is the problem this PR solves?

Split the single "Run FIPS unit tests" in the buildkite pipeline into two steps,

  • "Run FIPS unit tests" - runs unit tests with -tags=requirefips
  • "Run fips140=only unit tests" - runs unit tests with -tags=requirefips and GODEBUG=fips140=only

This way it is easier to determine what causes a failure with FIPS functionality.

This also makes fleet-server consistent with the testing approach being implemented in elastic-agent and elastic-agent-libs.

@michel-laterman michel-laterman added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team tech debt backport-8.x Automated backport to the 8.x branch with mergify labels Apr 3, 2025
@michel-laterman michel-laterman requested a review from a team as a code owner April 3, 2025 19:54
@elastic-sonarqube
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Copy Markdown
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

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

Looks good.

@michel-laterman michel-laterman merged commit 9f65d34 into elastic:main Apr 4, 2025
9 checks passed
@michel-laterman michel-laterman deleted the split-fips-test-pipelines branch April 4, 2025 20:56
mergify bot pushed a commit that referenced this pull request Apr 4, 2025
split FIPS unit tests from fips140=only test

(cherry picked from commit 9f65d34)
michel-laterman added a commit that referenced this pull request Apr 9, 2025
split FIPS unit tests from fips140=only test

(cherry picked from commit 9f65d34)

Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-8.x Automated backport to the 8.x branch with mergify Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants