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
azure: better error messages / allow_anonymous_login option #5833
Conversation
7e3ad5d
to
4f05aa5
Compare
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.
Not sure if it's worth me trying out various azure methods to understand the UX better. From just looking through it, it looks good to me!
"Authentication to Azure Blob Storage requires either " | ||
"account_name or connection_string.\nLearn more about " | ||
"configuration settings at " | ||
+ format_link("https://man.dvc.org/remote/modify") |
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 not 100% sure I get this change (see Q in iterative/dvc.org#2489 (review)) but maybe:
"Authentication to Azure Blob Storage requires either " | |
"account_name or connection_string.\nLearn more about " | |
"configuration settings at " | |
+ format_link("https://man.dvc.org/remote/modify") | |
"Authentication to Azure Blob Storage requires setting either " | |
"account_name or connection_string.\nLearn more about " | |
"remote storage settings at " | |
+ format_link("https://man.dvc.org/remote/modify") |
Some quick explanation of the change in the PR desc. would be great. Or link to a specific comment in #5797. Cc @casperdcl
210d9a0
to
46055a8
Compare
46055a8
to
89d50bc
Compare
if ( | ||
login_info["account_name"] | ||
and not any_secondary | ||
and not config.get("allow_anonymous_login") |
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 don't see this option in config schema or anywhere in DVC outside of tests.
β I have followed the Contributing to DVC checklist.
π If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here. (azure: update docs for explicit anonymous loginΒ dvc.org#2489)
Thank you for the contribution - we'll try to review it as soon as possible. π
Part of #5797. Implements the
allow_anonymous_option
and support for anonymous logins.