node: Stop GC once termination signal recieved #366

Closed
opened 2023-05-18 08:12:23 +00:00 by acid-ant · 0 comments
Member

https://git.frostfs.info/TrueCloudLab/frostfs-node/src/branch/master/pkg/local_object_storage/shard/gc.go#L199

Expected Behavior
GC stops once termination signal received.

Current Behavior
GC continue working on even we stop it.
Also, it is possible that timer continues working. See tickRemover():
https://git.frostfs.info/TrueCloudLab/frostfs-node/src/branch/master/pkg/local_object_storage/shard/gc.go#L163

Possible Solution
Use main context in Shard.removeGarbage()

Node version:

service@az:~$ frostfs-node --version
FrostFS Storage node
Version: v0.0.1-414-gc60029d3
GoVersion: go1.18.4

Routines list before node killing by system in attachment.

https://git.frostfs.info/TrueCloudLab/frostfs-node/src/branch/master/pkg/local_object_storage/shard/gc.go#L199 **Expected Behavior** GC stops once termination signal received. **Current Behavior** GC continue working on even we stop it. Also, it is possible that timer continues working. See tickRemover(): https://git.frostfs.info/TrueCloudLab/frostfs-node/src/branch/master/pkg/local_object_storage/shard/gc.go#L163 **Possible Solution** Use main context in Shard.removeGarbage() Node version: ``` service@az:~$ frostfs-node --version FrostFS Storage node Version: v0.0.1-414-gc60029d3 GoVersion: go1.18.4 ``` Routines list before node killing by system in attachment.
acid-ant added the
bug
triage
labels 2023-05-18 08:12:23 +00:00
fyrchik added the
frostfs-node
label 2023-05-18 08:20:33 +00:00
fyrchik added this to the v0.37.0 milestone 2023-05-18 08:27:36 +00:00
acid-ant self-assigned this 2023-05-18 08:36:26 +00:00
Sign in to join this conversation.
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#366
No description provided.