Cody J. Hanson

Results 27 comments of Cody J. Hanson

@WillDaSilva thanks for the info, that's definitely what I should have used here 😅

FYI @tayloramurphy and @aaronsteers, for some reason I'm not seeing a `weight` field for this, but I'd weight this at a `2`

@aaronsteers Actually the symptom is more that the feature flag works as expected when used correctly in the code base, but the way it's written right now makes it easy...

Note: current behavior is that schedule env only takes precedence over terminal and .env layers and _does not do variable expansion_.

@tayloramurphy -- there's work in progress (see #5982) to update the precedence order to be in line with [this spec in Miro](https://miro.com/app/board/uXjVO3j_ADw=/). New precedence order would be: ``` - environment...

@tayloramurphy - at a glance, it looks like this is supported in [Schedule](https://github.com/meltano/meltano/blob/33ea40987bdaeffc8b9ab2b48fc0e6efc4984ecf/src/meltano/core/schedule.py#L32) object and [ScheduleService](https://github.com/meltano/meltano/blob/33ea40987bdaeffc8b9ab2b48fc0e6efc4984ecf/src/meltano/core/schedule_service.py#L300). But it is not included in the `meltano.yml` jsonschema, meaning that most IDEs will...

@tayloramurphy I think it's reasonable to include fixing the behavior in the scope of this issue. With adding testing for other levels of `env` support we were starting from scratch,...

Do we necessarily want to treat secrets that differently than other settings? One approach to this would be to just implement secrets backends as SettingsValueStores. So we still automatically fall...

^ can't think of a specific use case, but then secrets could also be inherited/referenced from other secrets. Something like, if we have `some_aws_arn_we_want_to_keep_secret` in a secrets backend and then...

@aaronsteers @pnadolny13 -- [I think this is the source of the issue](https://github.com/meltano/meltano/blob/e4cf960955befe90c66218d50a1d73f3572bc08f/src/meltano/core/utils/__init__.py#L469), basically an exception isn't raised for the case where the string being expanded is a full match rather...