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

pool:fix outofspace error sending pool into diasbled mode #7753

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

mksahakyan
Copy link
Contributor

@mksahakyan mksahakyan commented Feb 19, 2025

Motivation

we are seeing the this error wen there is no space on the pool left, and pool goes into disabled mode,

AB56598C55] WRITE failed : IOError
AB56598C55] Transfer failed due to a disk error: CacheException(rc=204;msg=Disk I/O Error )
AB56598C55] Pool mode changed to disabled(fetch,store,stage,p2p-client,p2p-server): Pool disabled: Disk I/O Error
AB56598C55] Pool: dcache-xfel487-01, fault occurred in transfer: Disk I/O Error . Pool disabled: , cause: CacheException(rc=204;msg=Disk I/O Error )

however we want it to go into READ-ONLY mode and files could stay accesable.

the privous patch (https://rb.dcache.org/r/14357/diff/3/#index_header) did not fix the issue.

so the new changes are still a try to fix the issue.

Acked-by: Dmitry Litvintsev
Target: master, 10.2
Require-book: no
Require-notes: yes
Patch: https://rb.dcache.org/r/14368/

Motivation

we are seeing the this error wen there is no space on the pool left, and pool goes into disabled mode,

AB56598C55] WRITE failed : IOError
AB56598C55] Transfer failed due to a disk error: CacheException(rc=204;msg=Disk I/O Error )
AB56598C55] Pool mode changed to disabled(fetch,store,stage,p2p-client,p2p-server): Pool disabled: Disk I/O Error
AB56598C55] Pool: dcache-xfel487-01, fault occurred in transfer: Disk I/O Error . Pool disabled: , cause: CacheException(rc=204;msg=Disk I/O Error )

however we want it to go into READ-ONLY mode and files could stay accesable.

the privous patch (https://rb.dcache.org/r/14357/diff/3/#index_header) did not fix the issue.

so the new changes are still a try to fix the issue.

Acked-by: Dmitry Litvintsev
Target: master. 10.2, 10.1, 10.0, 9.2
Require-book: no
Require-notes: yes
Patch: https://rb.dcache.org/r/14368/
Copy link
Contributor

@khys95 khys95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not a fan of leaving TODOs .. I'm not sure about standard procedure on the team.
If none, I'd ask for them to be removed

Copy link
Contributor

@khys95 khys95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some can be turned into comments (so just remove the TODO?)

Copy link
Contributor

@khys95 khys95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update: this PR to 10.2 will be merged as is (since it's already on master)

@khys95 khys95 merged commit 18601a9 into dCache:10.2 Feb 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants