Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 18, 2025

Proposed commit message

The aerospike container used for testing was failing with:

  CRITICAL (config): (cfg.c:4387) 1024 system file descriptors not
  enough, config specified 15000

This commit fixes it by setting the ulimit to 20000

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. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

## Disruptive User Impact
## Author's Checklist

How to test this PR locally

Run the tests

cd metricbeat/module/aerospike
go test -tags=integration -v -count=1 ./...

Or, try running the container:

docker compose up

Related issues

## Use cases
## Screenshots
## Logs


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

The aerospike container used for testing was failing with:

  CRITICAL (config): (cfg.c:4387) 1024 system file descriptors not
  enough, config specified 15000

This commit fixes it by setting the ulimit to 20000

(cherry picked from commit 3d17cad)

# Conflicts:
#	metricbeat/module/aerospike/docker-compose.yml
@mergify mergify bot added backport conflicts There is a conflict in the backported pull request labels Nov 18, 2025
@mergify mergify bot requested a review from a team as a code owner November 18, 2025 09:03
@mergify mergify bot added backport conflicts There is a conflict in the backported pull request labels Nov 18, 2025
@mergify
Copy link
Contributor Author

mergify bot commented Nov 18, 2025

Cherry-pick of 3d17cad has failed:

On branch mergify/bp/7.17/pr-47682
Your branch is up to date with 'origin/7.17'.

You are currently cherry-picking commit 3d17cad1a.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   metricbeat/module/aerospike/docker-compose.yml

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Nov 18, 2025
@botelastic
Copy link

botelastic bot commented Nov 18, 2025

This pull request doesn't have a Team:<team> label.

@belimawr
Copy link
Contributor

Interestingly this is passing when I run locally 🤔

@belimawr
Copy link
Contributor

For some odd reason the test seems to be failing to find the aerospike docker image, but it works locally. I'll try re-running it.

@belimawr belimawr merged commit 65cae80 into 7.17 Nov 19, 2025
26 checks passed
@belimawr belimawr deleted the mergify/bp/7.17/pr-47682 branch November 19, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport conflicts There is a conflict in the backported pull request needs_team Indicates that the issue/PR needs a Team:* label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants