TrueCloudLab/frostfs-node

Fork 27

Add evacuate cancellation support #299

New issue

Closed

opened 2023-05-02 09:56:40 +00:00 by dstepanov-yadro · 0 comments

dstepanov-yadro commented

2023-05-02 09:56:40 +00:00

Member

EvacuateShard uses context.Context only for tracing. If ctx cancelled, the execution of the command should be canceled.

Describe the solution you'd like

Add ctx.Done() checks.

## Is your feature request related to a problem? Please describe. `EvacuateShard` uses context.Context only for tracing. If ctx cancelled, the execution of the command should be canceled. ## Describe the solution you'd like Add ctx.Done() checks.

dstepanov-yadro added the

frostfs-node

good first issue

triage

labels 2023-05-02 09:56:40 +00:00

fyrchik added this to the v0.37.0 milestone 2023-05-02 09:57:22 +00:00

dstepanov-yadro referenced this issue from a pull request that will close it,

2023-05-02 13:31:13 +00:00

Add context.Done checks to evacuate shard #301

fyrchik closed this issue

2023-05-02 13:31:53 +00:00

fyrchik referenced this issue from a commit

2023-05-02 13:31:54 +00:00

[#299] ir: Drop unused structs

fyrchik referenced this issue from a commit

2023-05-02 13:31:54 +00:00

[#299] evacuate: Add context cancel checks

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#299

No description provided.