Skip to content

feat: add secrets fetch command to CDK CLI#494

Merged
Aaron ("AJ") Steers (aaronsteers) merged 21 commits intoaj/feat/add-standard-tests-clifrom
devin/1744945591-add-secrets-fetch-command
Apr 18, 2025
Merged

feat: add secrets fetch command to CDK CLI#494
Aaron ("AJ") Steers (aaronsteers) merged 21 commits intoaj/feat/add-standard-tests-clifrom
devin/1744945591-add-secrets-fetch-command

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

Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
@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
devin-ai-integration bot and others added 7 commits April 18, 2025 06:30
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
…ntributing guide

Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
devin-ai-integration bot and others added 2 commits April 18, 2025 21:37
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
devin-ai-integration bot and others added 2 commits April 18, 2025 21:39
@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 6fdf2c6 into aj/feat/add-standard-tests-cli Apr 18, 2025
12 of 13 checks passed
@aaronsteers Aaron ("AJ") Steers (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.

1 participant