-
Notifications
You must be signed in to change notification settings - Fork 1.1k
PYTHON-5213 Convert AWS Lambda and Search Index tests to use new test scripts #2205
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two non-blocking comments. Overall, LGTM!
@@ -672,7 +672,7 @@ def create_search_index_variants(): | |||
python = CPYTHONS[0] | |||
return [ | |||
create_variant( | |||
["test_atlas_task_group_search_indexes"], | |||
[".index_management"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about making the name more verbose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to search_index
in all dev-facing places.
@@ -175,6 +175,28 @@ def handle_test_env() -> None: | |||
if not config: | |||
AUTH = "noauth" | |||
|
|||
if test_name in ["aws_lambda", "index_management"]: | |||
env = os.environ.copy() | |||
env["MONGODB_VERSION"] = "7.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Earlier we set the MONGODB_VERSION to 6.0 for index_management
and this is overriding that from the copy. Do we need to set 6.0 whatsoever?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Forgot to update markers, new patch build: https://spruce.mongodb.com/version/67daf04c72edb2000767c733/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC |
All green again |
Passing build: https://spruce.mongodb.com/version/67d47d610f5a9c0007917b13/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC