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.

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

add Google secrets #859

Merged
merged 10 commits into from Dec 29, 2023
Merged

add Google secrets #859

merged 10 commits into from Dec 29, 2023

Conversation

AstrakhantsevaAA
Copy link
Contributor

@AstrakhantsevaAA AstrakhantsevaAA commented Dec 27, 2023

Description

Enable Google Secrets for doc tests (code-snippets).

Related Issues

  • Fixes #...
  • Closes #...
  • Resolves #...

Additional Context

Copy link

netlify bot commented Dec 27, 2023

Deploy Preview for dlt-hub-docs ready!

Name Link
🔨 Latest commit 299d0ab
🔍 Latest deploy log https://app.netlify.com/sites/dlt-hub-docs/deploys/658dd0ba6a24c200080bafdf
😎 Deploy Preview https://deploy-preview-859--dlt-hub-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@rudolfix rudolfix left a comment

Choose a reason for hiding this comment

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

LGTM! you could move qdrant to destinations in google secrets (see my comment)

# Slack hook for chess in production example
RUNTIME__SLACK_INCOMING_HOOK: ${{ secrets.RUNTIME__SLACK_INCOMING_HOOK }}
# Mongodb url for nested data example
MONGODB_PIPELINE__SOURCES__CONNECTION_URL: ${{ secrets.MONGODB_PIPELINE__SOURCES__CONNECTION_URL }}

# Qdrant credentials
DESTINATION__QDRANT__CREDENTIALS__LOCATION: ${{ secrets.DESTINATION__QDRANT__CREDENTIALS__LOCATION }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe you could add qdrant to destination secret in google secrets and remove that? optional.

@AstrakhantsevaAA AstrakhantsevaAA merged commit 2c1dbde into devel Dec 29, 2023
56 checks passed
@AstrakhantsevaAA AstrakhantsevaAA deleted the dlt_secret_toml branch December 29, 2023 13:56
Pipboyguy pushed a commit that referenced this pull request Jan 1, 2024
* add Google secrets

* test_config_providers doesn't return anything

* return test utils

* config.toml not from google secrets

* set working dir

* remove return typing

* remove unused line

* add google client to deps

* delete sources ENVs

* return back performance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants