-
Notifications
You must be signed in to change notification settings - Fork 1
feat: Support Airbyte on Azure #22
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
Conversation
7a5d3d8
to
095cd50
Compare
095cd50
to
05de99f
Compare
@@ -0,0 +1,28 @@ | |||
output "connection_string" { | |||
value = data.azurerm_storage_account.airbyte.primary_connection_string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i feel like there's got to be a more secure way to configure this if possible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I should be able to put it in a secret, but we will still need to provide it. Connection string is the only method presented in Airbyte chart values.
To get the test to work, you need to run |
See: https://artifacthub.io/packages/helm/airbyte/airbyte.