[LU-4512] POSIX copytool option "--report" is non-obvious and functionality is broken

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.6.0, Lustre 2.5.1
• Severity: 3
• Project: HSM
• Assignee: Michael MacDonald (Inactive)
• Reporter: Michael MacDonald (Inactive)

Description

The POSIX copytool supports a "--report N" option, where N is an interval in seconds between copytool progress reports to the CDT. This option is intended to allow a change from the default interval of 30 seconds. At least, this is what the code indicates is expected to happen. Unfortunately, it seems supplying this option with any positive integer (other values were not tested) results in the copytool attempting to send reports many times per second. There is also no short version of the switch.

In a forthcoming patch, I intend to fix this bug and also propose a rename of the "-report" option to "-update-progress, -u".

Comments

[Michael MacDonald (Inactive)]

landed on
master: 8ddf4e2aa863e3100a3dff916821f2739e9427cd
b2_5: 9059ed44eff9e33afa0da7cc1f0e1311830293e4

[Michael MacDonald (Inactive)]

b2_5: http://review.whamcloud.com/#/c/8934/
master: http://review.whamcloud.com/#/c/9034/

Both have +1 from Jenkins/Maloo, and the b2_5 changeset has 3 +1 code-reviews.

[Michael MacDonald (Inactive)]

Patch created for master too: http://review.whamcloud.com/#/c/9034/

The b2_5-based patch has 2 +1s for code review and a +1 from Maloo.

[Michael MacDonald (Inactive)]

Doug: Not for me to say, though I will point out that --report was completely undocumented to begin with.

[Doug Oucharek (Inactive)]

Does this need an LUDOC ticket?

[Michael MacDonald (Inactive)]

Fix pushed for review: http://review.whamcloud.com/#/c/8934/