-
Notifications
You must be signed in to change notification settings - Fork 553
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
rptest: add ability to restart pods in RedpandaServiceCloud
#16319
Conversation
delete_cmd = ['delete', 'pod', pod_name, '-n=redpanda'] | ||
self.logger.info( | ||
f'deleting pod {pod_name} so the cluster can recreate it') | ||
# kubectl delete pod rp-clo88krkqkrfamptsst0-0 -n=redpanda' |
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.
from docs, this sends a SIGTERM
example test run in description, ready for review |
@@ -733,7 +733,7 @@ def test_ht003_kgofailure(self): | |||
timeout_sec=self.msg_timeout) | |||
|
|||
# Run a rolling restart. | |||
self.stage_rolling_restart() |
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.
test_ht003_kgofailure()
is the only method that uses stage_rolling_restart()
. so, i've replaced stage_rolling_restart()
with the service level rolling_restart_pods()
b403b78
to
608bb96
Compare
putting this into draft to resolve merge conflicts |
d7c3e47
to
5224800
Compare
addressed merge conflicts from PR #16232. re-ran tests, stuff still works. ready for review |
new failures in https://buildkite.com/redpanda/redpanda/builds/44492#018d5c25-3521-4364-831e-6dfa8ee0484d:
|
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/44492#018d5c25-3524-49e3-b615-b656ac40f13f |
5224800
to
2060f16
Compare
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.
LGTM
Fixes https://github.com/redpanda-data/core-internal/issues/1001
Add
RedpandaServiceCloud.restart_pod()
andRedpandaServiceCloud.rolling_restart_pods()
to aid in tests that need to simulate a graceful restart of these k8s resources.Implemented using the
kubectl delete pod
command to allow k8s to gracefully manage the termination process.Also added a simple self test of these 2 methods in
redpanda_cloud_tests/rolling_restart_test.py
to the cloud test suite.verified with a
tier-3-aws-v2-arm
cluster:output:
Backports Required
Release Notes