-
Notifications
You must be signed in to change notification settings - Fork 577
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
rpk tune ducktape tests #5407
rpk tune ducktape tests #5407
Conversation
We want to make sure that the start/installation path that we guide the user to follow is covered in our CI, running rpk redpanda mode prod is a crucial step before running the tuners.
Tests will be only available in CDT since they rely on the environment and can't be run in a container.
This golden test will allow us to catch when a new tuner is either added or removed from production mode
Both errors related to: #4772 |
fstrim was disabled in production mode https://github.com/redpanda-data/redpanda/issues/3068 | ||
""" | ||
node = self.redpanda.nodes[0] | ||
rpk = RpkRemoteTool(self.redpanda, node) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these tests check rpk.tune_fstrim
, and below, rpk.transparent_hugepages
are false before swapping to true?
This will avoid the (not problematic, but redundant) scenario if THP or fstrim are set to true in mode prod: tune all will check these, so we'll be able to delete these tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(not sure how compelling this comment/question is so feel free to dismiss)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's true, we also catch this change in test_config_change_mode_prod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, I was thinking that if one of those is set to true, a person wont know to go into this file and remove these redundant tests
Cover letter
Added the following ducktape tests:
The rpk redpanda tune test will only run in Clustered Ducktape since they can't run in a normal containerized environment.
Tested in AWS:
Fixes of #4628
Release notes