Skip to content
This repository was archived by the owner on Jul 20, 2026. It is now read-only.

sdk: use default conf instead of raising error on buggy/no conf file. - #196

Merged
jerome-jutteau merged 1 commit into
masterfrom
no_conf_file_fix
Jun 8, 2022
Merged

sdk: use default conf instead of raising error on buggy/no conf file.#196
jerome-jutteau merged 1 commit into
masterfrom
no_conf_file_fix

Conversation

@outscale-mgo

Copy link
Copy Markdown

This is so we can use osc-cli without conf file,
either for calls that doesn't need auth or using environement variables

Signed-off-by: Matthias Gatto matthias.gatto@outscale.com

@outscale-mgo
outscale-mgo force-pushed the no_conf_file_fix branch 6 times, most recently from 78a583e to 303ebec Compare June 6, 2022 17:04
This is so we can use osc-cli without conf file,
either for calls that doesn't need auth or using environement variables

Note that we don't support environement variables, but this is a good start

Signed-off-by: Matthias Gatto <matthias.gatto@outscale.com>

@jerome-jutteau jerome-jutteau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

manually tested, OK for me
We need to run test_int.sh in CI and add a specific case for this
#198
#199

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants