Skip to content

Add AWS_PARTITIONS_FILE_OVERRIDE env variable#2794

Merged
mtdowling merged 1 commit intomainfrom
partition-override-envvar
Oct 14, 2025
Merged

Add AWS_PARTITIONS_FILE_OVERRIDE env variable#2794
mtdowling merged 1 commit intomainfrom
partition-override-envvar

Conversation

@mtdowling
Copy link
Member

This allows for easily overriding the partitions.json file used to resolve partition metadata for the endpoints rules engine.

Background

  • What do these changes do?
  • Why are they important?

Testing

  • How did you test these changes?

Links

  • Links to additional context, if necessary
  • Issue #, if applicable (see here for a list of keywords to use for linking issues)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mtdowling mtdowling requested a review from a team as a code owner October 13, 2025 18:40
@mtdowling mtdowling requested a review from sugmanue October 13, 2025 18:40
This allows for easily overriding the partitions.json file used to
resolve partition metadata for the endpoints rules engine.
@mtdowling mtdowling force-pushed the partition-override-envvar branch from 622c026 to fee246e Compare October 13, 2025 18:49
@mtdowling mtdowling merged commit 4a2ac32 into main Oct 14, 2025
11 checks passed
@mtdowling mtdowling deleted the partition-override-envvar branch October 14, 2025 15:21
.getPartitions());
initializeRegionMap();
// Use the override if present in the environment. Ignore an empty string value.
String override = System.getenv(AWS_PARTITIONS_FILE_OVERRIDE);
Copy link
Contributor

@yasmewad yasmewad Oct 15, 2025

Choose a reason for hiding this comment

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

I know this was merged but do we need to update any existing tests or add new test cases?

Copy link
Member Author

Choose a reason for hiding this comment

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

Since it needs to reload the class and use a custom environment variable to test this behavior, I didn't see a reasonable way to test this.

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.

3 participants