-
Notifications
You must be signed in to change notification settings - Fork 14
CLOUDP-338084 - removing and refactoring agent matrix from pipeline.py and atomic_pipeline.py #346
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
Merged
Merged
Changes from 13 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d95b296
removing pipeline and atomic matrix
nammn 3357e31
removing pipeline and atomic matrix
nammn a2bb281
fix bc
nammn 832739c
fix python tests
nammn c581be0
push agent
nammn 115b072
push agent
nammn 7d78b09
add a way to push all agents
nammn 18e2687
add a way to push all agents
nammn a08ab76
Merge branch 'master' into remove-static-pipeline
nammn 3477571
push agent
nammn 8f71280
Merge branch 'remove-static-pipeline' of github.com:mongodb/mongodb-k…
nammn 931a774
change release folder
nammn 5ad2a77
linter
nammn 3ad71b7
linter
nammn c3dd864
add release for agent
nammn 937e953
add release for agent
nammn 365f97e
add release for agent support back to pipeline.py
nammn 7615478
add logger
nammn 37c5369
linter
nammn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -61,14 +61,14 @@ variables: | |
variant: init_test_run | ||
- name: build_test_image | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
variant: init_test_run | ||
- name: build_readiness_probe_image | ||
variant: init_test_run | ||
- name: build_upgrade_hook_image | ||
variant: init_test_run | ||
- name: build_mco_test_image | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we still use all run this on every patch, the script just checks whether its required and potentially skips it then if there are no changes. Why still run it? |
||
variant: init_test_run | ||
|
||
- &setup_group | ||
setup_group_can_fail_task: true | ||
|
@@ -347,19 +347,6 @@ tasks: | |
image_name: init-ops-manager | ||
include_tags: release | ||
|
||
- name: release_agent_operator_release | ||
tags: [ "image_release" ] | ||
allowed_requesters: [ "patch", "github_tag" ] | ||
commands: | ||
- func: clone | ||
- func: setup_building_host | ||
- func: quay_login | ||
- func: setup_docker_sbom | ||
- func: legacy_pipeline | ||
vars: | ||
image_name: agent | ||
include_tags: release | ||
|
||
# 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) | ||
|
@@ -392,21 +379,6 @@ tasks: | |
working_dir: src/github.com/mongodb/mongodb-kubernetes | ||
binary: scripts/evergreen/precommit_bump.sh | ||
|
||
# Pct only triggers this variant once a new agent image is out | ||
# these releases the agent with the operator suffix (not patch id) on ecr to allow for digest pinning to pass. | ||
# For this to work, we rely on skip_tags which is used to determine whether | ||
# we want to release on quay or not, in this case - ecr instead. | ||
# We rely on the init_database from ecr for the agent x operator images. | ||
# This runs on agent releases that are not concurrent with operator releases. | ||
- name: release_agents_on_ecr_conditional | ||
commands: | ||
- func: clone | ||
- func: run_task_conditionally | ||
vars: | ||
condition_script: scripts/evergreen/should_release_agents_on_ecr.sh | ||
variant: init_release_agents_on_ecr | ||
task: release_agents_on_ecr | ||
|
||
- name: release_agents_on_ecr | ||
# 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" ] | ||
|
@@ -420,20 +392,16 @@ tasks: | |
skip_tags: release | ||
|
||
- name: release_all_agents_on_ecr | ||
# this enables us to run this manually (patch) and release all agent versions to ECR | ||
# it's needed during operator new version release process - e2e tests (especially olm tests) | ||
# will look for agent with new operator version suffix, but during PR checks we only build | ||
# agent versions for most recent major OM versions and the tests will fail. Before running the PR | ||
# we have to manually release all agents to ECR by triggering this patch | ||
# this enables us to run this manually (patch) and release all agent versions to ECR to verify | ||
# Dockerfile, script changes etc. | ||
allowed_requesters: [ "patch" ] | ||
commands: | ||
- func: clone | ||
- func: setup_building_host | ||
- func: legacy_pipeline | ||
- func: pipeline | ||
vars: | ||
image_name: agent-pct | ||
skip_tags: release | ||
all_agents: true | ||
image_name: agent | ||
all_agents: "--all-agents" | ||
|
||
- name: build_test_image | ||
commands: | ||
|
@@ -1334,8 +1302,7 @@ buildvariants: | |
variant: init_test_run | ||
- name: build_init_database_image_ubi | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
variant: init_test_run | ||
|
||
tasks: | ||
- name: e2e_custom_domain_task_group | ||
|
||
|
@@ -1369,8 +1336,7 @@ buildvariants: | |
variant: init_test_run | ||
- name: build_init_database_image_ubi | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
variant: init_test_run | ||
|
||
run_on: | ||
- ubuntu2204-small | ||
tasks: | ||
|
@@ -1617,8 +1583,7 @@ buildvariants: | |
variant: init_tests_with_olm | ||
- name: build_init_database_image_ubi | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
variant: init_test_run | ||
|
||
tasks: | ||
- name: e2e_kind_olm_group | ||
|
||
|
@@ -1683,18 +1648,6 @@ buildvariants: | |
- name: build_upgrade_hook_image | ||
- name: prepare_aws | ||
|
||
- name: init_release_agents_on_ecr | ||
display_name: init_release_agents_on_ecr | ||
# 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" ] | ||
tags: [ "release_agents_on_ecr" ] | ||
# We want that to run first and finish asap. Digest pinning depends on this to succeed. | ||
priority: 70 | ||
run_on: | ||
- ubuntu2204-large | ||
tasks: | ||
- name: release_agents_on_ecr_conditional | ||
|
||
- name: run_pre_commit | ||
priority: 70 | ||
display_name: run_pre_commit | ||
|
@@ -1722,8 +1675,7 @@ buildvariants: | |
variant: init_test_run | ||
- name: build_init_om_images_ubi | ||
variant: init_test_run | ||
- name: build_agent_images_ubi | ||
variant: init_test_run | ||
|
||
run_on: | ||
- ubuntu2204-small | ||
tasks: | ||
|
@@ -1809,13 +1761,6 @@ buildvariants: | |
- name: release_init_database | ||
- name: release_init_ops_manager | ||
- name: release_database | ||
# Once we release the operator, we will also release the init databases, we require them to be out first | ||
# such that we can reference them and retrieve those binaries. | ||
# Since we immediately run daily rebuild after creating the image, we can ensure that the init_database is out | ||
# such that the agent image build can use it. | ||
- name: release_agent_operator_release | ||
depends_on: | ||
- name: release_init_database | ||
|
||
- name: preflight_release_images | ||
display_name: preflight_release_images | ||
|
@@ -1847,13 +1792,13 @@ buildvariants: | |
|
||
# It will be called by pct while bumping the agent cloud manager image | ||
- name: release_agent | ||
display_name: (Static Containers) Release Agent matrix | ||
display_name: release_agent | ||
tags: [ "release_agent" ] | ||
run_on: | ||
- release-ubuntu2204-large # This is required for CISA attestation https://jira.mongodb.org/browse/DEVPROD-17780 | ||
depends_on: | ||
- variant: init_release_agents_on_ecr | ||
name: '*' | ||
- variant: init_test_run | ||
name: build_agent_images_ubi # this ensures the agent gets released to ECR as well | ||
- variant: e2e_multi_cluster_kind | ||
name: '*' | ||
- variant: e2e_static_multi_cluster_2_clusters | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
all_agents expansion is empty, but in the manual release agents on ecr variant it will be set to
--all-agents