airbyte-ci: reduce required env var when running in CI (fixed)

[alafanechere]

Reading the self.ci_gcs_credentials_secret property in a context before the Dagger client instantiation led to AssertionError: The dagger client was not set on this PipelineContext.
This property used the self.dagger_client but it was used by self.remote_storage_enabled in a code path where the dagger client was not set on the PipelineContext.

Fixing it by accessing self.ci_gcs_credentials instead of self.ci_gcs_credentials_secret in the self.remote_storage_enabled property.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	⬜️ Ignored (Inspect)	Visit Preview		May 3, 2024 7:27am

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: airbyte-ci/connectors/pipelines/pipelines/models/contexts/pipeline_context.py

Function	Unhandled Issue
github_commit_status	AssertionError: The dagger client was not set on this PipelineContext pipelines.models.contexts.pipeline_con... Event Count: 44

---

_{Did you find this useful? React with a 👍 or 👎}

[alafanechere]

• airbyte-ci: reduce required env var when running in CI (fixed) #37786 👈
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alafanechere and the rest of your teammates on Graphite