Skip to content

Conversation

phlogistonjohn
Copy link
Collaborator

@phlogistonjohn phlogistonjohn commented Oct 7, 2025

Depends on: #176

I don't always like 'dueling prs' but I needed to try some stuff out and this was the simplest way.

Overlaps with #173 and #175 - but I can't help it because I wanted to test adding a new CI job.

@phlogistonjohn phlogistonjohn changed the title WIP centos 10 Add centos 10 CI job Oct 7, 2025
@phlogistonjohn phlogistonjohn marked this pull request as ready for review October 7, 2025 18:39
Copy link

dpulls bot commented Oct 8, 2025

🎉 All dependencies have been resolved !

Copy link
Contributor

@andrewschoen andrewschoen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. I should be able to incorporate the changes from #175 into this pretty easily.

@andrewschoen
Copy link
Contributor

This looks good to me. I should be able to incorporate the changes from #175 into this pretty easily.

#175 might not be needed though as python3dist(inotify-simple) might pull in what I need.

Start to prepare for centos stream10 build support by moving
things that not all versions support out of the packages list
and adding a set of distro-conditional actions in.

Signed-off-by: John Mulligan <[email protected]>
@phlogistonjohn
Copy link
Collaborator Author

might not be needed though as python3dist(inotify-simple) might pull in what I need.

It should. Unless rhel is doing something very odd, it should work similarly to how the COPR enables the package on centos 10.

@phlogistonjohn
Copy link
Collaborator Author

Speaking of the COPR - @anoopcs9 do you want to request an EPEL 10 package for python-inotify_simple or shall I?

@anoopcs9
Copy link
Collaborator

anoopcs9 commented Oct 8, 2025

Speaking of the COPR - @anoopcs9 do you want to request an EPEL 10 package for python-inotify_simple or shall I?

Done. bz #2402542

Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to ask last time.

So we are OK to not have py39-mypy on CentOS Stream 10? How long do we plan to carry this forward with CentOS Stream 9?

@phlogistonjohn
Copy link
Collaborator Author

phlogistonjohn commented Oct 8, 2025

I forgot to ask last time.

So we are OK to not have py39-mypy on CentOS Stream 10? How long do we plan to carry this forward with CentOS Stream 9?

We have it explicitly on fedora and implicitly on centos 9, so it has coverage, just not on in the c10 job. (and that applies to py39, the unit tests env, as well as py39-mypy for type checking)

I figure we can continue building/testing on centos stream 9 until it becomes an annoyance. Possibly because we want to use a python 3.10 or later feature :-) and it would also have to be after we switch the base images of samba-container (devbuilds and/or whatever we are pairing with ceph) to something EL10 based.

Make sense?

Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to ask last time.
So we are OK to not have py39-mypy on CentOS Stream 10? How long do we plan to carry this forward with CentOS Stream 9?

We have it explicitly on fedora and implicitly on centos 9, so it has coverage, just not on in the c10 job. (and that applies to py39, the unit tests env, as well as py39-mypy for type checking)

Sorry, I didn’t get the part that says "implicitly on centos 9" in relation to the py39 test env.

I figure we can continue building/testing on centos stream 9 until it becomes an annoyance. Possibly because we want to use a python 3.10 or later feature :-) and it would also have to be after we switch the base images of samba-container (devbuilds and/or whatever we are pairing with ceph) to something EL10 based.

Make sense?

Well, I don't mind dropping the py39 test env and have something which runs on all distro versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants