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

Feature/azure blob storage for profile store #156

Conversation

hassansuhaib
Copy link
Contributor

@hassansuhaib hassansuhaib commented Aug 2, 2023

Issue: #102

Implemented Azure Blob storage for Profile Store

This approach requires these environment variables to be set by the user: AZURE_TENANT_ID, AZURE_CLIENT_ID, AZURE_CLIENT_SECRET because we can't use other methods for creating a blob service account as they require different keys and we only get URL as a param.

@hassansuhaib hassansuhaib force-pushed the feature/azure_blob_storage_for_profile_store branch 2 times, most recently from e5ff378 to d08f842 Compare August 15, 2023 05:53
@hassansuhaib hassansuhaib force-pushed the feature/azure_blob_storage_for_profile_store branch from d08f842 to 649874e Compare August 26, 2023 02:12
@hassansuhaib hassansuhaib force-pushed the feature/azure_blob_storage_for_profile_store branch from 48548e1 to 701f41c Compare September 18, 2023 06:16
@hassansuhaib hassansuhaib marked this pull request as ready for review September 18, 2023 07:51
Copy link
Collaborator

@royra royra left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

  • Some comments below to address
  • Make sure the code is tested manually, or add automated tests
  • Add the option to store the profile on Azure in the init command
  • Add to the docs

packages/driver-azure/src/fs/index.ts Outdated Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Outdated Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Outdated Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Outdated Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Outdated Show resolved Hide resolved
packages/driver-azure/src/fs/index.ts Show resolved Hide resolved
packages/cli/src/fs.ts Outdated Show resolved Hide resolved
@hassansuhaib hassansuhaib force-pushed the feature/azure_blob_storage_for_profile_store branch from 81c0900 to b1af71c Compare November 27, 2023 07:47
@hassansuhaib hassansuhaib force-pushed the feature/azure_blob_storage_for_profile_store branch from b1af71c to 426c37c Compare December 20, 2023 12:03
@royra
Copy link
Collaborator

royra commented Feb 6, 2024

fixed by #423

@royra royra closed this Feb 6, 2024
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.

3 participants