Skip to content

env vars for s3 buckets#4580

Open
boozallendanny wants to merge 16 commits intoqatfrom
mod/env-vars-s3-buckets
Open

env vars for s3 buckets#4580
boozallendanny wants to merge 16 commits intoqatfrom
mod/env-vars-s3-buckets

Conversation

@boozallendanny
Copy link
Contributor

Description:

Move env specific buckets to env vars

Technical Details:

os.environ or None

Requirements for PR Merge:

  1. Unit & integration tests updated
  2. API documentation updated (examples listed below)
    1. API Contracts
    2. API UI
    3. Comments
  3. Data validation completed (examples listed below)
    1. Does this work well with the current frontend? Or is the frontend aware of a needed change?
    2. Is performance impacted in the changes (e.g., API, pipeline, downloads, etc.)?
    3. Is the expected data returned with the expected format?
  4. Appropriate Operations ticket(s) created
  5. Jira Ticket(s)
    1. DEV-0

Explain N/A in above checklist:

Copy link
Contributor

@sethstoudenmier sethstoudenmier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but added two cleanup comments that this change just carries over from previous work.

@sethstoudenmier sethstoudenmier added the do not merge [PR] shouldn't be merged label Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge [PR] shouldn't be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants