support authentication from OS_* environment variables - #81
Merged
Merged
Conversation
akyriako
approved these changes
Oct 2, 2026
Owner
|
Hej @thomaslaurenson, thanks a lot taking an interest in the project and taking the time to contribute as well. The OS_* env var support is a very useful addition, and I really appreciate the effort you put into the implementation, testing, and docs. Which other OpenStack services are you using regularly? and are there particular commands/actions you’d like to see implemented in o7k? Real-world feedback in this phase is extremely useful for figuring out where to take the project next. Thanks again and feel free to ⭐ the repo and spread the word so more OpenStack folks get to know the project. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Firstly, very cool project - I have been using a local build against our OpenStack environment. However, one issue for us is that most team member using the standard
OS_*environment variables, either sourced from anopenrcfile or injected per process by a secret manager - I have a password store extension I use (https://github.com/thomaslaurenson/pass-env).Changes
OS_AUTH_URLis set, the Contexts screen lists an extra context namedenvvars(name taken from openstacksdk)envvarsauthenticates with gophercloud from theOS_*variables; everything downstream sees an ordinary contextclouds.yamlwhenOS_AUTH_URLis set; without either, the error and exit code are unchangedLimitations
clouds.yaml, user and project are assumed to be in the same domain; useOS_PROJECT_IDotherwiseOS_CACERT,OS_CERTandOS_KEYare not applied to this context yetThis was a first effort to add envvar support. I did some testing locally with envvars set, and
clouds.ymlfiles in default locations. Open to any feedback and changes, and have "allowed edits by maintainers".