Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Debuging Github CI #4603

Closed

Conversation

chenyukang
Copy link
Collaborator

What problem does this PR solve?

Spec SendLargeCyclesTxToRelay may be failing in MacOS CI, but it's not reproducible on local dev machine.

There is a low chance for such a scenario, when a worker is wake up, the current status is not Resume, if we return directly the job is left in the queue.

What is changed and how it works?

Renotify verify worker when status is in Suspend.

What's Changed:

Related changes

  • PR to update owner/repo:
  • Need to cherry-pick to the release branch

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code ci-runs-only: [ quick_checks,linters ]

Side effects

  • Performance regression
  • Breaking backward compatibility

Release note

None: Exclude this PR from the release note.

@chenyukang chenyukang requested a review from a team as a code owner August 22, 2024 00:14
@chenyukang chenyukang requested review from zhangsoledad and removed request for a team August 22, 2024 00:14
@chenyukang chenyukang force-pushed the yukang-fix-verify-ci-issue branch 5 times, most recently from 54c6a93 to 340ca9b Compare August 22, 2024 08:52
@chenyukang chenyukang closed this Aug 22, 2024
@chenyukang chenyukang changed the title [WIP] renotify verify worker when status is not in Resume [WIP] Debuging Github CI Aug 22, 2024
@chenyukang chenyukang reopened this Aug 22, 2024
@chenyukang chenyukang force-pushed the yukang-fix-verify-ci-issue branch 2 times, most recently from 9f91194 to b14eccc Compare August 23, 2024 03:58
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.

1 participant