Skip to content

Conversation

blink1073
Copy link
Member

@blink1073 blink1073 requested a review from NoahStapp March 25, 2025 14:24
@blink1073
Copy link
Member Author

I have no idea why there are so many merge commits. 😅

def load_config_from_file(path: str | Path) -> dict[str, str]:
config = read_env(path)
for key, value in config.items():
write_env(key, value)
Copy link
Contributor

Choose a reason for hiding this comment

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

Does write_env() write the key-value pairs out to the runtime environment?

Copy link
Member Author

Choose a reason for hiding this comment

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

It writes it to test-env.sh.

SUB_TEST_REQUIRED = ["auth_aws", "auth_oidc", "kms", "mod_wsgi", "perf"]

EXTRA_TESTS = ["mod_wsgi", "aws_lambda", "search_index"]
EXTRA_TESTS = ["mod_wsgi", "aws_lambda"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Is search_index run somewhere else now?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is one of the test suites, not an extra test.

@blink1073 blink1073 requested a review from NoahStapp March 25, 2025 14:54
Copy link
Contributor

@NoahStapp NoahStapp left a comment

Choose a reason for hiding this comment

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

Can you schedule an EG patch with every task and variant so we can check for hidden bugs/failures before merging?

@blink1073
Copy link
Member Author

@blink1073
Copy link
Member Author

I opened https://jira.mongodb.org/browse/PYTHON-5232 for the existing AWS Lambda failure.

@blink1073
Copy link
Member Author

All set!

@blink1073 blink1073 merged commit 894d5e1 into mongodb:master Mar 25, 2025
36 of 38 checks passed
@blink1073 blink1073 deleted the cli-cleanup branch March 25, 2025 17:53
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