-
Notifications
You must be signed in to change notification settings - Fork 32
CLOUDP-295785 - release tasks integration with atomic_pipeline.py
#344
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.
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
Conversation
MCK 1.3.0 Release NotesNew FeaturesMulti-Architecture SupportWe've added comprehensive multi-architecture support for the kubernetes operator. This enhancement enables deployment on IBM Power (ppc64le) and IBM Z (s390x) architectures alongside
Bug Fixes
Other Changes
|
93648fd to
9ced848
Compare
1f997c8 to
1f0f78d
Compare
77524ca to
ffbec20
Compare
viveksinghggits
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, can we please add the ToDos that we talked about?
nammn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! A few question I would like to have answered though
| scenario = BuildScenario.RELEASE | ||
| logger.info(f"Build scenario: {scenario} (git_tag: {git_tag})") | ||
| elif is_patch or is_evg: | ||
| elif is_patch and is_evg: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the is_evg required? Doesn't is_patch already say that its a patch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was a bug because in next condition we check for is_evg, which would not be entered if previous condition would be is_patch or is_evg. It is still commented out, but I wanted to fix it. We can also remove the is_evg from this check, but it is maybe more verbose as is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i will leave it to you, for me it reads weird as it seems redundant, but not blocking!
| tags: [ "e2e_smoke_release_test_suite" ] | ||
| allowed_requesters: [ "patch", "github_tag" ] | ||
| run_on: | ||
| - release-ubuntu2404-small # This is required for CISA attestation https://jira.mongodb.org/browse/DEVPROD-17780 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I doubt we need this for the test image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought about scenario when someone interferes with our test image and it will give false positive, although the image was corrupted.
# Summary This PR cleans up all sonar and pipeline.py related code. It is not necessary after we have merged #344 which moves all remaining tasks to use new `atomic-pipeline` logic. Changes include: - removed unused inventory .yaml files - removed sonar lib and pipeline.py code - replaced `Dockerfile.atomic` with `Dockerfile`. Other Dockerfiles were deleted because now not used - removed unused evergreen extensions from the code ## Proof of Work Passing CI tests is enough. ## 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? - use `skip-changelog` label if not needed - refer to [Changelog files and Release Notes](https://github.com/mongodb/mongodb-kubernetes/blob/master/CONTRIBUTING.md#changelog-files-and-release-notes) section in CONTRIBUTING.md for more details
…ongodb#344) # Summary⚠️ **Important notice** This PR contains some changes from mongodb#336, but they are not used yet and don't impact the PRs or patches. They are included because previously this PR was stacked on the staging PR and it is much easier to include them. The changes that are included: - `latest_tag` support - this is needed for staging builds, but like mentioned earlier, staging builds are not yet used - replace `268558157000.dkr.ecr.us-east-1.amazonaws.com/dev` with `BASE_REPO_URL`. This will be used to distinguish different repo urls: dev, staging and release. Currently hardcoded to `268558157000.dkr.ecr.us-east-1.amazonaws.com/dev` --- **This change is made to unblock the release of MCK 1.3.0. It is not final state of the release mechanism and most of it will be replaced by image promotion process.** Created new `.evergreen-release.yml` file that contains all release tasks including integration with `kubectl-mongodb` plugin release task. All of the variants are triggered only when `github_tag` is added. Additional changes: - each released image will be also released with additional `olm_tag` that has dynamic timestamp part. It will prevent accidental overriding the tags used by OLM. The tag syntax is `{version}-olm-{timestamp_suffix}` where timestamp suffix is in `%Y%m%d%H%M%S` format - created separate `release_operator_pipeline` evergreen function that uses `release` build scenario and version provided by `git_tag` - fixed and bumped preflight script ## Proof of Work List of tasks that are triggered when doing manual patch: <img width="2036" height="1017" alt="Screenshot 2025-09-03 at 11 00 16" src="https://github.com/user-attachments/assets/b3e7e707-3929-4f88-bc4f-2f998a16482a" />⚠️ This PR was tested by running evergreen command locally: ``` sudo evergreen patch -p mongodb-kubernetes -a release -d "Release test" -f -y -u --browse --path .evergreen.yml --param RELEASE_OPERATOR_VERSION=1.3.0-rc ``` Link to evg job -> https://spruce.mongodb.com/version/68b81b45285a950007bc8398 ## Checklist - [x] Have you linked a jira ticket and/or is the ticket in the title? - [x] Have you checked whether your jira ticket required DOCSP changes? - [x] Have you added changelog file? - use `skip-changelog` label if not needed - refer to [Changelog files and Release Notes](https://github.com/mongodb/mongodb-kubernetes/blob/master/CONTRIBUTING.md#changelog-files-and-release-notes) section in CONTRIBUTING.md for more details
# Summary This PR cleans up all sonar and pipeline.py related code. It is not necessary after we have merged mongodb#344 which moves all remaining tasks to use new `atomic-pipeline` logic. Changes include: - removed unused inventory .yaml files - removed sonar lib and pipeline.py code - replaced `Dockerfile.atomic` with `Dockerfile`. Other Dockerfiles were deleted because now not used - removed unused evergreen extensions from the code ## Proof of Work Passing CI tests is enough. ## 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? - use `skip-changelog` label if not needed - refer to [Changelog files and Release Notes](https://github.com/mongodb/mongodb-kubernetes/blob/master/CONTRIBUTING.md#changelog-files-and-release-notes) section in CONTRIBUTING.md for more details
Summary
This PR contains some changes from #336, but they are not used yet and don't impact the PRs or patches. They are included because previously this PR was stacked on the staging PR and it is much easier to include them. The changes that are included:
latest_tagsupport - this is needed for staging builds, but like mentioned earlier, staging builds are not yet used268558157000.dkr.ecr.us-east-1.amazonaws.com/devwithBASE_REPO_URL. This will be used to distinguish different repo urls: dev, staging and release. Currently hardcoded to268558157000.dkr.ecr.us-east-1.amazonaws.com/devThis change is made to unblock the release of MCK 1.3.0. It is not final state of the release mechanism and most of it will be replaced by image promotion process.
Created new
.evergreen-release.ymlfile that contains all release tasks including integration withkubectl-mongodbplugin release task. All of the variants are triggered only whengithub_tagis added.Additional changes:
olm_tagthat has dynamic timestamp part. It will prevent accidental overriding the tags used by OLM. The tag syntax is{version}-olm-{timestamp_suffix}where timestamp suffix is in%Y%m%d%H%M%Sformatrelease_operator_pipelineevergreen function that usesreleasebuild scenario and version provided bygit_tagProof of Work
List of tasks that are triggered when doing manual patch:

Link to evg job
-> https://spruce.mongodb.com/version/68b81b45285a950007bc8398
Checklist
skip-changeloglabel if not needed