Skip to content

Fix : Release bug#140

Merged
ferrandp merged 1 commit intodevelopfrom
fix/ignoreBreakSystemPackages
Oct 16, 2025
Merged

Fix : Release bug#140
ferrandp merged 1 commit intodevelopfrom
fix/ignoreBreakSystemPackages

Conversation

@camilosierrar
Copy link
Contributor

Ubuntu latest introduced a crash on the release pipeline.
It's an error that appears when using pip install because it might interferes with other python directories for the user.

Since this is in the context of a short lived docker container I decided to just ignore this error adding the corresponding flag.

@camilosierrar
Copy link
Contributor Author

@ferrandp Y a moyen de merger ça ?
Ou peut-être de me donner les droits ?

@camilosierrar camilosierrar force-pushed the fix/ignoreBreakSystemPackages branch 2 times, most recently from 7a34864 to b180069 Compare October 15, 2025 13:15
@camilosierrar camilosierrar force-pushed the fix/ignoreBreakSystemPackages branch from b180069 to 6b7ae2e Compare October 16, 2025 07:44
@camilosierrar camilosierrar force-pushed the fix/ignoreBreakSystemPackages branch from 6b7ae2e to 3aa22d4 Compare October 16, 2025 08:28
@ferrandp ferrandp merged commit 1b1193e into develop Oct 16, 2025
26 checks passed
@ferrandp ferrandp deleted the fix/ignoreBreakSystemPackages branch October 16, 2025 08:45
@WestFR
Copy link

WestFR commented Nov 28, 2025

Now, that means we can used custom python script directly for this image?

@camilosierrar
Copy link
Contributor Author

Now, that means we can used custom python script directly for this image?

Python is included on the ubuntu-latest Docker image which is the base of ours.

We use it as a dependency of the google cloud CLI which is added to support firebase (see README). When upgrading to latest version of ubuntu it introduced a crash because of PEP 668. I fixed this issue in this PR to be compliant.

This PR isn't supposed to add python support. Though I guess we already supported it somehow, it isn't a first class feature of the image.

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.

4 participants