Skip to content

fix: fix flask e2e smoke tests pipeline#17423

Merged
Andepande merged 7 commits into
mainfrom
fix-flask-e2e-build
Jul 21, 2025
Merged

fix: fix flask e2e smoke tests pipeline#17423
Andepande merged 7 commits into
mainfrom
fix-flask-e2e-build

Conversation

@Andepande

@Andepande Andepande commented Jul 21, 2025

Copy link
Copy Markdown
Contributor

Description

flask_smoke_e2e_pipeline was failing due to a recent change that was made to aid caching builds for faster e2e during PR's, this introduces a small fix to remedy that error.

Changelog

CHANGELOG entry:

Related issues

Fixes:
https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/5d65a37e-0b15-4971-972f-0f7a153014c9

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@Andepande Andepande added the No QA Needed Apply this label when your PR does not need any QA effort. label Jul 21, 2025
@Andepande Andepande requested a review from a team July 21, 2025 17:26
@github-actions

github-actions Bot commented Jul 21, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ flask_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 8942f58
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/80bf94e2-3db1-44f2-bcd8-0e2e062fda81

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another flask_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

cmd-ob
cmd-ob previously approved these changes Jul 21, 2025
View reviewed changes

@cmd-ob cmd-ob left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

cursor[bot]

This comment was marked as outdated.

Sign in to view

@Andepande Andepande enabled auto-merge July 21, 2025 19:24
@sonarqubecloud

sonarqubecloud Bot commented Jul 21, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Andepande Andepande added this pull request to the merge queue Jul 21, 2025
Merged via the queue into main with commit 720f72a Jul 21, 2025
52 of 53 checks passed
@Andepande Andepande deleted the fix-flask-e2e-build branch July 21, 2025 20:07
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 21, 2025
@metamaskbot metamaskbot added the release-7.53.0 Issue or pull request that will be included in release 7.53.0 label Jul 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

@tommasini tommasini tommasini approved these changes

@cmd-ob cmd-ob cmd-ob approved these changes

Assignees

@chrisleewilcox chrisleewilcox

@cmd-ob cmd-ob

Labels

No QA Needed Apply this label when your PR does not need any QA effort. release-7.53.0 Issue or pull request that will be included in release 7.53.0 team-qa QA team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Andepande @tommasini @cmd-ob @chrisleewilcox @Mrtenz @metamaskbot