Skip to content

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

Merged
merged 38 commits into from
Apr 7, 2025

Conversation

maciaszczykm
Copy link
Member

@maciaszczykm maciaszczykm commented Mar 14, 2025

Copy link

linear bot commented Mar 14, 2025

@maciaszczykm maciaszczykm changed the title feat: Support deploying Airbyte on Azure feat: Support Airbyte on Azure Mar 14, 2025
@maciaszczykm maciaszczykm force-pushed the marcin/prod-3112-airbyte-azure-setup branch from 7a5d3d8 to 095cd50 Compare March 21, 2025 15:49
@maciaszczykm maciaszczykm force-pushed the marcin/prod-3112-airbyte-azure-setup branch from 095cd50 to 05de99f Compare March 21, 2025 15:49
@@ -0,0 +1,28 @@
output "connection_string" {
value = data.azurerm_storage_account.airbyte.primary_connection_string
Copy link
Member

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?

Copy link
Member Author

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.

@maciaszczykm maciaszczykm marked this pull request as ready for review April 1, 2025 14:40
@michaeljguarino
Copy link
Member

To get the test to work, you need to run just test and add it to your commit

@maciaszczykm maciaszczykm merged commit 78f30f2 into main Apr 7, 2025
1 check passed
@maciaszczykm maciaszczykm deleted the marcin/prod-3112-airbyte-azure-setup branch April 7, 2025 07:01
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.

2 participants