Skip to content

feature request: completely align toml and class instantiation configuration options #457

Open
@a-dubs

Description

@a-dubs

Any value that can be configured via the pycloudlib.toml should be able to be passed to the cloud class constructor and vice versa.

The disparity and inconsistency in these options is something that has plague pycloudlib for quite some time. This issue reared its ugly head recently when @Bryzizzle and I were working on #454.

Also while working on #454, @Bryzizzle and I realized that the way the required_values works is absolutely ridiculous. if all required values are passed to a class constructor, it will just IGNORE the toml... So if you give the required values at runtime, but have various other config options set in the pycloudlib.toml, it will just ignore the toml file entirely and not read in those other config options.

Similarly, SSH keys can't be passed at runtime to an instance and must either be:

  • set in the pycloudlib.toml
  • or configured at runtime after the class is instantiated using the use_key method.

See #452 for one of many examples of why this is an awful design paradigm.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions