Skip to content

fix: increase TiKV file descriptor limit to resolve failure in integration tests#2843

Merged
lumjjb merged 1 commit intoguacsec:mainfrom
gaganhr94:fix/file-descriptors-issue-in-ci-tests
Feb 9, 2026
Merged

fix: increase TiKV file descriptor limit to resolve failure in integration tests#2843
lumjjb merged 1 commit intoguacsec:mainfrom
gaganhr94:fix/file-descriptors-issue-in-ci-tests

Conversation

@gaganhr94
Copy link
Copy Markdown
Contributor

Description of the PR

Fixes #2841

TiKV requires atleast ~80000 file descriptors but was only getting the default 65,536, causing it to fail on startup with a FATAL error. This prevented the TiKV cluster from bootstrapping properly.

Currently the ci-integration workflow on this PR would fail because the test update in #2830 is not merged yet. Post merging that fix, we would get the error mentioned in #2841. Merging this PR would resolve that issue as well, and the test pipelines will run without any issues.

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If GraphQL schema is changed, GraphQL client updates/additions have been made
  • If OpenAPI spec is changed, make generate has been run
  • If ent schema is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

…ation tests

Signed-off-by: Gagan H R <hrgagan4@gmail.com>
@lumjjb lumjjb force-pushed the fix/file-descriptors-issue-in-ci-tests branch from 852b843 to 29e03a4 Compare February 9, 2026 15:45
@lumjjb
Copy link
Copy Markdown
Contributor

lumjjb commented Feb 9, 2026

Force merging due to having multiple CI issues

1 similar comment
@lumjjb
Copy link
Copy Markdown
Contributor

lumjjb commented Feb 9, 2026

Force merging due to having multiple CI issues

@lumjjb lumjjb merged commit b8663fd into guacsec:main Feb 9, 2026
8 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] error in ci-integration pipeline

3 participants