Fix: update tests for --no-intersection

Signed-off-by: Antoine Busque <abusque@efficios.com>

Merge branch 'staging'

Merge pull request #47 from abusque/stream-intersect

Stream intersect

Generate trace for intersect mode test

This is a basic trace generated to test the intersect mode. The first
three events are in the same stream, the last one is in a different
stream with no overlapping. This trace parsed with babeltrace outputs 4
events by default and 0 with babeltrace --stream-intersection.

Please adapt the test to work with the --disable-intersect-mode.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Add support for stream intersect mode

Signed-off-by: Antoine Busque <abusque@efficios.com>

Merge pull request #48 from eepp/mi-sched-cpu-as-cpu

Fix: cli/sched: use mi.Cpu class for CPU IDs

Fix: cli/sched: use mi.Cpu class for CPU IDs

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

Use --no-color in integration tests

Signed-off-by: Antoine Busque <abusque@efficios.com>

Add --no-color argument

Signed-off-by: Antoine Busque <abusque@efficios.com>

Add debug mode to allow printing traceback

Signed-off-by: Antoine Busque <abusque@efficios.com>

Remove unused time_utils import

Signed-off-by: Antoine Busque <abusque@efficios.com>

Remove unused parameter from _error

Signed-off-by: Antoine Busque <abusque@efficios.com>

Set analysis version using version_utils string parser

Fixes: #42
Signed-off-by: Antoine Busque <abusque@efficios.com>

Add factory method to create Version from version string

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: check whether analysis has ended before setting last timestamp

Fixes: #45

Fix: use begin/end values from _analysis_conf for IO filtering

The values in `_args` previously used were the raw string values from
the command line, whereas those in `_analysis_conf` are parsed into
integers.

Fixes: #44

Fix: only skip analysis if in MI mode with --test-compatibility

Signed-off-by: Antoine Busque <abusque@efficios.com>

cli: add "catch all" for exceptions to sanitize output

This prevents the user from seeing Python tracebacks, and in MI
mode, it makes sure that only the error object is printed if there is
an error, wherever this error happens.

Eventually, log statements should be added at strategic locations to
optionally provide more information about a failure to the user (and
to bug reports).

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli: add --test-compatibility option to check the trace only

This is a "dry-run" option to check if the trace could be analyzed,
without actually analyzing it.

This option is specified by LAMI 1.0.

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli: add --mi-version to print produced MI version

The MI version can be printed as a simple plain text string
with this option to allow an eventual major bump of the protocol
changing the format from JSON to something else.

This option is specified by LAMI 1.0.

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

Remove doc/lami-v0.1.md, move to separate repo

Having the LAMI spec in a separate repository allows to
version it using Git, with separate Git tags, etc.

The new repository is located at https://github.com/lttng/lami-spec

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

fix: softirq entry with raise

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #43 from eepp/update-mi

Update MI

cli: refactor progress indication

This patch:

  * Increases the encapsulation of the different progress
    indicators: the progress code does not access its
    owner's attributes anymore
  * Makes the progress update at most every 100 ms
  * Makes the progress indication use the current timestamp
    and the trace collection's begin/end timestamps to
    approximate its position; falls back on approximating
    with the size if those timestamps are not available
  * Adds the general --progress-use-size CLI option to
    reverse to the old default behaviour of approximating
    with the size

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli/mi: "float" -> "number"

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli: add optional progress lines to MI mode

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli: output JSON error object in MI mode

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

cli/mi: add "mi-version" (latest supported LAMI version)

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

doc: move mi.md -> lami-v0.1.md

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

doc/mi.md: update document

Major differences:

  * The document has a version
  * New Definitions section
  * Clarify the roles of the analysis and the consumer
  * Describe input format arguments more formally
  * Specify the exit status of the analysis as part of
    the output format
  * Add progress indication (with --output-progress
    argument) feature
  * New Common objects section
  * New Error object section: an error object can be
    written by the analysis during both the metadata
    and analysis phases
  * Add mandatory `class` property in the properties
    table of the individual data object sections
  * Style fixes

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>

Merge pull request #41 from lttng/jgalar-typo-children

Typo fix: plural of child is children

Typo fix: plural of child is children

Merge pull request #40 from lttng/staging

Refactor common utilities and introduce unit testing

fix: enable kmem events for lttng-memtop by default

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Tests: add unit tests for common/parse_utils.py

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: return an integer as timestamp from parse_trace_collection_date

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: set microseconds to 0 when parsing timestamp

Signed-off-by: Antoine Busque <abusque@efficios.com>

Re-use _split_value_units in parse_duration

Signed-off-by: Antoine Busque <abusque@efficios.com>

Replace superfluous math.pow uses by ** operator

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix erroneous comments in parse_utils

Signed-off-by: Antoine Busque <abusque@efficios.com>

Tests: add unit tests for common/trace_utils.py

Signed-off-by: Antoine Busque <abusque@efficios.com>

Tests: add unit tests for common/format_utils.py

Signed-off-by: Antoine Busque <abusque@efficios.com>

Move integration tests into their own subdir

Signed-off-by: Antoine Busque <abusque@efficios.com>

Refactor utils from linuxautomaton/common into the common package

This refactor/clean-up targets the utility functions previously found
at linuxautomaton/common.py, despite them being used across all
packages. Unused methods have been culled, while others have been
merged, split, or simplified.

All methods are extensively documented. One of the main goals of this
refactor was to improve testability. Unit tests will therefore be
implemented for these methods shortly.

The methods have been pythonified quite a bit as well. Most now make
use of exceptions, in concordance with the python philosophy of EAFP
over LBYL.

Signed-off-by: Antoine Busque <abusque@efficios.com>

Style: fix minor pep8 style issues

Signed-off-by: Antoine Busque <abusque@efficios.com>

Remove unused StateVariable class

Signed-off-by: Antoine Busque <abusque@efficios.com>

changelog for v0.4.3

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #39 from abusque/save-test-results

Fix: keep the trace and output if a test fails

Tests: clean-up test utility functions

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: keep the trace and output if a test fails

Instead of always removing the traces and only outputting a diff, if the
test fails, we now keep the trace and the output we got, so it is easy
to reproduce afterwards or just copy the new output to the "expected"
folder if it is correct.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Fix: test lines too long

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Fix: use GMT clock for tests

Force the use of the GMT reference clock for our tests to run outside
the eastern time zone.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #37 from abusque/license-copyright

Add year and copyright holder to license file

Merge pull request #36 from mjeanson/master

Add license and Changelog to sdist

Add year and copyright holder to license file

Signed-off-by: Antoine Busque <abusque@efficios.com>

Add license and Changelog to sdist

changelog for v0.4.2

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #35 from mjeanson/master

Fix: Add wrapper scripts to sdist for tests

Fix: Add wrapper scripts to sdist for tests

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

changelog for v0.4

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #34 from mjeanson/master

Fix: Include tests dir in source dist

Fix: Include tests dir in source dist

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>

Merge pull request #33 from abusque/staging

Staging fixes

Extract prio list formatting into format_utils

Signed-off-by: Antoine Busque <abusque@efficios.com>

Set SyscallEvent.ret to None if missing due to lttng bug

Signed-off-by: Antoine Busque <abusque@efficios.com>

fix test for new formatting

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Add spaces where columns are too close

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Compact priority list

Instead of displaying the full list of priorities a process had, only
display a summary (number of time a process had each priority).

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Warn if ret field missing in syscall

Detect and output a warning if a system call does not have a "ret"
field, it happened with development version of lttng-modules and it is
good to detect it instead of crashing or ignoring it silently.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Merge pull request #31 from abusque/staging

v0.4 changeset

Fix: check for current_task_start_ts, not last_sched_ts, in cputop

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: remove reference to deprecated ascii_graph module in setup.py

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: generic filename formatting

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: correctly fill per proc block I/O write data

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: erroneous iadd operator for Stats.IO

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: set proc name in I/O analysis if previously unknown

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: handle max values of 0 in termgraph

Signed-off-by: Antoine Busque <abusque@efficios.com>

Handle multiple hard IRQ handler names

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: handle missing vec and empty softirq list on exit

Signed-off-by: Antoine Busque <abusque@efficios.com>

fix: double append in rq_list

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Remove accept from net_open syscalls

accept and accept4 block voluntarily, so computing latencies on these
system calls do not make much sense.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

fix: vec presence check

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Style: indentation issues

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Tests: refactor test framework to eliminate py.test

This replaces the external py.test dependency with the standard
`unittest` package.

Signed-off-by: Antoine Busque <abusque@efficios.com>

Tests: clean-up and pythonify test framework

Signed-off-by: Antoine Busque <abusque@efficios.com>

Tests: initial test framework implementation

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Try to read the text metadata if babeltrace failed

Reading a trace with babeltrace -o ctf-metadata fails if the metadata is
in text format instead of binary format. When we detect this error, try
to read the metadata with 'cat'.

Also, the traces generated with the CTFWriter API can only write strings
in the environment section for now, so we add optional quotes around the
version regex to allow extracting the version in these traces.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Remove eventual trailing slash in trace path

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Replace Pyasciigraph by termgraph for freq graphs

The only remaining uses of Pyasciigraph were limited to frequency
graphs, so this commit introduces a replacement under the termgraph
module to display such frequency graphs.

This effectively eliminates the Pyasciigraph dependency, which is now
removed from the source tree.

Signed-off-by: Antoine Busque <abusque@efficios.com>

Remove obsolete convert_size function

Signed-off-by: Antoine Busque <abusque@efficios.com>

Fix: remove unused proc from io_rq filter

Signed-off-by: Antoine Busque <abusque@efficios.com>

Replace Pyasciigraph by termgraph for bar graphs in io.py

Signed-off-by: Antoine Busque <abusque@efficios.com>

Replace Pyasciigraph by termgraph in memtop

Signed-off-by: Antoine Busque <abusque@efficios.com>

Replace Pyasciigraph by termgraph in cputop, display prios

Signed-off-by: Antoine Busque <abusque@efficios.com>

Style: fix spacing issue

Signed-off-by: Antoine Busque <abusque@efficios.com>

Initial implementation of the termgraph module

Signed-off-by: Antoine Busque <abusque@efficios.com>

Add a format_utils module to the common package

Signed-off-by: Antoine Busque <abusque@efficios.com>

Remove temporary mitest files

Signed-off-by: Antoine Busque <abusque@efficios.com>

Use core/stats in cputop and sched, add prio_list support

Signed-off-by: Antoine Busque <abusque@efficios.com>

Use core/stats in memtop and syscalls analysis

Signed-off-by: Antoine Busque <abusque@efficios.com>