test: Don't use ifconfig, as it's less available on linux

Use the 'ip' command to test for the presence of an interface
instead.

Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
* Fixed the path to the IP binary to be more compatible
agent: Document and implement cancelling of restjson requests.

Fixes #372
Closes #344

Reviewed-by: Stef Walter <stefw@redhat.com>
web: Tell the channel when a restjson request is cancelled.

Fixes #372
Closes #444

Reviewed-by: Stef Walter <stefw@redhat.com>
daemon: Fix invalid memory accesses in resource monitors.

Wrap the index correctly in GetSamples and don't run it in a thread.
The samples can not be read from multiple threads.

Closes #449

Reviewed-by: Stef Walter <stefw@redhat.com>
travis: Do a distcheck in our travis CI build

Allows us to catch even more issues

Closes #447
Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
agent: Fix dead code in CockpitRestJson

Found by Coverity scan

Closes #446
Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
daemon: Fix memory leak in CGroup show code

Found by Coverity scan

Closes #446
Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
ws: Track when we sent an EOF to each agent

This prevents races while shutting down, and is a patch
that was accidentally not pushed in the earlier multi-agent
work.

Closes #443
Remove last traces of libtool from our build and tests

We got rid of libtool usage a while back, remove some last vestiges
which were breaking our check-memory target if libtool wasn't installed
and other little bits.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
Don't run test-server phantomjs tests under valgrind

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
tools: Make tap-gtester more compatible with older python

The argparse in older python seems to behave differently
with regards to the '--' argument so use argparse.REMAINDER
to catch the command line.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
Run 'check-memory' target in travis as well

This allows us to catch various memory related problems
although, not leaks (yet). Also files not distributed properly
or cleaned up properly.

Closes #450
ws: Don't run test-transport unit tests twice

This was left behind when we moved test-transport.c to libcockpit.

Closes #448
Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
web: Fix container page graphs when not using lxc

When not using lxc cgroups, we need to account for different
cgroup names when piecing together the graph consumers at the
top of the page.

Closes #442
Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
agent: Add cockpit_channel_get_strv_option() function

Used by channel implementors to get a string array from the
channel options. Implemented this using cockpit_json_get_strv()
utility function.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
common: Simplify the mock service code a bit

We use the GDBusObjectManager to look up the extra objects rather
than having a separate hash table for them.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
common: Add RequestPropertyMods method to mock service

A simple way to check PropertiesChanged signal

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
common: Add a path which fails introspection to mock service

We can use this for checking our introspection logic.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
common: Bug fix for using mock service more than once in a process

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>
common: Force mock service to always use a new dbus connection

Otherwise this can lead to races between tests.

Reviewed-by: Marius Vollmer <marius.vollmer@redhat.com>