Skip to content

Fix ollama segfault#1161

Merged
acroca merged 2 commits into
dapr:mainfrom
acroca:ollama-segfault
Jul 16, 2026
Merged

Fix ollama segfault#1161
acroca merged 2 commits into
dapr:mainfrom
acroca:ollama-segfault

Conversation

@acroca

@acroca acroca commented Jul 16, 2026

Copy link
Copy Markdown
Member

Description

We're having multiple issues with ollama segfaults, like this one
In this PR:

  • Pin ollama to the current latest version
  • Add a suggested workaround for the segfaults issues.

I've run the validate step 26 times in the CI and none of the runs failed due to ollama sigfault.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the CI workflow to reduce Ollama-related segfaults by pinning the Ollama binary to a specific upstream release and applying a temporary workaround intended to avoid crashing CPU backends on certain GitHub runners.

Changes:

  • Pin Ollama download URL to v0.32.0 instead of using the latest release asset.
  • Remove the Sapphire Rapids CPU backend shared library to force a safer fallback backend during CI runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/run-tests.yaml Outdated
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.78%. Comparing base (a9445f9) to head (7bb71c0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1161   +/-   ##
=======================================
  Coverage   82.78%   82.78%           
=======================================
  Files         123      123           
  Lines       10127    10127           
=======================================
  Hits         8384     8384           
  Misses       1743     1743           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@acroca
acroca force-pushed the ollama-segfault branch from 5391b34 to 8744a7d Compare July 16, 2026 07:00
Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca
acroca force-pushed the ollama-segfault branch from 8744a7d to 13e5abb Compare July 16, 2026 07:00
@acroca
acroca marked this pull request as ready for review July 16, 2026 07:00
@acroca
acroca requested review from a team as code owners July 16, 2026 07:00
@acroca
acroca added this pull request to the merge queue Jul 16, 2026
Merged via the queue into dapr:main with commit 4ef193b Jul 16, 2026
22 of 24 checks passed
@acroca
acroca deleted the ollama-segfault branch July 16, 2026 13:24
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