Skip to content
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

revisit python discussion #1730

Open
dustymabe opened this issue May 14, 2024 · 8 comments
Open

revisit python discussion #1730

dustymabe opened this issue May 14, 2024 · 8 comments
Labels
-python card related to removing a python dependency area/usability

Comments

@dustymabe
Copy link
Member

dustymabe commented May 14, 2024

I think it may be time that we discuss python exclusion again.

To be clear here:

  1. I'm suggesting we no longer deny python inclusion
  2. I'm not actually advocating for including python, but for being able to include tools that require python.
    • For example, if we somehow in the future removed all tools that require python, or existing tools get rewritten to not require python, then python could drop out at that point (since nothing we include would need it any longer)

Now, why am I suggesting this? I argue that excluding python has caused us more work/pain than if we had just included it.

Now we could decide to implement a system python (where only files we ship in the OSTree could be executed with python), but we never made progress on investigating and implementing that IIUC. I think my suggestion is that we just stop resisting it and include it without new investigations to figure out if we can do something special here.

@dustymabe dustymabe added the meeting topics for meetings label May 14, 2024
@LorbusChris
Copy link
Contributor

Personally, I feel indifferent about the inclusion but I'd oppose the system python option, as it seems like the worst of both worlds to me (size increase && python still not universally usable).

@travier travier added -python card related to removing a python dependency area/usability labels May 14, 2024
@travier
Copy link
Member

travier commented May 14, 2024

@cgwalters
Copy link
Member

Also related is #1726 - when fcos derives from the base bootc image, then we'd pick up python by default.

Also integrating with things like ansible hasn't really been able to work well

Right, it is intentionally included there, see also https://docs.fedoraproject.org/en-US/bootc/dynamic-reconfiguration/

@jbtrystram
Copy link
Contributor

This was discussed at length today in the FCOS community meeting.

It was decided to think again and keep discussing next week :

!info take some time to consider the python discussion from this week's meeting (see transcript) and we will discuss next week

See the full meeting log : https://meetbot.fedoraproject.org/meeting-1_matrix_fedoraproject-org/2024-05-15/fedora-coreos-meeting.2024-05-15-16.29.log.html

@dustymabe
Copy link
Member Author

Another ancillary benefit if python were to get included is that we could take advantage of the new OSBuild buildroot support that was added in coreos/coreos-assembler@0f8b35a - of course this wouldn't be reason enough on it's own to include python, but would be a benefit if it were.

@travier
Copy link
Member

travier commented May 22, 2024

Also related is #1726 - when fcos derives from the base bootc image, then we'd pick up python by default.

I would like to note here that I don't think that's an argument for including Python. If the Fedora CoreOS working group and community decides to keep Python out of Fedora CoreOS, we'll have to figure out how to keep it out when we rebase on top of bootc manifests/images.

@c4rt0
Copy link
Member

c4rt0 commented May 29, 2024

This was once more discussed today in the FCOS community meeting.

We finally reached a consensus:

!agreed We will no longer exclude packages that require python from Fedora CoreOS. If we decide that we want the package in Fedora CoreOS we will include it, even if it's dependency chain includes python.

See the full meeting log here.

@c4rt0 c4rt0 removed the meeting topics for meetings label May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-python card related to removing a python dependency area/usability
Projects
None yet
Development

No branches or pull requests

6 participants