Skip to content

Commit

Permalink
roachtest: exclude lost+found directory
Browse files Browse the repository at this point in the history
This directory is created by the filesystem and unowned file chunks are
put there by `fsck`. The directory and its contents aren't readable by
anyone except `root`, so this can cause the `du -c /mnt/data1` that
`roachtest` performs to fail -- add an `--exclude` to handle this.

We already ignore this directory in other contexts (for example, see
`pkg/storage/mvcc.go`).

Fixes cockroachdb#53663.

Release justification: Non-production code change
Release note: None
  • Loading branch information
rickystewart authored and tbg committed Jun 24, 2021
1 parent 320d288 commit a7c1baf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -1574,7 +1574,7 @@ func (c *cluster) FetchDiskUsage(ctx context.Context) error {
}
if err := execCmd(
ctx, c.l, roachprod, "ssh", c.name, "--",
"/bin/bash", "-c", "'du -c /mnt/data1 > "+name+"'",
"/bin/bash", "-c", "'du -c /mnt/data1 --exclude lost+found > "+name+"'",
); err != nil {
// Don't error out because it might've worked on some nodes. Fetching will
// error out below but will get everything it can first.
Expand Down

0 comments on commit a7c1baf

Please sign in to comment.