Skip to content

Add GitHub action for e2e tests #117

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

Merged
merged 4 commits into from
May 9, 2025
Merged

Add GitHub action for e2e tests #117

merged 4 commits into from
May 9, 2025

Conversation

ioanarm
Copy link
Contributor

@ioanarm ioanarm commented May 8, 2025

Added a git action for the python e2e tests and also a schedule for the workflow on a daily basis.
e2e tests are not mandatory to merge for now as we are still iterating and they are still a bit flaky but at least we can get notified when something is systematically failing.

Added a make command for the e2e tests but I got some weird results so I didn't use it in the git action. I can't iterate on this now though. I can do it on a separate PR.

We can probably exclude the other actions from running on a daily basis if we think it's too much.

@ioanarm ioanarm force-pushed the iter-e2e-tests branch 4 times, most recently from b846411 to 8baf620 Compare May 8, 2025 14:28
@ioanarm ioanarm force-pushed the iter-e2e-tests branch 2 times, most recently from 3a32bf0 to 340aef9 Compare May 8, 2025 15:28
@ioanarm ioanarm changed the title wip Iter e2e tests Add git action for e2e tests May 8, 2025
@ioanarm ioanarm marked this pull request as ready for review May 8, 2025 15:48
@ioanarm ioanarm requested a review from a team as a code owner May 8, 2025 15:48
@sd2k sd2k changed the title Add git action for e2e tests Add GitHub action for e2e tests May 9, 2025
@sd2k sd2k self-assigned this May 9, 2025
@sd2k sd2k force-pushed the iter-e2e-tests branch from 8cb768a to 96172ed Compare May 9, 2025 08:29
@sd2k sd2k merged commit 1835175 into main May 9, 2025
10 checks passed
@sd2k sd2k deleted the iter-e2e-tests branch May 9, 2025 09:00
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.

2 participants