Skip to content

Conversation

matthewdale
Copy link
Collaborator

@matthewdale matthewdale commented Aug 7, 2025

GODRIVER-3631

Summary

Call function assume-test-secrets-ec2-role and pass in the AWS creds in the test-search-index-task-group task group setup.

Background & Motivation

Currently the test-search-index Evergreen tasks all fail with an error like:

 Getting secrets: drivers/atlas-qa...
 Using Python binary /opt/python/Current/bin/python3
 Traceback (most recent call last):
   File "drivers-evergreen-tools/.evergreen/secrets_handling/setup_secrets.py", line 30, in get_secrets
...
     raise ValueError(
         "Please provide a profile (typically using AWS_PROFILE)"
     ) from e
 ValueError: Please provide a profile (typically using AWS_PROFILE)

The other Evergreen task that calls .evergreen/atlas/setup.sh also calls assume-test-secrets-ec2-role first.

@matthewdale matthewdale added bug review-priority-normal Medium Priority PR for Review: within 1 business day labels Aug 7, 2025
Copy link
Contributor

API Change Report

No changes found!

@matthewdale matthewdale force-pushed the fix-test-search-index branch from 4ae58a4 to edb9847 Compare August 7, 2025 03:02
@matthewdale matthewdale force-pushed the fix-test-search-index branch from edb9847 to bc0a531 Compare August 7, 2025 04:32
@matthewdale matthewdale changed the title Assume correct EC2 role in search index test task. GODRIVER-3631 Assume correct EC2 role in search index test task. Aug 7, 2025
@matthewdale matthewdale marked this pull request as ready for review August 7, 2025 16:53
@matthewdale matthewdale requested a review from a team as a code owner August 7, 2025 16:53
@matthewdale matthewdale requested a review from qingyang-hu August 7, 2025 16:53
@matthewdale matthewdale merged commit 845fc8a into mongodb:master Aug 11, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug review-priority-normal Medium Priority PR for Review: within 1 business day

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants