Skip to content

test: Re-apply streams e2e test and add retries to commands MONGOSH-2300 #2484

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

malexandert
Copy link
Contributor

No description provided.

Comment on lines +9 to +14
const {
STREAMS_E2E_SPI_CONNECTION_STRING = '',
STREAMS_E2E_DB_USER = '',
STREAMS_E2E_DB_PASSWORD = '',
STREAMS_E2E_CLUSTER_CONNECTION_STRING = '',
} = process.env;
Copy link
Contributor Author

@malexandert malexandert Jun 24, 2025

Choose a reason for hiding this comment

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

Something that confuses me is that between the OG commit and the revert, the .evergreen.yml got updated by a bot which removed my env vars from .evergreen.yml, so reapplying the commit didn't put them back. However, the tests still seem to run, even though they should skip and log that if the right variables aren't present. So I guess having the right values in project settings without specifying them in the .evergreen.yml is sufficient?

Copy link
Collaborator

Choose a reason for hiding this comment

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

You'll need to add them to .evergreen/evergreen.yml.in, which the .evergreen.yml file is generated from

Having them in the project settings may currently work as a way of passing these env vars, but I'd definitely prefer to use an explicit way of passing them if we can

@malexandert malexandert requested a review from addaleax June 24, 2025 18:51
@malexandert malexandert marked this pull request as ready for review June 24, 2025 18:51
@malexandert malexandert requested a review from a team as a code owner June 24, 2025 18:51
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