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.

Sign up for GitHub

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

Jump to bottom

Use a longer driver timeout for drain #453

Closed
emerkle826 opened this issue Mar 5, 2024 · 0 comments · Fixed by #454
Closed

Use a longer driver timeout for drain #453

emerkle826 opened this issue Mar 5, 2024 · 0 comments · Fixed by #454
Assignees
@emerkle826
Labels
done Issues in the state 'done'

Comments

@emerkle826
Copy link
Contributor

In some scenarios, the drain operation can take longer than 30 seconds (the configured driver requesttimetout). Ideally, the call to drain a node would be asynchronous and a new endpoint could be added to poll the status of the drain. However, it would maybe be beneficial to simply adding a "slow" driver execution profile for this, and change the implementation of drain to make a CQL query with the slow profile enabled (see the Java driver docs for more details on execution profiles).

This "slow" execution profile could also be used for other Management API functions that could take longer than 30 seconds to complete under the hood, with the benefit of being able to make the change to management API and not having to make changes on down-stream projects (like cass-operator).
@emerkle826 emerkle826 mentioned this issue Mar 11, 2024
Merged
@emerkle826 emerkle826 moved this to Ready For Review in K8ssandra Mar 13, 2024
@adejanovski adejanovski added the ready-for-review Issues in the state 'ready-for-review' label Mar 13, 2024
@emerkle826 emerkle826 self-assigned this Mar 13, 2024
@burmanm burmanm moved this from Ready For Review to Review in K8ssandra Mar 14, 2024
@adejanovski adejanovski added review Issues in the state 'review' and removed ready-for-review Issues in the state 'ready-for-review' labels Mar 14, 2024
@github-project-automation github-project-automation bot moved this from Review to Done in K8ssandra Mar 20, 2024
@adejanovski adejanovski added done Issues in the state 'done' and removed review Issues in the state 'review' labels Mar 20, 2024
emerkle826 added a commit that referenced this issue Mar 21, 2024
@emerkle826
Release v0.1.74
8dec53e 
* [FEATURE] [#453](#453) Use a longer driver timeout for drain
* [FEATURE] [#455](#455) Add DSE 6.8.43 to the build matrix
* [FEATURE] [#458](#458) Update MCAC to v0.3.5
* [ENHANCEMENT] [#441](#441) Replace ShellUtils arbitrary command execution
alaye-ms added a commit to microsoft/management-api-for-apache-cassandra that referenced this issue Apr 1, 2024
@alaye-ms
Merge tag 'v0.1.74' into users/alaye/upstreamPlus5.0Fix
c398809 
Release v0.1.74

* [FEATURE] [k8ssandra#453](k8ssandra#453) Use a longer driver timeout for drain
* [FEATURE] [k8ssandra#455](k8ssandra#455) Add DSE 6.8.43 to the build matrix
* [FEATURE] [k8ssandra#458](k8ssandra#458) Update MCAC to v0.3.5
* [ENHANCEMENT] [k8ssandra#441](k8ssandra#441) Replace ShellUtils arbitrary command execution
alaye-ms added a commit to microsoft/management-api-for-apache-cassandra that referenced this issue Apr 2, 2024
@alaye-ms
Merge tag 'v0.1.74' into users/alaye/upstreamPlus5.0Fix
9d5610b 
Release v0.1.74

* [FEATURE] [k8ssandra#453](k8ssandra#453) Use a longer driver timeout for drain
* [FEATURE] [k8ssandra#455](k8ssandra#455) Add DSE 6.8.43 to the build matrix
* [FEATURE] [k8ssandra#458](k8ssandra#458) Update MCAC to v0.3.5
* [ENHANCEMENT] [k8ssandra#441](k8ssandra#441) Replace ShellUtils arbitrary command execution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emerkle826 emerkle826

Labels
done Issues in the state 'done'
Projects
No open projects
K8ssandra
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use a longer driver timeout for drain k8ssandra/management-api-for-apache-cassandra
2 participants
@emerkle826 @adejanovski