MetaMask / MetaMask/metamask-mobile
Temporarily remove caching steps in Bitrise
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
### What is this about?
Currently, we perform a handful of caching steps between both the build and testing flows in Bitrise. Over the past year, we've been observing a steady stream of failed runs as a result of cache misses or sometimes sporadic behavior. While caching does save us some time for our builds, the time trade-off has been canceled out by the need to support or react whenever e2e failures occur. The purpose of this ticket is to completely remove all caching steps for now and to follow up with tasks to slowly reintroduce cache steps incrementally and to monitor the behavior over a period of time to ensure stability. To contain the scope, we will only focus on removing cache steps for `pr_smoke_e2e_pipeline`
### Scenario
_No response_
### Design
_No response_
### Technical Details
- Remove all cache steps that is used in the `pr_smoke_e2e_pipeline`
- Ensure tests are still running without cache steps
- Note the diff in run duration relative to main
### Threat Modeling Framework
_No response_
### Acceptance Criteria
- All cache steps should be removed in `pr_smoke_e2e_pipeline`
- Tests in `pr_smoke_e2e_pipeline` should be successful with the exception of failing tests on main (non related to this task)
### Stakeholder review needed before the work gets merged
- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)
### References
_No response_
Contributor guide
Assessment
This issue has not been assessed yet.