Fix the issue where compiling containers-storage with exclude_disk_qu…

[Syx5290]

Reference:
#1978 (comment)

To ensure that getComposeFsHelper and others are included, I attempted to move the ReadWriteDiskUsage function from no nocgo into a separate file, creating overlay_disk_quota_unsupported.go. However, the overlay_cgo.go file should not be moved out as it would then be an empty file, and it does not affect the existing logic. What are your thoughts on this modification?

Compile:

root@syx:/storage# make clean
rm -f -f containers-storage containers-storage.* docs/*.1 docs/*.5
root@syx:/storage# make binary
go build -compiler gc -tags " libdm_no_deferred_remove   exclude_disk_quota"  ./cmd/containers-storage
root@syx:/storage# vim Makefile 
root@syx:/storage# make binary
go build -compiler gc -tags " libdm_no_deferred_remove  "  ./cmd/containers-storage
root@syx:/storage#

[mtrmac]

Thanks!

The approach LGTM, just one more cleanup.

(You can force-push over your existing branch instead of opening a new PR.)

[rhatdan]

Squash your commits and make sure the one commit is signed.

git rebase -i origin
git push --force

[Syx5290]

Squash your commits and make sure the one commit is signed.

git rebase -i origin git push --force

Thank you for your reminder, I have made the modifications as per the standard requirements, thank you again.

[rhatdan]

/approve
/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rhatdan, xuegege5290

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [rhatdan]

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

[rhatdan]

Thanks @xuegege5290