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

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

Open
github-actions bot opened this issue Dec 24, 2024 · 24 comments

Comments

@github-actions
Copy link
Contributor

🚨 Failure Summary 🚨:

  • 📋 Job Name: e2ePerformanceTests / Run E2E tests in AWS device farm
  • 🔧 Failure in Workflow: Process new code merged to main
  • 🔗 Triggered by PR: PR Link
  • 👤 PR Author: @christianwen
  • 🤝 Merged by: @iwiznia
  • 🐛 Error Message:
    warning: No matching artifact of type LOG found for debug.log, skipping
    failure: Test run failed after 3090 seconds with: undefined. Timeout is set to 7200
    warning: No matching artifact of type LOG found for debug.log, skipping
    failure: Test run failed after 2915 seconds with: undefined. Timeout is set to 7200
    failure: Process returned exit code 1
    Launched as "/home/runner/runners/2.321.0/externals/node20/bin/node /home/runner/work/_actions/Wandalen/wretry.action/v3.5.0_js_action/src/Runner.mjs /home/runner/work/_actions/Wandalen/aws-devicefarm/test-application/dist/index.js"
    Launched at "/home/runner/work/App/App"
    Attempts exhausted, made 2 attempts :
    Attempt Some initial fixes and code style updates #1 started at : 18:06:37 GMT+0000 (Coordinated Universal Time)
    Attempt Fix spaces #2 started at : 18:59:47 GMT+0000 (Coordinated Universal Time)
    failure: Process completed with exit code 1.

⚠️ 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!

@iwiznia
Copy link
Contributor

iwiznia commented Dec 24, 2024

@christianwen @DylanDylann got this issue assigned, should I have not merged the PR #51172?

@christianwen
Copy link
Contributor

christianwen commented Dec 25, 2024

@iwiznia I think the error log here is unrelated to my PR and I can run this job on my local device.

Warning: No matching artifact of type LOG found for debug.log, skipping
Error: Test run failed after 2915 seconds with: undefined. Timeout is set to 7200

I see there're 2 PRs that fails job:

And re-run successfully. Should we give it another try?

Copy link

melvin-bot bot commented Dec 25, 2024

📣 @christianwen! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@iwiznia
Copy link
Contributor

iwiznia commented Dec 26, 2024

And re-run successfully. Should we give it another try?

It failed again https://github.com/Expensify/App/actions/runs/12484590267/job/34890814512

@iwiznia
Copy link
Contributor

iwiznia commented Jan 3, 2025

@christianwen @DylanDylann can you take a look please?

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jan 3, 2025
@iwiznia
Copy link
Contributor

iwiznia commented Jan 6, 2025

Bump @christianwen @DylanDylann

@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2025
@DylanDylann
Copy link
Contributor

I just checked it but still struggle with this one. @christianwen Do you have any input on it?

@christianwen
Copy link
Contributor

I have no idea about this job, can we run it locally @iwiznia ?

@iwiznia
Copy link
Contributor

iwiznia commented Jan 8, 2025

I am not sure honestly, it seems it runs in aws device farm, but maybe you can run it locally.

@DylanDylann
Copy link
Contributor

Hmmm, It is so strange to me. I don't see any valuable log that can help investigate this. @iwiznia Could you ask someone who is familiar with this flow to support us?

@iwiznia
Copy link
Contributor

iwiznia commented Jan 9, 2025

Oh I thought you were familiar with it, since you said this. Can you post it in slack to get help from people more familiar with this?

@DylanDylann
Copy link
Contributor

@iwiznia If that is the reason, I think we can ignore it because it is intentional. Is there any problem with leaving it?

@iwiznia
Copy link
Contributor

iwiznia commented Jan 10, 2025

Ignore what? The fact that we made this fail? I don't think we can or it will keep failing, no?

@melvin-bot melvin-bot bot added the Overdue label Jan 13, 2025
@DylanDylann
Copy link
Contributor

DylanDylann commented Jan 13, 2025

I am trying to simulate this test locally. I will try to give an update tomorrow

@christianwen
Copy link
Contributor

I'm also trying to mimic the test. Will give an update by EOD

@iwiznia
Copy link
Contributor

iwiznia commented Jan 14, 2025

ok cool, let me know

@melvin-bot melvin-bot bot removed the Overdue label Jan 14, 2025
@DylanDylann
Copy link
Contributor

DylanDylann commented Jan 16, 2025

@iwiznia Do we have any document/guideline for e2ePerformanceTests jobs? It seems we need to simulate an environment on AWS to run it locally

@christianwen Do you have any update here?

@DylanDylann
Copy link
Contributor

@kirillzyusko I noticed you already worked on this area recently #51182. Could you help on this one?

@christianwen
Copy link
Contributor

@DylanDylann @kirillzyusko I'm stuck on the way to simulate this test on local machine. It would be great if we have the instruction for this.

@iwiznia
Copy link
Contributor

iwiznia commented Jan 16, 2025

@iwiznia Do we have any document/guideline for e2ePerformanceTests jobs? It seems we need to simulate an environment on AWS to run it locally

Not really, but I assume someone must have it, post it in the public slack channel (and then post the link to that here) and I bet someone that knows this will help you out.

@DylanDylann
Copy link
Contributor

@kirillzyusko
Copy link
Contributor

I think the problem is solved in #55000 right?

@DylanDylann
Copy link
Contributor

@iwiznia Could you please run this job again?

@iwiznia
Copy link
Contributor

iwiznia commented Jan 17, 2025

Re-run here https://github.com/Expensify/App/actions/runs/12484590267 but not sure if that's what we want since I think that will run it with the old code... 🤔

@melvin-bot melvin-bot bot added the Overdue label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants