Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

This PR targets the following PR:


Added a 'secrets fetch' command to the airbyte-cdk CLI that can fetch connector secrets from Google Secret Manager. This provides functionality similar to the PyAirbyte script 'run_gsm_connector_secret_fetch.py'.

The command follows the same patterns as existing commands like 'connector test', allowing users to specify either a connector name or directory.

The Google Secret Manager dependency is added as an optional dependency in the 'dev' extras to avoid adding it as a requirement for production use.

Link to Devin run: https://app.devin.ai/sessions/121e177095fa41d4ae43dcfae597f3de
Requested by: Aaron ("AJ") Steers

@devin-ai-integration
Copy link
Contributor Author

Original prompt from Aaron:

@Devin - Take a look at the pyairbyte script called `examples/run_gsm_connector_secret_fetch.py`. I want to build the same into a CDK CLI, building on the work from this PR: <https://github.com/airbytehq/airbyte-python-cdk/pull/493>

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the enhancement New feature or request label Apr 18, 2025
@aaronsteers aaronsteers merged commit 6fdf2c6 into aj/feat/add-standard-tests-cli Apr 18, 2025
12 of 13 checks passed
@aaronsteers aaronsteers deleted the devin/1744945591-add-secrets-fetch-command branch April 18, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants