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

Backport of CSI: make plugin health_timeout configurable in csi_plugin stanza into release/1.1.x #13366

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #13340 to be assessed for backporting due to the inclusion of the label backport/1.1.x.

WARNING automatic cherry-pick of commits failed. Commits will require human attention.

The below text is copied from the body of the original PR.


Fixes #13179

Test results: Configured health_timeout as 5m and introduced an artificial error to make sure Portworx was killed after 5m:

Recent Events:
Time                  Type                     Description
2022-06-10T20:05:40Z  Killed                   Task successfully killed
2022-06-10T20:05:40Z  Terminated               Exit Code: 2, Exit Message: "Docker container exited with non-zero exit code: 2"
2022-06-10T20:05:40Z  Killing                  CSI plugin did not become healthy before configured 5m0s health timeout
2022-06-10T20:05:40Z  Plugin became unhealthy  Error: CSI plugin failed probe: timeout while connecting to gRPC socket: failed to stat socket: stat /etc/nomad.d/client/csi/plugins/62bdddd8-3955-92d2-cc23-02b6f6204e62/csi.sock: no such file or directory
2022-06-10T19:58:15Z  Started                  Task started by client
2022-06-10T19:58:13Z  Task Setup               Building Task Directory
2022-06-10T19:58:13Z  Received                 Task received by client

Signed-off-by: Grant Griffiths ggriffiths@purestorage.com

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/csi_healthtimeout_configurable/deeply-brief-mutt branch 2 times, most recently from 6cf39e7 to 7cde452 Compare June 14, 2022 18:16
…3340)

Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
@tgross tgross force-pushed the backport/csi_healthtimeout_configurable/deeply-brief-mutt branch from 7cde452 to d68ee22 Compare June 14, 2022 18:21
@tgross tgross merged commit 69a1ba9 into release/1.1.x Jun 14, 2022
@tgross tgross deleted the backport/csi_healthtimeout_configurable/deeply-brief-mutt branch June 14, 2022 18:30
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants