Skip to content

LMR-7 Feature/lhsm plugin gcs #6

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

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

mhugues
Copy link
Contributor

@mhugues mhugues commented Aug 27, 2019

Feature to add a plugin for supporting google cloud storage (GCS)

@mhugues mhugues closed this Aug 27, 2019
@mhugues mhugues deleted the feature/lhsm-plugin-gcs branch August 27, 2019 19:56
Maxime Hugues added 11 commits August 27, 2019 20:03
`credentials` configuration option name of GCS plugin
is changed to `service_account_key` for clarity.
For GCS lhsm plugin, the service account key file is now
optional. Credentials to acces bucket are created by either
retrieving the defalt credentials with read/write scope to GCS or by
using the service account key file.
@mhugues mhugues restored the feature/lhsm-plugin-gcs branch August 27, 2019 20:15
@mhugues mhugues reopened this Aug 27, 2019
@mhugues mhugues force-pushed the feature/lhsm-plugin-gcs branch from 5c0d002 to bcd0d03 Compare August 27, 2019 20:17
@mhugues
Copy link
Contributor Author

mhugues commented Sep 25, 2019

Hello,
Do you have any questions or comments on this pull request that I can help with ?

Let me know how we can move this forward.
Thank you

@utopiabound utopiabound requested a review from mdiep25 December 10, 2019 17:15
edwardsp referenced this pull request in edwardsp/lemur Mar 24, 2021
* Retrieve SAS from a kv

* Refresh token after 24hrs

* Disable sharedkey auth

* Ze's comments
@rajachan
Copy link

rajachan commented Nov 3, 2021

This is a valuable addition to Lemur for GCP users. Is there a possibility of this getting merged upstream?

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