canonical / canonical/pycloudlib

[enhancement] Update handling of "required_values" in cloud class

Open
#466 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
22
Forks
42
PR merge metrics
No merged PRs in 30d

Description

Currently, if all "required values" for a given cloud class are given at runtime, the pycloudlib config file is NOT even looked at or parsed. This means that any additional or optional settings that are configured in the pycloudlib.toml are ignored if the required values are given. This prevents using the pycloudlib.toml as a "base" configuration that can have certain values overridden by passing them to the cloud constructor at runtime.

Ask:
Make it so that even if all required values are satisfied, the toml is still read into configuration.
Such that the pycloudlib.toml is the base configuration and any options passed at runtime override options set in the pycloudlib.toml

Relates to: #457
This should be done before #457 as a stop-gap until a full configuration overhaul is done.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.