Improve Fedora dependency repo instructions
Tests: Include machine id in disk image name.

So that we can run tests in parallel.

Closes #101

Signed-off-by: Stef Walter <stefw@redhat.com>
Tests: Don't allow NetworkManager messages anymore.

They were fixed with 648167f and 709da26.

Closes #109

Signed-off-by: Stef Walter <stefw@redhat.com>
Tests: Increase default timeout.

And also ever increase the timeout when the tests specify a different
value.

A tight timeout is mostly useful for debugging, but descreases the
robustness of the test suite needlessly otherwise.

Closes #111

Signed-off-by: Stef Walter <stefw@redhat.com>
Tests: Download journal database on failure

Closes #113

Signed-off-by: Stef Walter <stefw@redhat.com>
Tests: Fix qemu process leaks.

The machine.kill() call was erroneously only done when checking
journal messages.  Also, a qemu process might leak when the
multi-machine tearDown throws in the middle.

To fix this, we add a "new_machine" function that registers a cleanup
for the new machine.  The explicit kills can then be removed.

Closes #112

Signed-off-by: Stef Walter <stefw@redhat.com>
Tests: Allow parallel verification runs.

But default to sequential runs.

Closes #110

Signed-off-by: Stef Walter <stefw@redhat.com>
FEDORA: Use the $basearch substitution in repo file

This modifies the repo baseurl to use standard substitution
variables so it should work on all future versions and arches of
Fedora.

Closes 118.

Signed-off-by: Marius Vollmer <mvollmer@redhat.com>
testvm.py: Find /usr/libexec/qemu-kvm for RHEL

On RHEL, qemu is not in $PATH since it's not a supported API.
test/vm-prep: Handle qemu config location on RHEL7

On RHEL, qemu is packaged differently; attempt to find the bridge
location in both locations as found on Fedora/RHEL.
Tests: Change default TEST_OS to fedora-20.
Tests: One less race in the storage tests.

We need to wait until /mnt has actually appeared in the list before
attempting to format it.
websocketconnection: Fix web_socket_client_new

web_socket_client_new, as used in frob-weboscket, does not take an
IO stream on construction. Instead, it connects to an arbitrary URI
using g_socket_client_connect_to_uri, and then tries to set up the
WebSocketConnection using the private _web_socket_connect_take_io_stream
API. The issue is that WebSocketConnection tries to read from the IO
stream in an idle set after construction, but we can't be guaranteed
that the IO stream will exist at this time.

Create the idle in _web_socket_connection_take_io_stream, which should
always be called when we set the IO stream for a WebSocketConnection.

Closes #123

Signed-off-by: Stef Walter <stefw@redhat.com>
SRPM: make sure submodules are initialized

If we try to make an SRPM from a pristine checkout (as with
automation), it would be missing the necessary submodule to build
successfully. This patch just makes sure that the make-srpm tool
initializes the submodules.
ws: Fix problems that coverity detected
daemon: Fix problems that coverity detected
Tests: Allow "cd test && ./make-srpm" and don't write to stdout.
Logging: Honor G_MESSAGES_DEBUG also for journal logging.

This allows the use of structured logging and debug output at the same
time.
Tests: Log debug messages.

* Cockpitd and cockpit-ws run with G_MESSAGES_DEBUG=all.
* Only journal messages of level 5 or more urgent are checked.
Logging: Give LOG_ERR priority to assertion failure messages.

By explicitly redirecting g_printerr to the journal.