Bug 1750387 - Deprecated log rotate command still present in "# gluster v help"
Summary: Deprecated log rotate command still present in "# gluster v help"
Alias: None
Product: GlusterFS
Classification: Community
Component: cli
Version: mainline
Hardware: x86_64
OS: Linux
Target Milestone: ---
Assignee: Kshithij Iyer
QA Contact:
Depends On:
Blocks: 1784415
TreeView+ depends on / blocked
Reported: 2019-09-09 13:48 UTC by Kshithij Iyer
Modified: 2019-12-17 11:38 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1784415 (view as bug list)
Last Closed: 2019-12-17 11:33:50 UTC
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 23392 0 None Merged [Cli] Removing old log rotate command. 2019-12-17 11:33:49 UTC

Description Kshithij Iyer 2019-09-09 13:48:10 UTC
Description of problem:
{"volume log rotate <VOLNAME> [BRICK]", cli_cmd_log_rotate_cbk,
 "rotate the log file for corresponding volume/brick"
 " NOTE: This is an old syntax, will be deprecated from next release."},

Should be removed from cli-cmd-volume.c as the command seems to be depreciated now.

Version-Release number of selected component (if applicable):
Whatever is the latest version.

How reproducible:

Steps to Reproduce:
1.Run # gluster v help on a gluster cluster.

Actual results:
Command is displayed in the help menu.

Expected results:
Command should not be a part of the help menu.

Additional info:

Comment 1 Worker Ant 2019-09-09 13:58:12 UTC
REVIEW: https://review.gluster.org/23392 ([Cli] Removing old log rotate command from \"# gluster v help\".) posted (#1) for review on master by Kshithij Iyer

Comment 2 Sanju 2019-11-19 12:26:30 UTC
Kshithij, would you like to work on the fix for this?

Comment 3 Kshithij Iyer 2019-11-19 13:10:14 UTC
Yes! I would like to give it a try.

Comment 4 Worker Ant 2019-12-17 11:33:50 UTC
REVIEW: https://review.gluster.org/23392 ([Cli] Removing old log rotate command.) merged (#16) on master by Sunny Kumar

Note You need to log in before you can comment on or make changes to this bug.