[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: clickhouse - Please review



Hi Tobias,

well, having a CI for most of the packages is the goal if it
is technically possible, but is not a dogma. If it is very
difficult or not possible feel free to deactivate some of
the tests or in the worst-case scenario just disable them
completely.

Regards


Anton


Am Di., 1. Nov. 2022 um 11:36 Uhr schrieb Tobias Frost <tobi@debian.org>:
Hi,

I'm currently working on clickhoue for LTS and imported the repository
to the lts-team group [0].

As per git workflow instructions I ask for an exception to enable CI:

I can't get CI working as during linking it seems to go OOM
on the salsa workers. I've tried disabling lto (the package does
uncondtionally on amd64 enable it, already on buster) then the compiling
will succeed.

However, there will be then a test suit error, due to the salsa CI runners
are running a more recent than buster kernel, triggering cpuinfo being noisy
on stderr, [1] which fails the testsuite with message alike: [2]

 11: 00802_daylight_saving_time_shift_backwards_at_midnight:                  [ FAIL ] - having stderror:
 11: Warning in cpuinfo: kernel_max value of 8191 parsed from /sys/devices/system/cpu/kernel_max exceeds platform-default limit 1023

As said I believe this is due to the kernel on the CI is newer than buster's.
Buster's value of /sys/devices/system/cpu/kernel_max is (at leas in my buster VM) 511
while in my sid machine it is 8192.

On said buster VM, the package builds correctly and also the test suite
executes without error.

I expect that the build will fail on the buildds as seem to have recent
kernels, so I anticipate that I need to disable the offending test for the upload

TIA for any feedback you might have…

Cheers,
--
tobi

[0] https://salsa.debian.org/lts-team/packages/ClickHouse
[1] https://bugs.launchpad.net/ubuntu/+source/cpuinfo/+bug/1840847
[2] https://salsa.debian.org/lts-team/packages/ClickHouse/-/jobs/3455769#L1741

Reply to: