Project

General

Profile

Actions

Bug #522

closed

default channels should not be created when a channel has been explicitely created

Added by Mathieu Desnoyers almost 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
High
Assignee:
-
Target version:
Start date:
05/06/2013
Due date:
% Done:

100%

Estimated time:

Description

As noted in bug https://bugs.lttng.org/issues/516 , creating a "default" channel if a user forgets to put "-c channel_name" in the following add-context, enable-event, etc commands is misleading. The natural expected behavior would be that the command would apply the the last enabled channel (notion of a "current" channel, like we have for sessions), but certainly not that it creates a new channel with default attributes behind the user's back.

An intermediate fix that would not require to keep a notion of current channel could be to refuse creation of a default channel if at least one non-default channel has already been created in the session.

Since this is an issue of UI correctness (it's hiding a side-effect from the user), I think it should be treated as a bug.

Thoughts ?

Thanks,

Mathieu

Actions #1

Updated by Daniel U. Thibault almost 11 years ago

I would vote to make the -c option mandatory, the command being rejected with a clear error message. Having a "current channel" would not only be complicated (lttng-sessiond would need to keep each session's current channel in memory, so that when a user switches from one session to another the new session's current channel is recovered, plus the session-setting user feedback message would need to remind the user of the current channel for that session), it would lead to user mistakes (unintended effects) when re-ordering or copy-and-pasting lttng script lines.

Actions #2

Updated by David Goulet over 10 years ago

  • Status changed from New to Confirmed
  • Target version set to 2.3
Actions #3

Updated by Mathieu Desnoyers over 10 years ago

  • Status changed from Confirmed to Resolved
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF