SSSD / sssd-ci-containers

Setup containerized environment for testing and developing SSSD.
GNU General Public License v3.0
10 stars 21 forks source link

Add stream-10-devel container #89

Closed alexey-tikhonov closed 6 months ago

alexey-tikhonov commented 7 months ago

Currently c10s support is being added with virt_smartcard: No until required deps are made available.

alexey-tikhonov commented 7 months ago

@pbrezina , https://copr.fedorainfracloud.org/coprs/g/sssd/ci-deps/ got 'Centos-stream 10' but not 'EPEL 10'. Why / how to enable EPEL10?

pbrezina commented 7 months ago

Copr does not have epel10 chroot available at this moment, we will need to wait.

pbrezina commented 7 months ago

Or we can switch to centos10 when running dnf copr enable so we don't depend on epel.

alexey-tikhonov commented 7 months ago

Or we can switch to centos10 when running dnf copr enable so we don't depend on epel.

In centos10 playbooks only?

pbrezina commented 7 months ago

Or we can switch to centos10 when running dnf copr enable so we don't depend on epel.

In centos10 playbooks only?

Not necessarily, we can do it for all. It's just that copr enable defaults to epel on rhel/centos (maybe to save resources, I don't know). But we build it on all available chroots and the packages will be identical.

alexey-tikhonov commented 7 months ago

- No package tmate available.

alexey-tikhonov commented 7 months ago

hm... https://copr.fedorainfracloud.org/coprs/g/sssd/ci-deps/packages/ :: 'Last build status' is pretty "red"...

alexey-tikhonov commented 7 months ago

Btw, why do we need 'tmate' and 'lcov' at all?

pbrezina commented 7 months ago

Looks like there is issue with ppc64 - that will resolve on its own, I assume. tmate fails on all centos10 with No matching package to install: 'msgpack-devel >= 1.1.8' which is weird, since msgpack was built successfully on all but ppc64.

lcov was used for intg tests, I think we can drop it now.

tmate is useful when you want to share issues with team mate, but it is not needed for CI. I will look into it tomorrow and see if we can make it work. If not, we can drop it.

pbrezina commented 7 months ago

Ok, tmate is bow built. ppc64 fails everywhere and it looks like some issue in c10s, I don't think we can do anything but wait. The issue with tmate (and probably other packages) was order - tmate has to be build after msgpack, unfortunately, copr does not have dependency ordering afaik.

pbrezina commented 7 months ago

I rebuilt packages in order and the only one still failing is libfaketime, I don't remember what we use it for though. That one simply exits with

+ FAKETIME_COMPILE_CFLAGS=BOGUS
+ '[' BOGUS == BOGUS ']'
+ echo 'SHOULD NEVER REACH HERE ... YOU HAVE AN UNTESTED VERSION+ARCH, see rpm spec for details ... ABORT'
+ exit 1
error: Bad exit status from /var/tmp/rpm-tmp.1rnZcM (%build)
    %patchN is deprecated (3 usages found), use %patch N (or %patch -P N)
    Bad exit status from /var/tmp/rpm-tmp.1rnZcM (%build)
pbrezina commented 7 months ago

Btw, you can disable other distros for testing purpose to make pr ci faster. See: https://github.com/SSSD/sssd-ci-containers/pull/91

alexey-tikhonov commented 7 months ago

'virtualsmartcard' also fails : https://download.copr.fedorainfracloud.org/results/@sssd/ci-deps/centos-stream-10-x86_64/07325915-virtualsmartcard/builder-live.log.gz

UPD: https://github.com/frankmorgner/vsmartcard/issues/274

alexey-tikhonov commented 7 months ago

ibfaketime, I don't remember what we use it for though

It's for unit tests: https://github.com/SSSD/sssd/blob/7293eeea53458d1596262ee0f6593e08c38af7db/src/tests/test_CA/Makefile.am#L101

But if package is missing it's not a hardfail, just test will be skipped.

Btw, it's in https://kojihub.stream.centos.org/kojifiles/repos/c10s-build/latest/x86_64/pkglist list, not sure why it fails to install.

alexey-tikhonov commented 7 months ago

It's also unclear where 'vpcd' - https://github.com/SSSD/sssd-ci-containers/blob/master/src/ansible/roles/packages/tasks/Fedora.yml#L170 - should be installed from.

UPD: part of 'virtualsmartcard'

alexey-tikhonov commented 7 months ago

- No package mock available.

UPD: I think its use was removed in https://github.com/SSSD/sssd/commit/5def61fb9d434f92687842afabdfc26b7625a36e and not needed anymore

alexey-tikhonov commented 7 months ago

No package fakeroot available It's missing in https://kojihub.stream.centos.org/kojifiles/repos/c10s-build/latest/x86_64/pkglist