*.from_boot_config mixes credentials with config

Bug #1664300 reported by Aaron Bentley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-ci-tools
Triaged
Medium
Unassigned

Bug Description

The substrate from_boot_config methods typically mix credentials with config options.

This was done to support the transition from 1.x to 2.x more easily, but is tech debt. The problem is that it adds confusion about where a value is expected to come from. An account object may specify credential values (e.g. secret key) and config values (e.g. region).

It should be straightforward to explicitly get credential values from get_cloud_credentials and config values from get_config_option.

AzureARMAccount has already been updated, but other providers have not.

Tags: tech-debt
Revision history for this message
Anastasia (anastasia-macmood) wrote :

@Aaron,

Is this still valid?

Changed in juju:
status: Triaged → Incomplete
importance: Medium → Low
Revision history for this message
Aaron Bentley (abentley) wrote :

Yes, but it's on the wrong product.

no longer affects: juju
Changed in juju-ci-tools:
status: New → Triaged
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.