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.

Sign up for GitHub

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

Jump to bottom

ci(MM-63212): actions/cache brownout fix #8609

Closed
wants to merge 2 commits into from
Closed

ci(MM-63212): actions/cache brownout fix #8609

wants to merge 2 commits into from

Conversation

rahimrahman
Copy link
Contributor

@rahimrahman rahimrahman commented Feb 18, 2025 •
edited
Loading

Summary

We have started seeing failure in Github actions due to the use of actions/cache. This is caused by brownout, which can be seen here:https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down

Changes here pin the actions/cache to a specific version.

Ticket Link

Fixes: https://mattermost.atlassian.net/browse/MM-63212

Checklist

  • Added or updated unit tests (required for all new features)
  • Has UI changes
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.
  • Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on:

Screenshots

Release Note

@rahimrahman rahimrahman added the Build Apps for PR Build the mobile app for iOS and Android to test label Feb 18, 2025
@yasserfaraazkhan yasserfaraazkhan added E2E Android tests for PR Run Android E2E Detox tests and removed E2E Android tests for PR Run Android E2E Detox tests labels Feb 19, 2025
@toninis
Copy link
Contributor

toninis commented Feb 19, 2025

@rahimrahman Closing this in favor of #8613

@toninis toninis closed this Feb 19, 2025
@toninis toninis deleted the ci/MM-63212-actions-cache-brownout branch February 19, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@enzowritescode enzowritescode enzowritescode approved these changes

@Willyfrog Willyfrog Willyfrog approved these changes

@enahum enahum enahum approved these changes

@yasserfaraazkhan yasserfaraazkhan yasserfaraazkhan approved these changes

@larkox larkox Awaiting requested review from larkox

Assignees
No one assigned
Labels
Build Apps for PR Build the mobile app for iOS and Android to test release-note
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@rahimrahman @toninis @enzowritescode @Willyfrog @enahum @yasserfaraazkhan @mm-cloud-bot