"Dr. David Alan Gilbert" <dgilb...@redhat.com> wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> For some reason, compression is not working at the moment, test is
>> disabled until I found why.
>>
>> Signed-off-by: Juan Quintela <quint...@redhat.com>
>> ---
>> tests/migration-test.c | 51
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 51 insertions(+)
>>
>> diff --git a/tests/migration-test.c b/tests/migration-test.c
>> index f0734e4ba0..a8abc3d007 100644
>> --- a/tests/migration-test.c
>> +++ b/tests/migration-test.c
>> @@ -621,6 +621,54 @@ static void test_xbzrle_unix(void)
>> g_free(uri);
>> }
>>
>> +static void test_compress(const char *uri)
>> +{
>> + QTestState *from, *to;
>> +
>> + test_migrate_start(&from, &to, uri);
>> +
>> + /* We want to pick a speed slow enough that the test completes
>> + * quickly, but that it doesn't complete precopy even on a slow
>> + * machine, so also set the downtime.
>> + */
>> + /* 100 ms */
>> + migrate_set_parameter(from, "downtime-limit", "100");
>> + /* 1MB/s slow*/
>> + migrate_set_parameter(from, "max-bandwidth", "100000000");
>> +
>> + migrate_set_parameter(from, "compress-threads", "4");
>> + migrate_set_parameter(from, "decompress-threads", "3");
>
> Should that be 'to' ?
Yes.
> I worry about having this many threads in a loaded test environment.
Suggestion is?
I am not getting they working reliabely without having it loaded.
Later, Juan.
