Skip to content

[Due for payment 2025-05-13] Investigate workflow job failing on main: test / test (job 2) #61255

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

Closed
github-actions bot opened this issue May 1, 2025 · 6 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Weekly KSv2 Workflow Failure

Comments

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

🚨 Failure Summary 🚨:

  • 📋 Job Name: test / test (job 2)
  • 🔧 Failure in Workflow: Process new code merged to main
  • 🔗 Triggered by PR: PR Link
  • 👤 PR Author: @FitseTLT
  • 🤝 Merged by: @blimpich
  • 🐛 Error Message:
    failure: Process completed with exit code 1.

⚠️ Action Required ⚠️:

🛠️ A recent merge appears to have caused a failure in the job named test / test (job 2).
This issue has been automatically created and labeled with Workflow Failure for investigation.

👀 Please look into the following:

  1. Why the PR caused the job to fail?
  2. Address any underlying issues.

🐛 We appreciate your help in squashing this bug!

@blimpich
Copy link
Contributor

blimpich commented May 1, 2025

Probably a flake: https://expensify.slack.com/archives/C03V9A4TB/p1746119273412529?thread_ts=1746118626.579799&cid=C03V9A4TB

Strange though that it would flake on a brand new test. That would be a strange coincidence. Looks like contributor is looking into it here: #59551 (comment)

@FitseTLT
Copy link
Contributor

FitseTLT commented May 1, 2025

@blimpich the cause is createRandomReportAction setting a random pending Action here so it will fail if it is set to DELETE

pendingAction: rand(Object.values(CONST.RED_BRICK_ROAD_PENDING_ACTION)),

I have suggested previously that we should avoid this random functions but no one took action 😭
I have raised the pr

@blimpich
Copy link
Contributor

blimpich commented May 1, 2025

Sounds good! Just merged your PR, thank you @FitseTLT for speedy action! 🙏🙂

@blimpich
Copy link
Contributor

blimpich commented May 1, 2025

Closing this as this was handled.

@blimpich blimpich closed this as completed May 1, 2025
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels May 6, 2025
@melvin-bot melvin-bot bot changed the title Investigate workflow job failing on main: test / test (job 2) [Due for payment 2025-05-13] Investigate workflow job failing on main: test / test (job 2) May 6, 2025
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label May 6, 2025
Copy link

melvin-bot bot commented May 6, 2025

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented May 6, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.39-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2025-05-13. 🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Weekly KSv2 Workflow Failure
Projects
None yet
Development

No branches or pull requests

2 participants