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

dt/crl: Increase inotify max_user_instances before testing CRL #20143

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

oleiman
Copy link
Member

@oleiman oleiman commented Jun 25, 2024

We run pretty close to the inotify instance limits when running tests with complicated tls configs. It's not exactly clear why the configs in the CRL tests blow these limits only occasionally, but it doesn't harm anything to increase the limit before running the tests, hopefully avoiding some flake.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.1.x
  • v23.3.x
  • v23.2.x

Release Notes

  • none

We run pretty close to the inotify instance limits when running tests
with complicated tls configs. It's not exactly clear why the configs
in the CRL tests blow these limits only occasionally, but it doesn't
harm anything to increase the limit before running the tests, hopefully
avoiding some flake.

Signed-off-by: Oren Leiman <[email protected]>
@oleiman oleiman self-assigned this Jun 25, 2024
@oleiman
Copy link
Member Author

oleiman commented Jun 25, 2024

/ci-repeat 5
dt-repeat=30
tests/rptest/tests/crl_test.py

@oleiman
Copy link
Member Author

oleiman commented Jun 25, 2024

/ci-repeat 1
dt-repeat=30
skip-units
skip-redpanda-build
tests/rptest/tests/crl_test.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant