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 support for getting Azure Storage Account key from ENV variable #2455

Merged

Conversation

jaygridley
Copy link
Contributor

@jaygridley jaygridley commented Apr 24, 2020

This PR closes #2376.

Signed-off-by: Martin Odstrcilik <martin.odstrcilik@gmail.com>
@skriss skriss added Area/Cloud/Azure Restic Relates to the restic integration labels Apr 27, 2020
@skriss
Copy link
Member

skriss commented Apr 27, 2020

@jaygridley this looks good to me - can you just add a changelog file (changelogs/unreleased/2455-jaygridley) with a brief description of the change?

Copy link
Contributor

@nrb nrb left a comment

Choose a reason for hiding this comment

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

Also LGTM pending changelog

@skriss skriss added this to the v1.4 milestone Apr 29, 2020
@skriss
Copy link
Member

skriss commented Apr 29, 2020

@jaygridley gentle reminder that we're just waiting on a changelog file from you here. Thanks!

@carlisia
Copy link
Contributor

👆

@skriss
Copy link
Member

skriss commented May 1, 2020

I'd say let's merge this and I'll add a changelog file in a separate PR.

skriss added a commit to skriss/velero that referenced this pull request May 1, 2020
Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss
Copy link
Member

skriss commented May 1, 2020

changelog PR added: #2491

let's get these merged!

@skriss
Copy link
Member

skriss commented May 1, 2020

also note I added #2476 to the backlog

@carlisia carlisia merged commit 26f67f1 into vmware-tanzu:master May 1, 2020
@jaygridley
Copy link
Contributor Author

Sorry guys. I was out with proper access. Is there any work needed?

@skriss
Copy link
Member

skriss commented May 1, 2020

nope, we took care of it. thanks for the PR!

ashish-amarnath pushed a commit that referenced this pull request May 1, 2020
Signed-off-by: Steve Kriss <krisss@vmware.com>
@jaygridley jaygridley deleted the azure-storage-account-keys branch June 11, 2020 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Cloud/Azure Restic Relates to the restic integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restic - add support setting Azure Storage Account Key via configuration
4 participants