insitro / redun

Yet another redundant workflow engine
https://insitro.github.io/redun/
Apache License 2.0
510 stars 43 forks source link

Pass AWS_DEFAULT_REGION env variable to container #61

Closed jsdir closed 1 year ago

jsdir commented 1 year ago

This fix supports the ability of Docker executors to access S3 buckets regardless of region.

mattrasmus commented 1 year ago

@jsdir Looks like there was a breaking change in tox and one of our dependencies. The fix is now in main. You can rebase this PR to pick up the fixes or I'm happy to can take care of rebasing and merging internally. Thanks again for contributing.

jsdir commented 1 year ago

@mattrasmus The PR has been rebased. Let me know of any other changes needed.

mattrasmus commented 1 year ago

FYI, I fixed a small lint issue. All merged now. Thanks again!