-
Notifications
You must be signed in to change notification settings - Fork 175
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
Implement backup/restore to Azure blob storage #944
Conversation
08fb01c
to
07aaa11
Compare
upload-azure: | ||
goal: FINISHED | ||
cmd: > | ||
az login -u $CLIENT_ID --service-principal --tenant $TENANT_ID -p $CLIENT_SECRET |
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'm sure this is correct, but I'll still ask. Does --service-principal
takes an argument like other flags in this command ?
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.
No, it's just a boolean flag that tells the Azure CLI client what type of principal the argument to -u
is. Seems to work as exercised by the integration test :)
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.
👍
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.
LGTM
ded1978
to
1eec204
Compare
@mesosphere-ci retest this please |
Cluster launch is failing. |
1eec204
to
60fb3f4
Compare
…ing the run ID change
60fb3f4
to
23d46dd
Compare
Failed in nightly, please take a look. |
Looks like I failed to add the relevant secrets for Azure access, fixed and rerunning now. |
No description provided.