Bug 45515 - Во время сборки zstd на riscv64 не проходят некоторые тесты по таймау
Summary: Во время сборки zstd на riscv64 не проходят некоторые тесты по таймау
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: zstd (show other bugs)
Version: unstable
Hardware: riscv64 Linux
: P5 normal
Assignee: placeholder@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-03-09 13:01 MSK by DVoropaev
Modified: 2023-04-07 00:51 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description DVoropaev 2023-03-09 13:01:27 MSK
Во время сборки на riscv64 не проходят некоторые тесты по таймау:  

Traceback (most recent call last):
  File "/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/./cli-tests/run.py", line 734, in <module>
    success = run_tests(tests, opts)
  File "/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/./cli-tests/run.py", line 601, in run_tests
    tests[test_case.name] = test_case.run()
  File "/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/./cli-tests/run.py", line 285, in run
    return self.analyze()
  File "/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/./cli-tests/run.py", line 275, in analyze
    self._join_test()
  File "/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/./cli-tests/run.py", line 330, in _join_test
    (stdout, stderr) = self._test_process.communicate(timeout=self._opts.timeout)
  File "/usr/lib64/python3.10/subprocess.py", line 1154, in communicate
    stdout, stderr = self._communicate(input, endtime, timeout)
  File "/usr/lib64/python3.10/subprocess.py", line 2006, in _communicate
    self._check_timeout(endtime, orig_timeout, stdout, stderr)
  File "/usr/lib64/python3.10/subprocess.py", line 1198, in _check_timeout
    raise TimeoutExpired(
subprocess.TimeoutExpired: Command '['/usr/src/RPM/BUILD/zstd-1.5.4-alt2/tests/cli-tests/compression/window-resize.sh']' timed out after 60 seconds


Прошу принять следующие изменение, увеличивающие таймаут: http://git.altlinux.ru/people/voropaevdmtr/packages/?p=zstd.git;a=commit;h=8cfe4aed7ae9b66050cbcd1506fe27837b602a23
#316416 TESTED #1 [test-only] sisyphus zstd.git=1.5.4-alt3
Comment 1 Dmitry V. Levin 2023-03-09 13:32:14 MSK
Вы считаете, что изменение таймаута лучше держать в пакете, или, может быть, лучше предложить это апстриму?
Comment 2 Ivan A. Melnikov 2023-03-09 13:59:20 MSK
Я думаю, что пакеты сталкиваются с более жёсткими условиями сборки, чем код на машинах разработчиков, поэтому повышение таймаута в пакете по сравнению с апстрмным, если оно нужно, оправданно.

Хотя можно, конечно, его с апстримом обсудить.
Comment 3 Dmitry V. Levin 2023-03-09 14:33:07 MSK
У zstd вполне вменяемый апстрим, так что имеет смысл как минимум рассказать им, что нынешнего таймаута недостаточно для riscv64. Может быть, они предложат какой-нибудь другой подход.
Comment 4 DVoropaev 2023-03-09 15:51:37 MSK
сделал pull request
https://github.com/facebook/zstd/pull/3540
Comment 5 DVoropaev 2023-04-06 13:35:18 MSK
(Ответ для Dmitry V. Levin на комментарий #3)
> У zstd вполне вменяемый апстрим, так что имеет смысл как минимум рассказать
> им, что нынешнего таймаута недостаточно для riscv64. Может быть, они
> предложат какой-нибудь другой подход.

мой pull-request приняли
https://github.com/facebook/zstd/pull/3540/commits/b7080f4c67bdb9d190bda529f7309e34fb990b23
изменения попали в новый релиз v1.5.5
Могу сделать таску с новой версией в сизиф
Comment 6 Dmitry V. Levin 2023-04-06 14:02:29 MSK
Отлично, попробую тогда собрать новую версию в Сизиф.
Comment 7 Repository Robot 2023-04-07 00:51:55 MSK
zstd-1.5.5-alt1 -> sisyphus:

 Tue Apr 04 2023 Dmitry V. Levin <ldv@altlinux> 1.5.5-alt1
 - 1.5.4 -> 1.5.5 (closes: #45515).