Skip to content

Conversation

@Julien-Ben
Copy link
Collaborator

Summary

We often encounter setup failures in the CI.

Today I had many with helm install, example:
https://parsley-beta.corp.mongodb.com/evergreen/mongodb_kubernetes_e2e_multi_cluster_kind_e2e_multi_cluster_sharded_external_access_no_ext_domain_patch_fd4450bdd5aab857c0b9570cfd1cb4f3b8bb42f8_68f77aa949cdc3000709dd57_25_10_21_12_21_00/1/task?bookmarks=0,172

This PR adds retries to mitigate these transient installation failures.

Proof of Work

CI is green, and we see less setup failures

@Julien-Ben Julien-Ben added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Oct 21, 2025
@github-actions
Copy link

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.6.0 Release Notes

New Features

  • MongoDBCommunity: Added support to configure custom cluster domain via newly introduced spec.clusterDomain resource field. If spec.clusterDomain is not set, environment variable CLUSTER_DOMAIN is used as cluster domain. If the environment variable CLUSTER_DOMAIN is also not set, operator falls back to cluster.local as default cluster domain.
  • Helm Chart: Introduced two new helm fields operator.podSecurityContext and operator.securityContext that can be used to configure securityContext for Operator deployment through Helm Chart.

Bug Fixes

  • Fixed parsing of the customEnvVars Helm value when values contain = characters.

@Julien-Ben Julien-Ben changed the title Add retries to tool install Add retries to tool installs Oct 22, 2025
<<: *community_dependency
run_on:
- ubuntu2004-large
- ubuntu2404-large
Copy link
Collaborator Author

@Julien-Ben Julien-Ben Oct 22, 2025

Choose a reason for hiding this comment

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

tags: [ "pr_patch", "staging", "e2e_test_suite" ]
run_on:
- ubuntu1804-xlarge
- ubuntu2404-xlarge
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same here

@Julien-Ben Julien-Ben marked this pull request as ready for review October 22, 2025 07:18
@Julien-Ben Julien-Ben requested a review from a team as a code owner October 22, 2025 07:18
Copy link
Collaborator

@MaciejKaras MaciejKaras left a comment

Choose a reason for hiding this comment

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

Very nice!

@Julien-Ben Julien-Ben merged commit 326072a into master Oct 22, 2025
35 of 37 checks passed
@Julien-Ben Julien-Ben deleted the jben/retry-tool-install branch October 22, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use this label in Pull Request to not require new changelog entry file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants