Skip to content

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Aug 26, 2025

What changes were proposed in this pull request?

This PR aims to add integration-test-ubuntu GitHub Action job.

Why are the changes needed?

We had integration-test-linux before and moved it to daily job due to the flakiness.

This is the current daily linux integration test coverage.

Since all integration test (on MacOS) jobs are failing currently, this PR aims to bring Linux integration test job back to main CI to have an integration test coverage.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Newly added linux integration test job should pass.

Was this patch authored or co-authored using generative AI tooling?

No.

@viirya
Copy link
Member

viirya commented Aug 26, 2025

Looks like all mac related integration tests are failed.

@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Aug 26, 2025

Yes, @viirya . It suddenly starts to happen on main branch Today.

Screenshot 2025-08-25 at 22 57 58

Today, GitHub Action runner is changed.

So, I tried to switch MacOS or Xcode back, but there was no luck.

This is the last resort to recover the integration test coverage via Linux (as I described in the PR description) while investigating MacOS issue.

@dongjoon-hyun
Copy link
Member Author

As we can see in the commit result, the MacOS failures are irrelevant to this repository code itself. I reverted SPARK-53371, the failure situation is identical.

In addition, as we see in Linux integration tests, this repository code works functionally without any issue. I hope to merge this PR to recover the integration test coverage first.

@dongjoon-hyun
Copy link
Member Author

I'm going to merge this because the failures are irrelevant to this new GitHub Action job which passed already.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-53379 branch August 26, 2025 06:16
@viirya
Copy link
Member

viirya commented Aug 26, 2025

Okay. Looks good.

@dongjoon-hyun
Copy link
Member Author

Thank you~

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.

2 participants