Skip to content

backend: allow configuring S3_REGION#870

Closed
alxndrsn wants to merge 1 commit intogetodk:nextfrom
alxndrsn:s3-region
Closed

backend: allow configuring S3_REGION#870
alxndrsn wants to merge 1 commit intogetodk:nextfrom
alxndrsn:s3-region

Conversation

@alxndrsn
Copy link
Contributor

@alxndrsn alxndrsn commented Jan 13, 2025

With getodk/central-backend#1357 & getodk/docs#1912, closes #846

What has been done to verify that this works as intended?

e2e tests in central-backend. Suggestions for higher-level tests appreciated!

Why is this the best possible solution? Were any other approaches considered?

It seems simple and in-line with existing s3 configuration.

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Should not have a direct effect on users, although changes to central-backend may.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

See linked PR for https://github.com/getodk/docs/

Before submitting this PR, please make sure you have:

  • branched off and targeted the next branch OR only changed documentation/infrastructure (master is stable and used in production)
  • verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced

@alxndrsn alxndrsn changed the title central: allow configuring S3_REGION backend: allow configuring S3_REGION Jan 13, 2025
alxndrsn pushed a commit to alxndrsn/odk-docs that referenced this pull request Jan 13, 2025
alxndrsn pushed a commit to alxndrsn/odk-central-backend that referenced this pull request Jan 13, 2025
@alxndrsn alxndrsn requested a review from lognaturel January 14, 2025 04:35
@lognaturel
Copy link
Member

We may want to add this feature at some point but for now the short-term need was addressed by a minio upgrade: #846 (comment)

@lognaturel lognaturel closed this Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants