-
Notifications
You must be signed in to change notification settings - Fork 96
Add longer delay after kill command #1533
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
Add longer delay after kill command #1533
Conversation
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
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.
[Container] 2023/03/07 15:20:44 Running command if $(grep -q -- "--- FAIL:" logs/*.log); then echo "TESTS FAILED SEE THESE LOGS:"; echo ; grep -l -- "--- FAIL:" logs/*.log; export fail_test=true; fi
TESTS FAILED SEE THESE LOGS:
logs/test_operator_ha_upgrades.log
Failure:
• [FAILED] [881.392 seconds]
Operator HA Upgrades Upgrading a multi-DC cluster, with a random pod deleted during the staging phase [It] Upgrade, with a random pod deleted during the staging phase, from 6.3.25 to 7.1.27
/codebuild/output/src356759027/src/github.com/FoundationDB/fdb-kubernetes-operator/fdb-kubernetes-tests/tests/fixtures/command_line_flags.go:298
[FAILED] Timed out after 600.001s.
Expected
<bool>: false
to be true
In [It] at: /codebuild/output/src356759027/src/github.com/FoundationDB/fdb-kubernetes-operator/fdb-kubernetes-tests/tests/test_operator_ha_upgrades/operator_ha_upgrade_test.go:317 @ 03/07/23 14:37:51.792
------------------------------
48548fd
to
c7f4bfa
Compare
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Had one failure in the operator test for converting the public IP source. I noticed that the exclude was blocked by |
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Description
Adding back the
status
request when killing fdbserver processes to ensure the reboot request was sent to all processes.Type of change
Please select one of the options below.
Discussion
We had this check in an earlier version of the fdbclient but then removed it. This change adds it back again to ensure the reboot command is sent to all processes.
Testing
e2e test.
Documentation
Follow-up