MITLibraries / timdex-dataset-api / 16971768607

93%

main: 93%

Pull #164

github

Omit chain from DuckDB S3 secret

Why these changes are being introduced:

It sounds like the best option for ECS tasks is using 'instance'
as the provider chain type, where for local dev and/or lambdas
it might be 'sso' or 'env'.  Not having 'instance' appears to cause
failures in the ECS task.

How this addresses that need:

By omitting the 'chain' option entirely from DuckDB secret creation
we allow the default provider chain to take effect.  Given our fairly
normal usage of DuckDB and S3, this is probably the best approach.

Side effects of this change:
* DuckDB to S3 connections work in ECS

Relevant ticket(s):
* https://mitlibraries.atlassian.net/browse/TIMX-540

1 of 1 new or added line in 1 file covered. (100.0%)

460 of 492 relevant lines covered (93.5%)

0.93 hits per line