-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add stream-10-devel container #89
Conversation
@pbrezina , https://copr.fedorainfracloud.org/coprs/g/sssd/ci-deps/ got 'Centos-stream 10' but not 'EPEL 10'. |
Copr does not have epel10 chroot available at this moment, we will need to wait. |
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. |
|
hm... https://copr.fedorainfracloud.org/coprs/g/sssd/ci-deps/packages/ :: 'Last build status' is pretty "red"... |
Btw, why do we need 'tmate' and 'lcov' at all? |
Looks like there is issue with ppc64 - that will resolve on its own, I assume. tmate fails on all centos10 with 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. |
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. |
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
|
Btw, you can disable other distros for testing purpose to make pr ci faster. See: #91 |
It's for unit tests: 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. |
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' |
UPD: I think its use was removed in SSSD/sssd@5def61f and not needed anymore |
|
Currently c10s support is being added with
virt_smartcard: No
until required deps are made available.