Bug #1081
closedlttng-create behaviour does not match its command-line specification
100%
Description
The lttng-create (2.9) synopsis reads in part:
Live mode:
lttng [GENERAL OPTIONS] create --live[=DELAYUS] [--shm-path=PATH]
[--set-url=URL | --ctrl-url=URL --data-url=URL] [SESSION]
Which means this should be legal:
lttng create --live testsession
This is what we actually get:
$ lttng create --live testsession Error: Wrong value in --live parameter: testsession Error: Command error
Either the code of lttng-create should be changed to correctly process its command line, or its man pages need to be adjusted.
Updated by Daniel U. Thibault almost 8 years ago
Note that this also means the example currently given in http://lttng.org/docs/v2.9/#doc-lttng-live fails on step 1.
Updated by Jonathan Rajotte Julien almost 8 years ago
- Status changed from New to Confirmed
Updated by Jonathan Rajotte Julien almost 8 years ago
- Status changed from Confirmed to Feedback
Updated by Daniel U. Thibault almost 8 years ago
This leaves the http://lttng.org/docs/v2.9/#doc-lttng-live page to be fixed, because it now gives illegal command lines for some of the examples ('lttng create --output=/tmp/some-directory my-session', 'lttng create --live my-session', 'lttng create --snapshot my-session').
Updated by Jonathan Rajotte Julien almost 8 years ago
Those changes are conditional to [1] acceptance.
The pull request for lttng-doc [2] is the minimum functional change. Only "--live session_name" order poses an immediate problem. Changes of arguments order for other create command will be changed depending on the result of [1].
Cheers
[1] https://lists.lttng.org/pipermail/lttng-dev/2017-January/026855.html
[2] https://github.com/lttng/lttng-docs/pull/19
Updated by Jonathan Rajotte Julien almost 8 years ago
- Status changed from Feedback to Resolved
- % Done changed from 0 to 100
Applied in changeset lttng-tools|1f3c3a24e3a07f6fd032ad50bac92f1f58d33a12.
Updated by Daniel U. Thibault almost 8 years ago
Don't forget to fix http://lttng.org/docs/v2.9/#doc-lttng-live so it shows the new syntax.