Skip to content

Conversation

Jibola
Copy link
Contributor

@Jibola Jibola commented Sep 6, 2024

This PR focuses on primarily converting the pre/post functions with a few additional artifact and bootstrapping functions included as well.

Functions with converted shell.exec --> subprocess.exec

  • "prepare resources"
  • "upload mo artifacts"
  • "bootstrap mongo orchestration"
  • "bootstrap data lake"
  • "stop mongo-orchestration"
  • "run aws ECS auth test"
  • "windows fix"
  • "make files executable"
  • "init test-results"
  • "install dependencies"

Functions Remaining (to be done in more PRs)

  • "run tests" -- combined both shell.exec calls
  • "download and merge coverage"
  • "run mod_wsgi tests"
  • "run mockupdb tests"
  • "run doctests"
  • "run enterprise auth tests"
  • "run atlas tests"
  • "run aws auth test with regular aws credentials"
  • "run aws auth test with assume role credentials"
  • "run aws auth test with aws EC2 credentials"
  • "run aws auth test with aws web identity credentials"
  • "run oidc auth test with aws credentials as environment variables"
  • "run aws auth test with aws credentials and session token as environment variables"
  • "cleanup"
  • "run-ocsp-test"
  • run-valid-ocsp-server
  • run-revoked-ocsp-server
  • run-valid-delegate-ocsp-server
  • run-revoked-delegate-ocsp-server
  • "run load-balancer"
  • "stop load-balancer"
  • "teardown_docker"
  • "teardown_aws"
  • "run perf tests"

@Jibola Jibola requested review from blink1073, ShaneHarvey, NoahStapp and a team and removed request for blink1073, ShaneHarvey and NoahStapp September 6, 2024 14:05
@blink1073 blink1073 requested review from blink1073 and removed request for caseyclements September 6, 2024 15:56
@Jibola
Copy link
Contributor Author

Jibola commented Sep 6, 2024

Re-working this to use some drivers-evergreen-tools version of scripts!

Copy link
Contributor

@caseyclements caseyclements left a comment

Choose a reason for hiding this comment

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

Linting errors in static tests are definitely real.

Not sure about missing log files in other tests. Seem like the type of difference one might find from making a shell change like this but should be picked up in all tests. Let's look through when you're back.

@blink1073 blink1073 marked this pull request as draft October 23, 2024 18:14
@blink1073
Copy link
Member

Superseded by #1997

@blink1073 blink1073 closed this Nov 8, 2024
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