search

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.43k stars 2.8k forks source link

Investigate workflow job failing on main: e2ePerformanceTests / Run E2E tests in AWS device farm #44844

Closed github-actions[bot] closed 2 months ago

github-actions[bot] commented 3 months ago

šŸšØ Failure Summary šŸšØ:

āš ļø Action Required āš ļø:

šŸ› ļø A recent merge appears to have caused a failure in the job named e2ePerformanceTests / Run E2E tests in AWS device farm. 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!

dangrous commented 3 months ago

ugggh this is that same one. Checking with margelo to see if they have ideas

dangrous commented 3 months ago

Forgot to link: https://expensify.slack.com/archives/C035J5C9FAP/p1720107180975539?thread_ts=1718788565.696269&cid=C035J5C9FAP

Given that this has come up a number of times and we've closed each time, I'm going to demote to daily, but continue to investigate.

dangrous commented 2 months ago

As per Slack, there's not really enough evidence here to find a cause/solution. We're going to wait for additional similar issues to crop up and then keep on from there! Demoting to weekly while we wait.

dangrous commented 2 months ago

Looks like we got at least one more - https://github.com/Expensify/App/issues/45223

dangrous commented 2 months ago

This might be the same as the one I just linked - #46129 - and if so the solution to that should fix both!

dangrous commented 2 months ago

Checking whether the solution for #46129 will also fix this.

dangrous commented 2 months ago

Sounds like it's a different solution - https://github.com/Expensify/App/issues/46129#issuecomment-2256099001 - but has been merged! I'm going to close this out.