Skip to content
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

Add go1.24 GODEBUG=fips140=only test #4581

Merged
merged 2 commits into from
Mar 19, 2025

Conversation

michel-laterman
Copy link
Contributor

What is the problem this PR solves?

Validate FIPS compliance by using go 1.24's fips features.

How does this PR solve the problem?

Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

Note that this target assumes golang/go and NOT microsoft/go.
In order to use microsoft/go to run these test we would need to install in the environment, and have a FIPS enabled OpenSSL on the system.

How to test this PR locally

FIPS=true make test-unit-fips

Design Checklist

  • I have ensured my design is stateless and will work when multiple fleet-server instances are behind a load balancer.
  • I have or intend to scale test my changes, ensuring it will work reliably with 100K+ agents connected.
  • I have included fail safe mechanisms to limit the load on fleet-server: rate limiting, circuit breakers, caching, load shedding, etc.

Checklist

  • 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

@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 backport-9.0 Automated backport to the 9.0 branch labels Mar 13, 2025
Copy link

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

@michel-laterman michel-laterman marked this pull request as ready for review March 18, 2025 19:48
@michel-laterman michel-laterman requested a review from a team as a code owner March 18, 2025 19:48
@michel-laterman michel-laterman merged commit 8e304bd into elastic:main Mar 19, 2025
9 checks passed
mergify bot pushed a commit that referenced this pull request Mar 19, 2025
Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

(cherry picked from commit 8e304bd)
mergify bot pushed a commit that referenced this pull request Mar 19, 2025
Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

(cherry picked from commit 8e304bd)
michel-laterman added a commit that referenced this pull request Mar 19, 2025
Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

