Skip to content

Conversation

@MaciejKaras
Copy link
Collaborator

@MaciejKaras MaciejKaras commented Oct 21, 2025

Summary

Combines all pipeline.py invocations into single pipeline.sh script which makes it easier to maintain and reduces amount of future bugs. The same principle was applied in .evergreen-functions.yml. Now we have only single pipeline evergreen function. For evergreen extensions I've decided to stick with lower-case variable naming. This is aligning better with existing evg functions + less conflicts with context variables.

This also fixes the bug, where the BUILD_SCENARIO: release for OM and agent release tasks was not properly overriding the default one from evg_private_context.

Proof of Work

  1. Custom rebuild_currently_used_agents task with flags was successful
  2. Image release tasks where successful and released images to quay.io with correct versions
  3. Published OM and agent images using fixed tasks

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?

@MaciejKaras MaciejKaras force-pushed the maciejk/ar-pipeline-sh-refac branch from ba6ef38 to a15e493 Compare October 21, 2025 10:43
@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.

@MaciejKaras MaciejKaras added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Oct 21, 2025
@MaciejKaras MaciejKaras marked this pull request as ready for review October 21, 2025 11:07
@MaciejKaras MaciejKaras requested a review from a team as a code owner October 21, 2025 11:07
@MaciejKaras MaciejKaras requested review from m1kola, mircea-cosbuc and viveksinghggits and removed request for m1kola October 21, 2025 11:07
working_dir: src/github.com/mongodb/mongodb-kubernetes
binary: scripts/release/pipeline_version_upgrade_hook.sh
IMAGE_NAME: ${image_name}
BUILD_SCENARIO_OVERRIDE: ${build_scenario}
Copy link
Contributor

Choose a reason for hiding this comment

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

q: did we add BUILD_SCENARIO_OVERRIDE so that we can build the images for different build scenarios manually?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no, we can still use BUILD_SCENARIO for that and pass it to evergreen cmd. The BUILD_SCENARIO_OVERRIDE is only used if we specify build_scenario in evergreen function call, so we can override BUILD_SCENARIO. Example:

  # pct only triggers this variant once a new agent image is out
  - name: release_agent
    # this enables us to run this variant either manually (patch) which pct does or during an OM bump (github_pr)
    allowed_requesters: [ "patch", "github_pr" ]
    commands:
      - func: clone
      - func: setup_building_host
      - func: quay_login
      - func: setup_docker_sbom
      - func: pipeline
        vars:
          image_name: agent
          build_scenario: release

Copy link
Member

@mircea-cosbuc mircea-cosbuc left a comment

Choose a reason for hiding this comment

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

LGTM, used it for the CM agent release.

@MaciejKaras MaciejKaras merged commit 18f79ff into master Oct 23, 2025
35 of 37 checks passed
@MaciejKaras MaciejKaras deleted the maciejk/ar-pipeline-sh-refac branch October 23, 2025 11:57
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