-
Notifications
You must be signed in to change notification settings - Fork 729
Use environment variable to control default verbosity #580
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
base: master
Are you sure you want to change the base?
Use environment variable to control default verbosity #580
Conversation
|
Can we think about moving the environment variable to the artifacts-keyring layer instead? My main concern is avoiding multiple places where verbosity can be set and all other protocols use cli inputs to specify it. Also, since PowerArgs is quite slow and we're aiming to remove it in the future, we'd like to reduce our dependency on it wherever possible. |
I did originally think this (that the verbosity control should be in artifacts-keyring). The problem is that although this would work for So that's what led me to putting this control in at the bottom of the stack. Regardless of how the credential provider is invoked, this provides debug capability. |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This PR implements suggestion #579
ARTIFACTS_CREDENTIALPROVIDER_VERBOSITY_DEFAULT-Verbositycommand line option.LogLevel.Information- no change to previous behaviour.These changes make it much easier to debug uses of the Credential Provider from (eg) pip where it is almost impossible to control the -V option on the command line.