-
Notifications
You must be signed in to change notification settings - Fork 186
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
Credentials from env #1463
Credentials from env #1463
Conversation
Hello @dmach! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-01-08 14:21:58 UTC |
48fff35
to
41b968b
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1463 +/- ##
==========================================
+ Coverage 31.07% 31.25% +0.18%
==========================================
Files 49 49
Lines 17779 17820 +41
==========================================
+ Hits 5524 5570 +46
+ Misses 12255 12250 -5 ☔ View full report in Codecov by Sentry. |
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.
Using this patch in https://src.opensuse.org/alexlau/bash-custom-action/actions/runs/9 container and it is working. Thanks
…r points to /dev/null
Options for apiurls can be set via OSC_HOST_<ALIAS>_<OPTION>=... This requires a configured alias in the config file. Setting the default apiurl via OSC_APIURL=... was working already. Also OSC_CONFIG= / --config= was already implemented to skip loading configuration entirely. Options for the default apiurl can be now set via: OSC_USERNAME=... OSC_CREDENTIALS_MGR_CLASS=... OSC_PASSWORD=... This, for example, makes running osc in containers with credentials stored in environmental variables possible: OSC_CONFIG= OSC_APIURL=<url> OSC_USERNAME=<user> OSC_PASSWORD=<password> osc ...
41b968b
to
82216c7
Compare
No description provided.