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 / Build apk from latest release as a baseline #45280

Closed
github-actions bot opened this issue Jul 11, 2024 · 7 comments
Assignees

Comments

@github-actions
Copy link
Contributor

🚨 Failure Summary 🚨:

⚠️ Action Required ⚠️:

🛠️ A recent merge appears to have caused a failure in the job named e2ePerformanceTests / Build apk from latest release as a baseline.
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!

@etCoderDysto
Copy link
Contributor

etCoderDysto commented Jul 12, 2024

Hi @hayata-suenaga, there seems to be some kind of workflow issue on main from our PR? Is there anything I can do to help fix it?

@melvin-bot melvin-bot bot added the Overdue label Jul 15, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Hourly KSv2 Overdue labels Jul 16, 2024
@kirillzyusko
Copy link
Contributor

apk building has been fixed in #46571 and https://github.com/Expensify/App/pull/45997/files

@roryabraham
Copy link
Contributor

e2e tests appear to still be failing

@roryabraham roryabraham removed the Reviewing Has a PR in review label Aug 2, 2024
@kirillzyusko
Copy link
Contributor

@roryabraham now it fails because of a different problem 😅 I fixed it here: #46708

@roryabraham
Copy link
Contributor

thanks. Merged that PR and we can see if the e2e tests pass here: https://github.com/Expensify/App/actions/runs/10221233824

@kirillzyusko
Copy link
Contributor

@roryabraham tests are failing because a new release has been made and offending changes (#46428) were included in base build. We have to wait for a new release and check whether tests are fixed after a new release.

@melvin-bot melvin-bot bot added the Overdue label Aug 13, 2024
@kirillzyusko
Copy link
Contributor

@roryabraham I think that issue can be closed because e2e tests are working now

@melvin-bot melvin-bot bot removed the Overdue label Aug 21, 2024
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