Skip to content

Skip installation on GCP CLI in run-firebase-test#3218

Merged
fire-at-will merged 6 commits into
mainfrom
skip-installation-on-gcp-cli
Mar 11, 2026
Merged

Skip installation on GCP CLI in run-firebase-test#3218
fire-at-will merged 6 commits into
mainfrom
skip-installation-on-gcp-cli

Conversation

@fire-at-will

@fire-at-will fire-at-will commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Description

The run-firebase-tests job in the native Android release pipeline is repeatedly failing today on the Install latest gcloud CLI version if not available step with the following failure message:

Failure:
/bin/bash: line 141: python: command not found
Detected platform: linux ()
The version installed (559.0.0) differs from the version requested (560.0.0).
Uninstalling v559.0.0...
sudo is required to uninstall the Google Cloud SDK.
Please install it and try again.
Failed to uninstall the current version.

Exited with code exit status 1

This address the failure by not installing GCP CLI and just using the version installed in the image.


Note

Low Risk
Low risk CircleCI config-only change that avoids a failing gcloud uninstall/reinstall path; main risk is relying on the preinstalled gcloud version in the executor image.

Overview
Updates CircleCI Firebase Test Lab workflows to stop installing/refreshing gcloud and instead use the version already present in the gcp-cli executor image.

Adds skip_install: true (with explanatory comments) to gcp-cli/setup in run-firebase-tests and the purchases integration-test Firebase jobs, preventing failures caused by attempted uninstall/reinstall requiring sudo.

Written by Cursor Bugbot for commit 47aa48d. This will update automatically on new commits. Configure here.

@fire-at-will fire-at-will added the pr:fix A bug fix label Mar 11, 2026
Comment thread .circleci/config.yml Outdated
@fire-at-will fire-at-will marked this pull request as ready for review March 11, 2026 18:25
@fire-at-will fire-at-will requested a review from a team as a code owner March 11, 2026 18:25
@fire-at-will fire-at-will requested a review from ajpallares March 11, 2026 18:25

@ajpallares ajpallares left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for fixing this!

@codecov

codecov Bot commented Mar 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.19%. Comparing base (56d8a66) to head (47aa48d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3218   +/-   ##
=======================================
  Coverage   79.19%   79.19%           
=======================================
  Files         356      356           
  Lines       14272    14272           
  Branches     1945     1945           
=======================================
  Hits        11302    11302           
  Misses       2165     2165           
  Partials      805      805           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Merged via the queue into main with commit 3bbde8e Mar 11, 2026
30 checks passed
@fire-at-will fire-at-will deleted the skip-installation-on-gcp-cli branch March 11, 2026 19:00
github-merge-queue Bot pushed a commit that referenced this pull request Mar 12, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* [EXPERIMENTAL]: Beta Galaxy Store Support (#2903) via Will Taylor
(@fire-at-will)
### 🐞 Bugfixes
* Skip installation on GCP CLI in run-firebase-test (#3218) via Will
Taylor (@fire-at-will)
* Fix reduced timeouts being used for HTTP requests when a proxy URL is
configured (#3188) via Rick (@rickvdl)

## RevenueCatUI SDK
### 🐞 Bugfixes
* Fix missing ripple effect in View-based paywall wrappers (#3206) via
Toni Rico (@tonidero)
### Paywallv2
#### ✨ New Features
* Rules v0 Integration branch (#3117) via Cesar de la Vega (@vegaro)

### 🔄 Other Changes
* [Galaxy]: Add promotionEligibilities comment (#3214) via Will Taylor
(@fire-at-will)
* [EXTERNAL] Migrate deprecated buildDir to layout API (#3202)
contributed by @AlexanderTalledo (#3212) via Toni Rico (@tonidero)
* Remove automatic Claude code review workflow (#3211) via Cesar de la
Vega (@vegaro)
* Remove unused convention plugin (#3195) via Toni Rico (@tonidero)
* [EXTERNAL] Integrate convention plugins into Version Catalogs (#3181)
contributed by @AlexanderTalledo (#3194) via Toni Rico (@tonidero)
* [EXTERNAL] Migrate androidx cardview dependency to version catalogs
(#3192) contributed by @AlenxanderTalledo (#3193) via Toni Rico
(@tonidero)
* Improve AdMob adapter test coverage (#3204) via Pol Miro (@polmiro)
* Bump fastlane-plugin-revenuecat_internal from `f5c099b` to `e146447`
(#3197) via dependabot[bot] (@dependabot[bot])
* Fix integration tests (#3196) via Toni Rico (@tonidero)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk release bookkeeping: version strings and deployment paths are
updated from `9.24.0-SNAPSHOT` to `9.24.0`, plus changelog/docs refresh.
Main risk is accidental publishing/docs deployment to the wrong
versioned location.
> 
> **Overview**
> Cuts the `9.24.0` release by updating all version references from
`9.24.0-SNAPSHOT` to `9.24.0` (root `.version`, `gradle.properties`, and
`Config.frameworkVersion`), and aligning sample/test app version
catalogs to consume the released artifact.
> 
> Updates documentation publishing to point at the `9.24.0` directory
(CircleCI S3 sync and `docs/index.html` redirect) and refreshes
`CHANGELOG.latest.md`/`CHANGELOG.md` with the `9.24.0` release notes.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
8e6d567. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants