Add -x to du commands to not traverse other file systems

[cwdsuzhou]

What type of PR is this?

/kind bug

What this PR does / why we need it:

See #71293
Since we can not get feedback from @ehmm, I would like open this PR to fix it

Which issue(s) this PR fixes:

Fixes #71012
Special notes for your reviewer:

/cc @jingxu97 @msau42

Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

[k8s-ci-robot]

@cwdsuzhou: The label(s) kind/ cannot be appled. These labels are supported: api-review, community/discussion, community/maintenance, community/question, cuj/build-train-deploy, cuj/multi-user, platform/aws, platform/azure, platform/gcp, platform/minikube, platform/other

In response to this:

What type of PR is this?

/kind bug

What this PR does / why we need it:

See #71293
Since we can not get feedback from @ehmm, I would like open this PR to fix it

Which issue(s) this PR fixes:

Fixes #71012
Special notes for your reviewer:

/cc @jingxu97 @msau42

Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[cwdsuzhou]

/assign @saad-ali

[msau42]

/assign @jingxu97 @dashpole
#71293 tried to fix it by changing the cadvisor call, whereas this PR is fixing it in the VolumeMetrics interface, which gets output as pod volume usage under summary metrics. Can you clarify where the cadvisor usage metric is used and if we should look into fixing that too?

[dashpole]

/lgtm
The cadvisor change was for collection of metrics for the container writable layer. However, we had already switched to a golang implementation to replace du and find. We should probably transition the volume metrics collection to use the golang implementation, but can do this in the meantime.

[msau42]

Thanks for the explanation! lgtm

@cwdsuzhou can you update your PR title to reflect that it's adding "-x" option, not "-s"

[msau42]

/priority important-soon

[k8s-ci-robot]

@cwdsuzhou: The label(s) kind/ cannot be appled. These labels are supported: api-review, community/discussion, community/maintenance, community/question, cuj/build-train-deploy, cuj/multi-user, platform/aws, platform/azure, platform/gcp, platform/minikube, platform/other

In response to this:

What type of PR is this?

/kind bug

What this PR does / why we need it:

See #71293
Since we can not get feedback from @ehmm, I would like open this PR to fix it

Which issue(s) this PR fixes:

Fixes #71012
Special notes for your reviewer:

/cc @jingxu97 @msau42

Does this PR introduce a user-facing change?:

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[msau42]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cwdsuzhou, msau42

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/volume/testing/OWNERS [msau42]
• pkg/volume/util/OWNERS [msau42]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[cwdsuzhou]

@msau42 should we cherry pick this PR to other release versions?