Skip to content
This repository was archived by the owner on Oct 10, 2025. It is now read-only.

ci: fix multiplatform builds - #5382

Merged
sdht0 merged 4 commits into
masterfrom
fix-multiplatform-ci
May 12, 2025
Merged

sdht0 merged 4 commits into
masterfrom
fix-multiplatform-ci

Conversation

@sdht0

@sdht0 sdht0 commented May 9, 2025

Copy link
Copy Markdown
Contributor

Just a few builds fail now. Just 1 build fails now. Made builds relwithdebinfo to share build output and reduce build times.

  • macos-13 -> macos-15
  • Removed windows-2019, builds were all failing
  • fedora-38 & fedora-39 -> fedora-41 & fedora-42
  • ubuntu-24.10 -> ubuntu-25.04
  • Moved to torch 2.6.0 on all platforms
  • Reorganized builds to share artifacts -> reduced runtime by about 30min on most platforms
  • Removed PIP_BREAK_SYSTEM_PACKAGES

Just rockylinux-8's tests are failing now:

 The following tests FAILED:
	268 - OptimizerTest.SingleNodeTwoHopJoins (Failed)
	2046 - e2e_test_function~gds~weighted_shortest.WeightedShortestPath (Failed)

@sdht0
sdht0 requested a review from mewim May 9, 2025 19:25
@codecov

codecov Bot commented May 9, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.85%. Comparing base (8c69a62) to head (500553c).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5382      +/-   ##
==========================================
+ Coverage   86.83%   86.85%   +0.01%     
==========================================
  Files        1414     1414              
  Lines       62061    61977      -84     
  Branches     7615     7591      -24     
==========================================
- Hits        53891    53828      -63     
+ Misses       7995     7974      -21     
  Partials      175      175              

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

This comment was marked as outdated.

@sdht0
sdht0 force-pushed the fix-multiplatform-ci branch from 9872845 to 8e6c986 Compare May 11, 2025 22:44
@sdht0

sdht0 commented May 11, 2025

Copy link
Copy Markdown
Contributor Author

Before:

    archlinux:
        Test: ❌
        Python test: ❌
    debian-sid:
        Test: ❌
        Python test: ❌
    rockylinux-8:
        Test: ❌
    rockylinux-9:
        Build: ❌
        Test: ❌
        C and C++ examples: ❌
        Python test: ❌
        Node.js test: ❌
        Java test: ❌
        Rust test: ❌
        Rust example: ❌
    ubuntu-22.04:
        Build: ❌
        Test: ❌
        C and C++ examples: ❌
        Python test: ❌
        Node.js test: ❌
        Java test: ❌
        Rust test: ❌
        Rust example: ❌
    windows-2019:
        Test: ❌
        Python test: ❌
        Node.js test: ❌
        Java test: ❌
        Rust test: ❌
    windows-2022:
        Java test: ❌

Now:

    rockylinux-8:
        Test: ❌

@github-actions

This comment was marked as outdated.

@sdht0
sdht0 marked this pull request as draft May 12, 2025 20:16
@sdht0
sdht0 force-pushed the fix-multiplatform-ci branch from 8e6c986 to 500553c Compare May 12, 2025 22:43
@sdht0
sdht0 marked this pull request as ready for review May 12, 2025 22:45
@sdht0
sdht0 merged commit 3a7c878 into master May 12, 2025
@sdht0
sdht0 deleted the fix-multiplatform-ci branch May 12, 2025 22:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants