Skip to content

[8.x](backport #42025) feat(libbeat): use math/rand/v2 and drop seed init#42441

Merged
rdner merged 1 commit into8.xfrom
mergify/bp/8.x/pr-42025
Feb 14, 2025
Merged

[8.x](backport #42025) feat(libbeat): use math/rand/v2 and drop seed init#42441
rdner merged 1 commit into8.xfrom
mergify/bp/8.x/pr-42025

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Jan 28, 2025

Proposed commit message

Since go 1.20 the rand package automatically seeds the global random number generator with a random value.

remove deprecate methods and use rand v2 api in libbeat

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

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs


This is an automatic backport of pull request #42025 done by [Mergify](https://mergify.com).

* feat(libbeat): use math/rand/v2 and drop seed init

Since go 1.20 the rand package automatically seeds
the global random number generator with a random value.

remove deprecate methods and use rand v2 api in libbeat

* ci: bump golangci-lint to fix errors

* ci(golangci): remove skip-go-installation option

it was removed in v4

* Update golangci-lint.yml

* Update golangci-lint.yml

* Update golangci-lint.yml

* lint: replace rand with t tempdir

* Update client_integration_test.go

* lint: fix linter issues

* lint: fix linter issues

* Apply suggestions from code review

Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>

---------

Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
(cherry picked from commit 12c36bd)
@mergify mergify bot added the backport label Jan 28, 2025
@mergify mergify bot requested a review from a team as a code owner January 28, 2025 02:14
@mergify mergify bot requested review from VihasMakwana and rdner and removed request for a team January 28, 2025 02:14
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 28, 2025
@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Feb 3, 2025

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

1 similar comment
@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Feb 10, 2025

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

@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label Feb 10, 2025
@elasticmachine
Copy link
Copy Markdown
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 Feb 10, 2025
@rdner rdner merged commit ca9dd09 into 8.x Feb 14, 2025
@rdner rdner deleted the mergify/bp/8.x/pr-42025 branch February 14, 2025 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 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.

4 participants