(cherry picked from commit 8e304bd)

Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
Co-authored-by: michel-laterman <michel.laterman@elastic.co>
michel-laterman added a commit that referenced this pull request Mar 20, 2025
* Add go1.24 GODEBUG=fips140=only test (#4581)

Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

(cherry picked from commit 8e304bd)

* Split 8.x TLS min version tests for FIPS

* Fix ES Output TLS tests

---------

Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
Co-authored-by: michel-laterman <michel.laterman@elastic.co>
andrzej-stencel added a commit that referenced this pull request Mar 24, 2025
* [updatecli] update elastic stack version for testing 9.1.0-80bdca58-SNAPSHOT (#4569)

* chore: Update snapshot.yml

Made with ❤️️ by updatecli

* chore: Update snapshot.yml

Made with ❤️️ by updatecli

* chore: Update snapshot.yml

Made with ❤️️ by updatecli

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* build(deps): bump github.com/elastic/elastic-agent-libs from 0.18.8 to 0.18.10 (#4565)

* build(deps): bump github.com/elastic/elastic-agent-libs

Bumps [github.com/elastic/elastic-agent-libs](https://github.com/elastic/elastic-agent-libs) from 0.18.8 to 0.18.10.
- [Release notes](https://github.com/elastic/elastic-agent-libs/releases)
- [Commits](elastic/elastic-agent-libs@v0.18.8...v0.18.10)

---
updated-dependencies:
- dependency-name: github.com/elastic/elastic-agent-libs
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Post dependabot file modifications

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>

* chore: Update snapshot.yml (#4588)

Made with ❤️️ by updatecli

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* build(deps): bump github.com/elastic/elastic-agent-libs from 0.18.10 to 0.18.11 (#4591)

* build(deps): bump github.com/elastic/elastic-agent-libs

Bumps [github.com/elastic/elastic-agent-libs](https://github.com/elastic/elastic-agent-libs) from 0.18.10 to 0.18.11.
- [Release notes](https://github.com/elastic/elastic-agent-libs/releases)
- [Commits](elastic/elastic-agent-libs@v0.18.10...v0.18.11)

---
updated-dependencies:
- dependency-name: github.com/elastic/elastic-agent-libs
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Post dependabot file modifications

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>

* chore: Update snapshot.yml (#4596)

Made with ❤️️ by updatecli

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* build(deps): bump the elastic-apm group with 6 updates (#4590)

* build(deps): bump the elastic-apm group with 6 updates

Bumps the elastic-apm group with 6 updates:

| Package | From | To |
| --- | --- | --- |
| [go.elastic.co/apm/module/apmchiv5/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |
| [go.elastic.co/apm/module/apmelasticsearch/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |
| [go.elastic.co/apm/module/apmhttp/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |
| [go.elastic.co/apm/module/apmprometheus/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |
| [go.elastic.co/apm/module/apmzerolog/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |
| [go.elastic.co/apm/v2](https://github.com/elastic/apm-agent-go) | `2.6.3` | `2.7.0` |


Updates `go.elastic.co/apm/module/apmchiv5/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

Updates `go.elastic.co/apm/module/apmelasticsearch/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

Updates `go.elastic.co/apm/module/apmhttp/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

Updates `go.elastic.co/apm/module/apmprometheus/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

Updates `go.elastic.co/apm/module/apmzerolog/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

Updates `go.elastic.co/apm/v2` from 2.6.3 to 2.7.0
- [Release notes](https://github.com/elastic/apm-agent-go/releases)
- [Changelog](https://github.com/elastic/apm-agent-go/blob/v2.7.0/CHANGELOG.asciidoc)
- [Commits](elastic/apm-agent-go@v2.6.3...v2.7.0)

---
updated-dependencies:
- dependency-name: go.elastic.co/apm/module/apmchiv5/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
- dependency-name: go.elastic.co/apm/module/apmelasticsearch/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
- dependency-name: go.elastic.co/apm/module/apmhttp/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
- dependency-name: go.elastic.co/apm/module/apmprometheus/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
- dependency-name: go.elastic.co/apm/module/apmzerolog/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
- dependency-name: go.elastic.co/apm/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: elastic-apm
...

Signed-off-by: dependabot[bot] <support@github.com>

* Post dependabot file modifications

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>

* Remove WithUsrPwd (#4600)

* build(deps): bump github.com/containerd/containerd in /testing (#4599)

Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.15 to 1.7.27.
- [Release notes](https://github.com/containerd/containerd/releases)
- [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md)
- [Commits](containerd/containerd@v1.7.15...v1.7.27)

---
updated-dependencies:
- dependency-name: github.com/containerd/containerd
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add go1.24 GODEBUG=fips140=only test (#4581)

Add a new target that uses GODEBUG=fips140=only to run unit tests.
This target should use the FIPS=true flag in order to pass -tags=requirefips.

* [Automation] Bump Golang version to 1.24.1 (#4611)

* chore: Update .go-version

Made with ❤️️ by updatecli

* chore: Update .golangci.yml

Made with ❤️️ by updatecli

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update golangci-lint.yml (#4615)

Pinning action to a full length commit SHA [see](https://docs.github.com/en/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions#using-third-party-actions)

* build(deps): bump github.com/elastic/elastic-agent-system-metrics from 0.11.9 to 0.11.10 (#4621)

* build(deps): bump github.com/elastic/elastic-agent-system-metrics

Bumps [github.com/elastic/elastic-agent-system-metrics](https://github.com/elastic/elastic-agent-system-metrics) from 0.11.9 to 0.11.10.
- [Release notes](https://github.com/elastic/elastic-agent-system-metrics/releases)
- [Commits](elastic/elastic-agent-system-metrics@v0.11.9...v0.11.10)

---
updated-dependencies:
- dependency-name: github.com/elastic/elastic-agent-system-metrics
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Post dependabot file modifications

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Co-authored-by: Blake Rouse <blake.rouse@elastic.co>
Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com>
Co-authored-by: Paul McCann <paul.mccann@elastic.co>
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 backport-9.0 Automated backport to the 9.0 branch 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.

2 participants