Skip to content

Conversation

sammysun0711
Copy link

Details:

  • Temporarily skip GGUF test in test_llm_pipeline.py on MacOS due to sporadic failure to unblock other PRs.
  • Will follow up to investigate the root cause for MacOS w/ help from validation team.

Tickets

@github-actions github-actions bot added the category: LLM LLM pipeline (stateful, static) label Jun 10, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

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 temporarily skips two GGUF-based pipeline tests on macOS to work around sporadic segmentation faults and allows other PRs to proceed. A follow-up investigation will identify the root cause with the validation team.

  • Added a sys import.
  • Inserted inline pytest.skip guard for darwin platform in two test functions.


import pytest
import torch
import sys
Copy link
Preview

Copilot AI Jun 10, 2025

Choose a reason for hiding this comment

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

Move import sys into the standard library import block (before third-party imports) to comply with PEP8 import ordering.

Copilot uses AI. Check for mistakes.

@Wovchena Wovchena added this pull request to the merge queue Jun 10, 2025
Merged via the queue into openvinotoolkit:master with commit cd199b2 Jun 10, 2025
89 of 91 checks passed
@sammysun0711 sammysun0711 deleted the skip_llm_gguf_macos_test branch June 10, 2025 08:43
sammysun0711 pushed a commit to sammysun0711/openvino.genai that referenced this pull request Jun 20, 2025
…it#2328)

**Details:** 
- Temporarily skip GGUF test in `test_llm_pipeline.py` on MacOS due to
sporadic failure to unblock other PRs.
- Will follow up to investigate the root cause for MacOS w/ help from
validation team.

**Tickets**
- CVS-168882
Wovchena pushed a commit that referenced this pull request Jun 20, 2025
Port skip llm gguf macos test
#2328 in 2025/2
branch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: LLM LLM pipeline (stateful, static)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants