Bug #343: When enable-event with invalid filter, error is given but the event is still enabled - LTTng-tools - LTTng bugs repository

Actions

Copy link

Bug #343

closed

TL DG

When enable-event with invalid filter, error is given but the event is still enabled

Bug #343: When enable-event with invalid filter, error is given but the event is still enabled

Added by Tan le tran about 13 years ago. Updated about 13 years ago.

Status:

Resolved

Priority:

High

Assignee:

David Goulet

Target version:

LTTng - 2.1 stable

Start date:

09/18/2012

Due date:

% Done:

100%

Estimated time:

Description

Scenario: =========

1)_ lttng create ses1 -o /cluster/temp/sessiondir/ses1
2)_ lttng enable-event ust_tests_demo2:loop -u --filter ' intfield => 2 '
get:
error syntax error
Parse error
Error: Error setting filter
3)_ lttng list ses1
get:
:
Events:
ust_tests_demo2:loop (type: tracepoint) [enabled]

Expected Behaviour: ===================
The event should not be enabled if the specified filter is invalid.

Files

log.txt (13.5 KB) log.txt

Terminal log with "sessiond -vvv"

Tan le tran, 09/18/2012 05:08 PM

History
Notes
Property changes
Associated revisions

TL Updated by Tan le tran about 13 years ago Actions
Copy link
#1

The above issue has been observed with the following commits:

userspace : (Thu Sep 6 ) a5a9f42 Ensure that read-side functions meet 10-line LGPL criterion
lttng-ust : (Tue Sep 18) 5d3bc5e Fix: get_wait_shm() ust mutex deadlock (add 2 missing exit calls)
lttng-tools:(Tue Sep 18) 4adabd6 Fix: Remove LPOLLNVAL from consumer metadata revents
babeltrace: (Thu Sep 13) f546472 Fix: Documentation cleanup

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#2

Project changed from LTTng-UST to LTTng-tools

DG Updated by David Goulet about 13 years ago Actions
Copy link
#3

Status changed from New to Confirmed
Assignee set to David Goulet
Priority changed from Normal to High
Target version set to 2.1 stable

DG Updated by David Goulet about 13 years ago Actions
Copy link
#4

Status changed from Confirmed to Feedback

So actually I don't think this is an error.

Enabling an event and a filter is two different API calls so the event is first enabled successfully but the filter is wrong so the fix is simply to retry lttng enable-event with the right filter and it works.

I'm not sure I can easily disable or even delete an event on a wrong filter... Rerun the command with the correct filter is I think the correct behavior and thing to do.

Thought?

Thanks

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#5

Yes, this is a bug. This is an unexpected behavior on the command line.

If we see that setting the filter triggers an error, we should disable the event.

This is an issue at the command line level, not at the API level.

DG Updated by David Goulet about 13 years ago Actions
Copy link
#6

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

I'm not sure to be honest.

There is a use case where enabling let say event ust-eventA, tracing
and then later on setting a filter on it. Disabling it seems to me a
bit harsh and not a right behavior for an already good enabled event.

David

bugs@lttng.org:

Issue #343 has been updated by Mathieu Desnoyers.

Yes, this is a bug. This is an unexpected behavior on the command
line.

If we see that setting the filter triggers an error, we should
disable the event.

This is an issue at the command line level, not at the API level.

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJQXJrXAAoJEELoaioR9I02zKIH/3pLValBcrJPrnRjEwjjL2Kw
r04j9J9UH93w+fEU0nl485JGphsEG/htOA73QRJUFKwgHP9vY6+iL96QUh2apPWf
QNgH0oz3rIb2orFEc0UtEiWxHvoAQMfPTu2B7xgj1mFDJyvIBVfZ1FGuhQyPzdqD
8bl4jCaMeHVZVAsQt931ySsXwHOFlkSS6bc+EOU+VgdKrvSw0TtkOQCSce6csWXT
jWkyUAoBc0DxM00h5eerBQNY8jpdrYQ83d79x9vEJKTlUtVdKwSIRUTdtmTSpm4/
N8AljAEjlyHDo4VvrNuj6qkIke+OqIbp7R4OQ3jcdL6ws5upzNEmMh5QPWzbBCg=
=qThR
-----END PGP SIGNATURE-----

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#7

That's a case where there has been a reported error when setting up the filter. It's OK to be harsh and disable a previously enabled event in that kind of condition. You might want to let the user know that the event id disabled due to the filter error.

DG Updated by David Goulet about 13 years ago Actions
Copy link
#8

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

I guess we could simply do a lttng_disable_event if an error
occured... that makes sense to me.

I'll keep this bug alive and fix this for 2.1-stable.

Thanks
David
-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJQXJ90AAoJEELoaioR9I0227AIAKTxM2IvOMMql7PsbzCLG99E
l7ZumDeXvm8BiR/zVXYqeLiMdWo6ACbhkfO+TivxyIyiqfZeTkR4WxX7NzInOSc6
4gA2WkmzqC8932xakPtbiHtWHaShLQdCYttEoj/Is6UIgbBxGHubZy+IQoCl/HT4
FouUnE03tjFK360voTiE75rqFdJSWR+PWYVnJb7rhOyQQEPA3kpwBbp9UNPYF2qE
5kO9u2Ak4RbOaZ/RYFc0mObkN6rtWx9QrXx6l6FeYtlRRw1/c32MR+3XWPxE2mph
XB3Kmrhyx40MdmlZlHRy7S2TM0YZi24KFJMVWTLwljtqmmyZRZLmJ0vlQM7/p1k=
=nL3H
-----END PGP SIGNATURE-----

DG Updated by David Goulet about 13 years ago Actions
Copy link
#9

Status changed from Feedback to Confirmed

DG Updated by David Goulet about 13 years ago Actions
Copy link
#10

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

Applied in changeset 1636365217be276e0a9e19ab4b8b6cdcb84afd90.

Actions

Copy link

Also available in: PDF Atom

Project

General

Profile

LTTng » LTTng-tools

Bug #343

When enable-event with invalid filter, error is given but the event is still enabled

TL Updated by Tan le tran about 13 years ago Actions
Copy link
#1

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#2

DG Updated by David Goulet about 13 years ago Actions
Copy link
#3

DG Updated by David Goulet about 13 years ago Actions
Copy link
#4

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#5

DG Updated by David Goulet about 13 years ago Actions
Copy link
#6

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#7

DG Updated by David Goulet about 13 years ago Actions
Copy link
#8

DG Updated by David Goulet about 13 years ago Actions
Copy link
#9

DG Updated by David Goulet about 13 years ago Actions
Copy link
#10

Project

General

Profile

LTTng » LTTng-tools

Bug #343

When enable-event with invalid filter, error is given but the event is still enabled

TL Updated by Tan le tran about 13 years ago ActionsCopy link #1

MD Updated by Mathieu Desnoyers about 13 years ago ActionsCopy link #2

DG Updated by David Goulet about 13 years ago ActionsCopy link #3

DG Updated by David Goulet about 13 years ago ActionsCopy link #4

MD Updated by Mathieu Desnoyers about 13 years ago ActionsCopy link #5

DG Updated by David Goulet about 13 years ago ActionsCopy link #6

MD Updated by Mathieu Desnoyers about 13 years ago ActionsCopy link #7

DG Updated by David Goulet about 13 years ago ActionsCopy link #8

DG Updated by David Goulet about 13 years ago ActionsCopy link #9

DG Updated by David Goulet about 13 years ago ActionsCopy link #10

TL Updated by Tan le tran about 13 years ago Actions
Copy link
#1

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#2

DG Updated by David Goulet about 13 years ago Actions
Copy link
#3

DG Updated by David Goulet about 13 years ago Actions
Copy link
#4

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#5

DG Updated by David Goulet about 13 years ago Actions
Copy link
#6

MD Updated by Mathieu Desnoyers about 13 years ago Actions
Copy link
#7

DG Updated by David Goulet about 13 years ago Actions
Copy link
#8

DG Updated by David Goulet about 13 years ago Actions
Copy link
#9

DG Updated by David Goulet about 13 years ago Actions
Copy link
#10