[SSSD] [PATCH] CI: Use default config for mock build
Nikolai Kondrashov
Nikolai.Kondrashov at redhat.com
Fri Sep 19 13:15:25 UTC 2014
Hi Lukas,
On 09/19/2014 08:51 AM, Lukas Slebodnik wrote:
> The mock config with name default is usually symbolic link
> to the configuration file of local architecture. The side effect
> of this patch is that we will not try to rebuild on old architectures
> src.rpm for new architectures(fedora). It caused issues with mock tmpfs
> plugin.
There is one other issue. This way we won't run RHEL6 mock builds locally
normally, as, AFAIK, people usually don't work on RHEL6.
Still, we would have our CI servers.
> + stage mock-build mock_privileged_deps "default" \
> + --resultdir ci-mock-result \
> rpmbuild/SRPMS/*.src.rpm
Would it be better to perhaps name the stage just "mock"? "mock-build" seems
redundant.
Otherwise, the patch looks and works fine to me.
Thank you, Lukas.
There is still the problem of Clang running on all hosts. I think it will be
better to run it just on one. OTOH, developers might run CI on different
distros and Clang might produce different issues there. I see that on Fedora
Rawhide at least. Anyway, we can live with it for now and solve it later.
Nick
More information about the sssd-devel
mailing list