[ovirt-devel] Failing tests for storage-py27.

[Petr Kotas]

The patch works as promised.
The issue is now gone.

Thanks,
Petr

On Tue, Aug 1, 2017 at 9:12 AM, Petr Kotas <pkotas at redhat.com> wrote:

> Thanks Nir,
>
> I am working on it right now, will post result.
>
> Petr
>
> On Mon, Jul 31, 2017 at 7:35 PM, Nir Soffer <nsoffer at redhat.com> wrote:
>
>> Pter, should be fixed by https://gerrit.ovirt.org/80027
>>
>> Can you test and report if it works for you?
>>
>> On Mon, Jul 31, 2017 at 8:24 PM Nir Soffer <nsoffer at redhat.com> wrote:
>>
>>> On Mon, Jul 31, 2017 at 8:09 PM Nir Soffer <nsoffer at redhat.com> wrote:
>>>
>>>> On Mon, Jul 31, 2017 at 5:52 PM Petr Kotas <pkotas at redhat.com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> I have encountered an issue with VDSM master when running tests.
>>>>> Maybe I am doing something wrong, or I am overlooking something.
>>>>>
>>>>> This is a clear master without any modification pulled with the latest
>>>>> commits.
>>>>> Did anyone please encountered the same issue:
>>>>>
>>>>> ================================================== FAILURES
>>>>> ==================================================
>>>>> ______________________________________ TestDirectioChecker.test_path_ok
>>>>> ______________________________________
>>>>>
>>>>> self = <storage.check_test.TestDirectioChecker
>>>>> testMethod=test_path_ok>
>>>>>
>>>>>     def test_path_ok(self):
>>>>>         self.checks = 1
>>>>>         with temporaryPath(data=b"blah") as path:
>>>>>             checker = check.DirectioChecker(self.loop, path,
>>>>> self.complete)
>>>>>             checker.start()
>>>>>             self.loop.run_forever()
>>>>>             pprint.pprint(self.results)
>>>>>             result = self.results[0]
>>>>> >           delay = result.delay()
>>>>>
>>>>> storage/check_test.py:82:
>>>>> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>>>>> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>>>>>
>>>>> self = <CheckResult path=/var/tmp/tmpPpV1Rj rc=0 err=bytearray(b'0+1
>>>>> records in\n0+1 ... copied, 0,000496228 s, 8,1 kB/s\n') time=4299448.73
>>>>> elapsed=0.00 at 0x3b72390>
>>>>>
>>>>>     def delay(self):
>>>>>         # TODO: Raising MiscFileReadException for all errors to keep
>>>>> the old
>>>>>         # behavior. Should probably use StorageDomainAccessError.
>>>>>         if self.rc != 0:
>>>>>             raise exception.MiscFileReadException(self.path, self.rc,
>>>>> self.err)
>>>>>         if not self.err:
>>>>>             raise exception.MiscFileReadException(self.path, "no
>>>>> stats")
>>>>>         stats = self.err.splitlines()[-1]
>>>>>         match = self._PATTERN.match(stats)
>>>>>         if not match:
>>>>>             raise exception.MiscFileReadException(self.path,
>>>>> >                                                 "no match: %r" %
>>>>> stats)
>>>>> E           MiscFileReadException: Internal file read failure:
>>>>> ('/var/tmp/tmpPpV1Rj', "no match: bytearray(b'4 bytes (4 B) copied,
>>>>> 0,000496228 s, 8,1 kB/s')")
>>>>>
>>>>> ../lib/vdsm/storage/check.py:372: MiscFileReadException
>>>>> -------------------------------------------- Captured stdout call
>>>>> --------------------------------------------
>>>>> [<CheckResult path=/var/tmp/tmpPpV1Rj rc=0 err=bytearray(b'0+1 records
>>>>> in\n0+1 records out\n4 bytes (4 B) copied, 0,000496228 s, 8,1 kB/s\n')
>>>>> time=4299448.73 elapsed=0.00 at 0x3b72390>]
>>>>>
>>>>
>>> This is the issue - your dd uses "," as decimal point: 0,000496228
>>> We expect: 0.000496228
>>>
>>> Can you share the output of "locale"? on the machine?
>>>
>>> Here I get:
>>>
>>> $ locale
>>> LANG=en_US.UTF-8
>>> LC_CTYPE="en_US.UTF-8"
>>> LC_NUMERIC="en_US.UTF-8"
>>> LC_TIME="en_US.UTF-8"
>>> LC_COLLATE="en_US.UTF-8"
>>> LC_MONETARY="en_US.UTF-8"
>>> LC_MESSAGES="en_US.UTF-8"
>>> LC_PAPER="en_US.UTF-8"
>>> LC_NAME="en_US.UTF-8"
>>> LC_ADDRESS="en_US.UTF-8"
>>> LC_TELEPHONE="en_US.UTF-8"
>>> LC_MEASUREMENT="en_US.UTF-8"
>>> LC_IDENTIFICATION="en_US.UTF-8"
>>> LC_ALL=
>>>
>>> Maybe we don't configure the test environment properly?
>>>
>>> Dan, do you have any insite on this?
>>>
>>> Nir
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170801/10c4a6f3/attachment.html>