Skip to content

Conversation

ycombinator
Copy link
Contributor

What does this PR do?

This PR restores the check for the ms_tls13kdf Go build tag on the packaged FIPS-capable elastic-agent binary and component binaries. However, it only checks for this build tag if the binary was built with Go 1.24.x, which is where this tag was supported and required to ensure FIPS compliance.

Why is it important?

To ensure FIPS compliance of binaries built with Go 1.24.x.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • 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/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

None; this is a change to a Go test that gets executed when Elastic Agent is packaged.

How to test this PR locally

Create a FIPS-capable Elastic Agent package. Make sure a Go version other than 1.24.x is being used.

cat .go-version
FIPS=true SNAPSHOT=true EXTERNAL=true PLATFORMS="linux/arm64" PACKAGES="tar.gz" mage -v package

Make sure the package builds without any errors.

@ycombinator ycombinator requested a review from a team as a code owner September 25, 2025 21:19
@ycombinator ycombinator added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team backport-8.19 Automated backport to the 8.19 branch backport-9.1 Automated backport to the 9.1 branch labels Sep 25, 2025
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

mergify bot commented Sep 25, 2025

This pull request does not have a backport label. Could you fix it @ycombinator? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@ycombinator ycombinator changed the title Conditionall check for ms_tls13kdf build tag Conditionall check for ms_tls13kdf build tag Sep 25, 2025
@ycombinator ycombinator changed the title Conditionall check for ms_tls13kdf build tag Conditionally check for ms_tls13kdf build tag Sep 25, 2025
Copy link

@elasticmachine
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

cc @ycombinator

@ycombinator ycombinator enabled auto-merge (squash) September 26, 2025 04:03
@ycombinator ycombinator merged commit aae77c4 into elastic:main Sep 26, 2025
28 of 32 checks passed
mergify bot pushed a commit that referenced this pull request Sep 26, 2025
mergify bot pushed a commit that referenced this pull request Sep 26, 2025
@ycombinator ycombinator deleted the package-test-fips-conditional-go-version branch September 26, 2025 10:16
ycombinator added a commit that referenced this pull request Sep 26, 2025
(cherry picked from commit aae77c4)

Co-authored-by: Shaunak Kashyap <[email protected]>
ycombinator added a commit that referenced this pull request Sep 26, 2025
(cherry picked from commit aae77c4)

Co-authored-by: Shaunak Kashyap <[email protected]>
v1v added a commit that referenced this pull request Sep 26, 2025
* upstream: (505 commits)
  Update journald tests now that Filebeat supports watching folders (#10131)
  [deploy/kubernetes]: add info about hostPID for Universal Profiling (#10173)
  Fall back to process runtime if otel runtime is unsupported (#10087)
  Conditionall check for ms_tls13kdf build tag (#10160)
  [docs][edot] add entry for profiles (#10163)
  edot/docs: add support for profiles (#10146)
  Add Logstash exporter (#10137)
  Add back publish to serverless. (#10159)
  Improve Integration test documentation (#10155)
  Fix multiarch service image push from main to serverless (#10129)
  Forward migrate action to endpoint (#9801)
  Comment out check for ms_tls13kdf tag for FIPS-capable binaries (#10148)
  [otel] add receivers: apache, iis, mysql, postgresql, sqlserver v0.135.0 (#9344)
  Add k8sevents receiver in kube-stack (#10086)
  feat: emit system resource metrics for EDOT subprocess (#10003)
  [AutoOps] Configure OTel Exporter to Send Maximum-sized Batches (#10126)
  keep enrollment token when replacing data with signed (#10115)
  Revert "Publish `elastic-agent-service` container directly to serverless from main (#9583)" (#10127)
  Add agent_policy_id and policy_revision_idx to checkin requests (#9931)
  remove resource/k8s processor and use k8sattributes processor for service attributes (#10108)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.19 Automated backport to the 8.19 branch backport-9.1 Automated backport to the 9.1 branch skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants