Skip to content

[CLOUDP-352109] Run MCK E2E tests against OCI published helm chart #509

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

[CLOUDP-352109] Run MCK E2E tests against OCI published helm chart #509

wants to merge 29 commits into from

Conversation

@viveksinghggits
Copy link
Contributor

@viveksinghggits viveksinghggits commented Oct 8, 2025 •
edited
Loading

Summary

As part of our work to move towards OCI compatible container registries for our helm chart, we are also planning to run our E2E tests against the helm chart that we publish to OCI. This will make sure that we are testing in our E2E what we are providing to our customers.
As part of this effort we have already raised a PR that starts publishing our helm chart to the OCI container registry during dev/staging workflows.
This PR goes and changes our E2E tests to start consuming the helm chart from OCI registry instead of the local helm chart repo.

Proof of Work

Successful CI on this PR.
I also ran the test e2e_replica_set_migration locally and it was successful. The logs are here.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you added changelog file?

@viveksinghggits viveksinghggits requested a review from a team as a code owner October 8, 2025 15:48
@viveksinghggits viveksinghggits requested review from MaciejKaras and fealebenpae and removed request for a team October 8, 2025 15:48
@viveksinghggits viveksinghggits marked this pull request as draft October 8, 2025 15:48
@viveksinghggits viveksinghggits added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Oct 8, 2025
@viveksinghggits viveksinghggits force-pushed the publish-chart-oci-prpatches branch from 9bd1970 to 5f0938b Compare October 8, 2025 15:50
@viveksinghggits viveksinghggits force-pushed the e2e-tests-on-published-chart branch from 9d6699b to 4ad79ae Compare October 8, 2025 15:51
@viveksinghggits viveksinghggits force-pushed the publish-chart-oci-prpatches branch from 5f0938b to 1b61956 Compare October 10, 2025 14:28
@viveksinghggits viveksinghggits force-pushed the e2e-tests-on-published-chart branch from 4ad79ae to 9394e7c Compare October 10, 2025 14:44
@MaciejKaras MaciejKaras removed their request for review October 10, 2025 14:58
@viveksinghggits viveksinghggits force-pushed the publish-chart-oci-prpatches branch from e885549 to 92d2fe6 Compare October 14, 2025 08:22
@viveksinghggits viveksinghggits force-pushed the e2e-tests-on-published-chart branch 2 times, most recently from 464fc81 to 5d55e9f Compare October 15, 2025 14:43
@viveksinghggits viveksinghggits changed the title E2e tests on published chart Run MCK E2E tests against OCI published helm chart instead of local chart Oct 17, 2025
@viveksinghggits viveksinghggits changed the title Run MCK E2E tests against OCI published helm chart instead of local chart [CLOUDP-352109] Run MCK E2E tests against OCI published helm chart Oct 17, 2025
Base automatically changed from publish-chart-oci-prpatches to master October 17, 2025 11:34
@viveksinghggits viveksinghggits force-pushed the e2e-tests-on-published-chart branch from b485e0c to e81238f Compare October 17, 2025 12:19
@github-actions
Copy link

github-actions bot commented Oct 17, 2025 •
edited
Loading

⚠️ (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.
  • ReplicaSet: Blocked disabling TLS and changing member count simultaneously. These operations must now be applied separately to prevent configuration inconsistencies.

Other Changes

  • kubectl-mongodb plugin: cosign, the signing tool that is used to sign kubectl-mongodb plugin binaries, has been updated to version 3.0.2. With this change, released binaries will be bundled with .bundle files containing both signature and certificate information. For more information on how to verify signatures using new cosign version please refer to -> https://github.com/sigstore/cosign/blob/v3.0.2/doc/cosign_verify-blob.md

@viveksinghggits viveksinghggits marked this pull request as ready for review October 17, 2025 13:48
MaciejKaras
MaciejKaras reviewed Oct 22, 2025
View reviewed changes
MaciejKaras
MaciejKaras reviewed Oct 22, 2025
View reviewed changes
MaciejKaras
MaciejKaras reviewed Oct 22, 2025
View reviewed changes
MaciejKaras
MaciejKaras reviewed Oct 22, 2025
View reviewed changes
MaciejKaras
MaciejKaras reviewed Oct 22, 2025
View reviewed changes
docker/mongodb-kubernetes-tests/kubetester/operator.py
helm_args: Optional[Dict] = None,
helm_options: Optional[List[str]] = None,
helm_chart_path: Optional[str] = "helm_chart",
helm_chart_path: Optional[str] = None,
Copy link
Collaborator

@MaciejKaras MaciejKaras Oct 22, 2025

Choose a reason for hiding this comment

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

q: how can we instruct this function to use local helm charts for local tests? Ideally we should be able to set this somewhere in the context files and override it by default locally, for example by specifying it in local-defaults-context.

Copy link
Contributor Author

@viveksinghggits viveksinghggits Oct 22, 2025

Choose a reason for hiding this comment

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

Yes, that's the plan. I am working on the change that will make sure that we are able to run the tests locally. Will push the changes soon.

@viveksinghggits viveksinghggits force-pushed the e2e-tests-on-published-chart branch from e2a0c99 to 5cb41a0 Compare October 22, 2025 15:19
viveksinghggits
viveksinghggits commented Oct 24, 2025
View reviewed changes
MaciejKaras
MaciejKaras reviewed Oct 28, 2025
View reviewed changes
@MaciejKaras MaciejKaras force-pushed the e2e-tests-on-published-chart branch from d32d374 to f53071f Compare October 30, 2025 11:28
@MaciejKaras MaciejKaras force-pushed the e2e-tests-on-published-chart branch from 5bdab30 to c482251 Compare October 30, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaciejKaras MaciejKaras MaciejKaras left review comments

@fealebenpae fealebenpae Awaiting requested review from fealebenpae fealebenpae is a code owner automatically assigned from mongodb/kubernetes-hosted

@lsierant lsierant Awaiting requested review from lsierant

+1 more reviewer

@semgrep-code-mongodb semgrep-code-mongodb[bot] semgrep-code-mongodb[bot] left review comments

Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@viveksinghggits @MaciejKaras