Bug: `meltano config meltano set` should always set a value

~/Development/meltano-projects/superset-test ❯❯❯ meltano config meltano set discovery_url http://localhost:4000/discovery.yml
2022-06-04T01:36:33.566497Z [info     ] Environment 'dev' is active
Meltano setting 'discovery_url' was set in `meltano_environment`: 'http://localhost:4000/discovery.yml'
Current value is still: 'https://discovery.meltano.com/discovery.yml' (from the default)

I’d expect this to store discovery_url: http://localhost:4000/discovery.yml in meltano.yml, but it’s nowhere to be found.

[aaronsteers]

My guess is we have two bugs here:

• first is in resolution order and/or finding the location store
• second is that Meltano is trying to be too clever and clear values instead of writing new ones, if it thinks the new value matches something set at same/higher precedence.

The second issue, I think, is the bigger issue we should tackle. The first issue might only be affecting this particular code path and (if so) might be best resolved by not trying to infer a hierarchy at all.

As a user, if I tell Meltano to store a setting, I don’t want Meltano to second guess whether it actually needs to be set.

I’ll take a deeper look tonight and @kgpayne I might assign to you since this is a part of the codebase I know you are very familiar with.

[DouweM]

@aaronsteers I’ve set the urgency to “Higher” here because this could be a bad bug.