Skip to content

Conversation

viveksinghggits
Copy link
Contributor

@viveksinghggits viveksinghggits commented Oct 8, 2025

Summary

In an effort to start using OCI compatible container registries as our helm chart repos, this PR makes the change package and publish our helm chart to ECR registry. A new task publish_helm_chart is introduced in the build variant init_test_run which packages and publishes the chart.

The details about the versioning of the chart can be found in the document here.

There were two wrappers (shell scripts) introduced as part of this and the reason was we wanted to call the python scripts (helm_registry_login.py and publish_helm_chart.py) from .evergreen-function.yaml passing an argument that's build scenario. Build scenario is already set as env var in the evg host, but there isn't a way to do this, in other words, we couldn't do below from the .evergreen-functions.yaml file

binary: binary: run_python.sh scripts/release/helm_registry_login.sh $build_scenario_that_is_set_as_env_on_host

And because of that we had to write the wrapper that would call the python script with correct argument after reading the env var.

Proof of Work

patch and staging evg runs. Notice at the end of the log that dev workflow is pushing to diff repository and staging to different.

Successful run of init_test_run variant in this PR would also work.

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 14:12
@viveksinghggits viveksinghggits marked this pull request as draft October 8, 2025 14:12
Copy link

github-actions bot commented Oct 8, 2025

⚠️ (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.

Bug Fixes

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

@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 2 times, most recently from 5f0938b to 1b61956 Compare October 10, 2025 14:28
@viveksinghggits viveksinghggits changed the title Publish helm chart to OCI registry for PR patches Publish helm chart to OCI registry for dev/staging workflows Oct 10, 2025
@viveksinghggits viveksinghggits changed the title Publish helm chart to OCI registry for dev/staging workflows [CLOUDP-350669]Publish helm chart to OCI registry for dev/staging workflows Oct 10, 2025
@viveksinghggits viveksinghggits changed the title [CLOUDP-350669]Publish helm chart to OCI registry for dev/staging workflows [CLOUDP-350669] Publish helm chart to OCI registry for dev/staging workflows Oct 10, 2025
@viveksinghggits viveksinghggits marked this pull request as ready for review October 10, 2025 14:39
@viveksinghggits viveksinghggits force-pushed the publish-chart-oci-prpatches branch from e885549 to 92d2fe6 Compare October 14, 2025 08:22
1. Move file from scripts to scripts/relese
2. call python files from .sh files so that we can use build_scenario env var
@viveksinghggits
Copy link
Contributor Author

Hi @MaciejKaras,
This can be looked into again as well.

@viveksinghggits
Copy link
Contributor Author

@anandsyncs can you please have a look as well?

@anandsyncs
Copy link
Contributor

@viveksinghggits It would also be great if you can mention in the PR description what exactly is the issue that prevents us from calling publish_helm_chart.py directly from .evergreen-functions.yaml

@viveksinghggits
Copy link
Contributor Author

@viveksinghggits It would also be great if you can mention in the PR description what exactly is the issue that prevents us from calling publish_helm_chart.py directly from .evergreen-functions.yaml

I can do that.

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.

3 participants