Skip to content

Pin pillow to avoid allocator mismatch#40248

Merged
sf1919 merged 1 commit intomainfrom
pin_pillow_less_than_twelve
Oct 30, 2025
Merged

Pin pillow to avoid allocator mismatch#40248
sf1919 merged 1 commit intomainfrom
pin_pillow_less_than_twelve

Conversation

@thomashampson
Copy link
Copy Markdown
Contributor

@thomashampson thomashampson commented Oct 30, 2025 •
edited
Loading

Description of work

The newest version of pillow (>12.0.0) was built against zlib-ng (see here). For some reason this is causing a segfault in jemalloc. This is the cause of our recent CI failures. The long term solution is to move away from jemalloc, since it is no longer maintained. The short term solution is to pin pillow back.

Closes #40244

To test:

Reviewer

Your comments will be used as part of the gatekeeper process. Comment clearly on what you have checked and tested during your review. Provide an audit trail for any changes requested.

As per the review guidelines:

  • Is the code of an acceptable quality? (Code standards/GUI standards)
  • Has a thorough functional test been performed? Do the changes handle unexpected input/situations?
  • Are appropriately scoped unit and/or system tests provided?
  • Do the release notes conform to the guidelines and describe the changes appropriately?
  • Has the relevant (user and developer) documentation been added/updated?
  • If the PR author isn’t in the mantid-developers or mantid-contributors teams, add a review comment rerun ci to authorize/rerun the CI

Gatekeeper

As per the gatekeeping guidelines:

  • Has a thorough first line review been conducted, including functional testing?
  • At a high-level, is the code quality sufficient?
  • Are the base, milestone and labels correct?

@thomashampson thomashampson added the DevOps Issues and pull requests related to DevOps label Oct 30, 2025
@thomashampson thomashampson added this to the Release 6.15 milestone Oct 30, 2025
@MialLewis
Copy link
Copy Markdown
Contributor

MialLewis commented Oct 30, 2025

Issue regarding long term fix: #40249

@MialLewis MialLewis marked this pull request as ready for review October 30, 2025 14:01
@sf1919 sf1919 self-assigned this Oct 30, 2025
@sf1919 sf1919 enabled auto-merge (squash) October 30, 2025 14:03
@sf1919 sf1919 merged commit 7c38d77 into main Oct 30, 2025
11 checks passed
@sf1919 sf1919 deleted the pin_pillow_less_than_twelve branch October 30, 2025 14:35
peterfpeterson pushed a commit to peterfpeterson/mantid that referenced this pull request Oct 30, 2025
@thomashampson @peterfpeterson
Pin pillow to avoid allocator mismatch (mantidproject#40248)
eb74b87 
### Description of work

Pin pillow to avoid allocator mismatch

Closes mantidproject#40244
@peterfpeterson peterfpeterson mentioned this pull request Oct 30, 2025
Merged
peterfpeterson added a commit that referenced this pull request Oct 30, 2025
@peterfpeterson @thomashampson
Fix jenkins errors (#40251)
ead1b01 
This pulls the following into `ornl-next`
* #40248
* #40127

---------

Co-authored-by: thomashampson <thomas.hampson@stfc.ac.uk>
@hmaarrfk
Copy link
Copy Markdown

hmaarrfk commented Nov 2, 2025

If you think this can cause "incompatibilities" beyond your particular usecase, please comment:
conda-forge/conda-forge.github.io#2638

@thomashampson thomashampson mentioned this pull request Nov 5, 2025
Open
@thomashampson thomashampson mentioned this pull request Feb 19, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MialLewis MialLewis MialLewis approved these changes

Assignees

@sf1919 sf1919

Labels

DevOps Issues and pull requests related to DevOps

Projects

None yet

Milestone

Release 6.15

Development

Successfully merging this pull request may close these issues.

Investigate Linux PR CI test failures

4 participants

@thomashampson @MialLewis @hmaarrfk @sf1919