rgw: modify documents and help infos' descriptions to the usage of option date when executing command "log show"

[clever215]

DOCUMENTATION #13235

This modification concerns documents and help infos. To the documents, we delete original incorrect instruction about the usage of the option date which induces user to type the value of date in the format of "YYYY-MM-DD" when executing "log show" command, and we add the actual usage of "YYYY-MM-DD-hh". To the help infos, we add the NOTE to make user notice that the usage of option date here is different with default's and of cource we give the right formatting.

Signed-off-by: Kongming Wu wu.kongming@h3c.com

[loic-bot]

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/8033/

• SUCCESS http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/8033/

[loic-bot]

FAILURE: http://jenkins.ceph.dachary.org/job/ceph/10330/

• FAILURE http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10330/

[loic-bot]

FAILURE: http://jenkins.ceph.dachary.org/job/ceph/10332/

• FAILURE http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10332/

[loic-bot]

FAILURE: http://jenkins.ceph.dachary.org/job/ceph/10334/

• FAILURE http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10334/

[smithfarm]

Requiring the user to type YYYY-MM-DD-HH seems odd. If that is really required, I wonder if it is a bug that should be fixed instead of adapting the documentation to it.

[smithfarm]

Also note that this does not pass "make check" (see bot failures, above - especially the "Console log" therein). It cannot be merged until that passes.

[loic-bot]

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/10337/

• SUCCESS http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10337/

[clever215]

@smithfarm I added this PR in Sept. and this Issue is existed. i.e. '-HH' (hour) must be appended.What weird is why loic-bot compiled failed, cause this part of codes does not change and I compiled successfully in local.

[clever215]

@smithfarm log searching has its inside mechanism, which parameter 'YYYY-MM-DD-HH' will be transfered into timestamp as a key word. I try to repushed changes, now, this is not what i want to submit.

[loic-bot]

FAILURE: http://jenkins.ceph.dachary.org/job/ceph/10338/

• FAILURE http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10338/

[loic-bot]

FAILURE: http://jenkins.ceph.dachary.org/job/ceph/10339/

• FAILURE http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10339/

[smithfarm]

@clever215 I'm not familiar with thehelp.t test but it appears that it is failing:

src/test/cli/radosgw-admin/help.t: failed
--- /home/jenkins/workspace/ceph/LABELS/ceph-centos-7-jenkins/src/test/cli/radosgw-admin/help.t
+++ ./src/test/cli/radosgw-admin/help.t.err
@@ -44,7 +44,8 @@
     log list                   list log objects
     log show                   dump a log from specific object or (bucket + date
                                + bucket-id)
-                               (NOTE: required to specify formatting of date to "YYYY-MM-DD-hh")
+                               (NOTE: required to specify formatting of date
+                               to "YYYY-MM-DD-hh")
     log rm                     remove log object
     usage show                 show usage (by user, date range)
     usage trim                 trim usage (by user, date range)
# Ran 1 tests, 0 skipped, 1 failed.

[clever215]

@smithfarm yeah, I see. radosgw-admin.cc and help.t are not in the same format that cause this failing.I try to fix it. Sorry about that!

[loic-bot]

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/10340/

• SUCCESS http://jenkins.ceph.dachary.org/job/ceph/LABELS=centos-7&&x86_64/10340/

[clever215]

@smithfarm It's OK now!

[clever215]

@smithfarm I wish it will be merged soon. Cause these notes mislead users for a long time.Happy weekends!

[clever215]

@smithfarm Hi, How's that, I means this change?

[clever215]

@yehudasa How do you think about this issue?