Skip to content

Retrying build coverage step on failure - #1640

Merged
antonwolfy merged 5 commits into
masterfrom
retry_on_failure
Dec 12, 2023
Merged

Retrying build coverage step on failure#1640
antonwolfy merged 5 commits into
masterfrom
retry_on_failure

Conversation

@antonwolfy

@antonwolfyantonwolfy commented Dec 11, 2023

Copy link
Copy Markdown
Contributor

The step with building coverage is failing quite often due to known issue with OCL RT.
The PR propose to automatically rerun the step multiple times until the issue is resolved and a fix is available.

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • If this PR is a work in progress, are you filing the PR as a draft?

@antonwolfyantonwolfy self-assigned this Dec 11, 2023
@antonwolfy
antonwolfyforce-pushed the retry_on_failure branch 2 times, most recently from f222896 to f1255b3CompareDecember 11, 2023 12:51
@github-actions

github-actionsBot commented Dec 11, 2023

Copy link
Copy Markdown
Contributor

View rendered docs @ https://intelpython.github.io/dpnp/index.html

@antonwolfy
antonwolfy marked this pull request as ready for review December 12, 2023 14:11
@antonwolfy
antonwolfy requested review from vlad-perevezentsev and vtavana and removed request for vtavanaDecember 12, 2023 14:11

@vlad-perevezentsevvlad-perevezentsev 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.

This is a very useful change
LGTM!
Many thanks @antonwolfy

@antonwolfy
antonwolfy merged commit 28d3348 into masterDec 12, 2023
@antonwolfy
antonwolfy deleted the retry_on_failure branch December 12, 2023 15:38
Sign up for freeto 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

@antonwolfy@vlad-perevezentsev