Skip to content

Conversation

@simonswine
Copy link
Contributor

@simonswine simonswine commented Dec 10, 2025

This fixes log messages like:

[2025-12-10 12:35:15.014094 +00:00] ERROR [rustfs_ahm::scanner::node_scanner] [crates/ahm/src/scanner/node_scanner.rs:441] [main:ThreadId(1)] create data directory failed "/tmp/rustfs_scanner": Read-only file system (os error 30)
[2025-12-10 12:35:15.014116 +00:00] ERROR [rustfs_ahm::scanner::local_stats] [crates/ahm/src/scanner/local_stats.rs:117] [main:ThreadId(1)] create stats data directory failed "/tmp/rustfs_scanner": Read-only file system (os error 30)
[2025-12-10 12:35:15.014129 +00:00] ERROR [rustfs_ahm::scanner::node_scanner] [crates/ahm/src/scanner/node_scanner.rs:519] [main:ThreadId(1)] Failed to create data directory "/tmp/rustfs_scanner": Read-only file system (os error 30)

@CLAassistant
Copy link

CLAassistant commented Dec 10, 2025

CLA assistant check
All committers have signed the CLA.

This fixes log messages like:

```
[2025-12-10 12:35:15.014094 +00:00] ERROR [rustfs_ahm::scanner::node_scanner] [crates/ahm/src/scanner/node_scanner.rs:441] [main:ThreadId(1)] create data directory failed "/tmp/rustfs_scanner": Read-only file system (os error 30)
[2025-12-10 12:35:15.014116 +00:00] ERROR [rustfs_ahm::scanner::local_stats] [crates/ahm/src/scanner/local_stats.rs:117] [main:ThreadId(1)] create stats data directory failed "/tmp/rustfs_scanner": Read-only file system (os error 30)
[2025-12-10 12:35:15.014129 +00:00] ERROR [rustfs_ahm::scanner::node_scanner] [crates/ahm/src/scanner/node_scanner.rs:519] [main:ThreadId(1)] Failed to create data directory "/tmp/rustfs_scanner": Read-only file system (os error 30)
```
@simonswine simonswine force-pushed the 20251210_mount-tmp-empty-dir branch from 31966f9 to 0db3b4e Compare December 10, 2025 18:26
@simonswine simonswine changed the title 20251210 mount tmp empty dir helm: Mount /tmp as emptyDir Dec 10, 2025
@majinghe
Copy link
Contributor

@simonswine This is related with #1086, scanner logic will change, maybe the dir will not be /tmp, after logic change finishes, will update helm chart. So, if the dir will not be /tmp, then mount will be useless.

@majinghe
Copy link
Contributor

majinghe commented Dec 15, 2025

@simonswine Just update the logic fixing process, team is trying to refactor data scanner part, will take some times. Will merge this PR to fix the issue currently. Thanks for you contribution.

@majinghe
Copy link
Contributor

/LGTM

/cc @houseme @loverustfs

@loverustfs loverustfs merged commit 2e4ce69 into rustfs:main Dec 15, 2025
3 checks passed
@loverustfs
Copy link
Contributor

Ok, merged. @majinghe

jurim76 pushed a commit to jurim76/rustfs that referenced this pull request Dec 20, 2025
Co-authored-by: houseme <housemecn@gmail.com>
Co-authored-by: loverustfs <hello@rustfs.com>
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.

5 participants