Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Mar 5, 2025

Proposed commit message

Enable FIPS compliant builds when the env var FIPS=true is set.

Artifacts are built with the microsfot/go toolchain with the env var GOEXPERIMENT=systemcrypto and the build tag "-tags=requirefips".
In order to run the resulting binary, the system must have a FIPS compliant crypto provider.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

None

How to test this PR locally

TODO


This is an automatic backport of pull request #42402 done by Mergify.

@mergify mergify bot requested a review from a team as a code owner March 5, 2025 16:14
@mergify mergify bot requested review from AndersonQ and khushijain21 and removed request for a team March 5, 2025 16:14
@mergify mergify bot added the backport label Mar 5, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Mar 5, 2025
@github-actions github-actions bot added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team labels Mar 5, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Mar 5, 2025
@michel-laterman
Copy link
Contributor

8.x is still on go v1.22.x
#43010 needs to be merged before this

@mergify
Copy link
Contributor Author

mergify bot commented Mar 10, 2025

This pull request has not been merged yet. Could you please review and merge it @michel-laterman? 🙏

@AndersonQ
Copy link
Member

I think the CI failures might be some flakiness. I'd suggest to update the branch and retry

Copy link
Member

@AndersonQ AndersonQ left a comment

Choose a reason for hiding this comment

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

As long as CI passes, it's good to go

Enable FIPS compliant builds when the env var FIPS=true is set.

Artifacts are built with the microsfot/go toolchain with the env var GOEXPERIMENT=systemcrypto and the build tag "-tags=requirefips".
In order to run the resulting binary, the system must have a FIPS compliant crypto provider.

(cherry picked from commit 83e0ec0)
@pchila pchila force-pushed the mergify/bp/8.x/pr-42402 branch from c891ac6 to 3291bd1 Compare March 14, 2025 07:37
@pchila pchila merged commit 79dd0e3 into 8.x Mar 14, 2025
142 of 145 checks passed
@pchila pchila deleted the mergify/bp/8.x/pr-42402 branch March 14, 2025 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants