Skip to content

chore(ci): use ECR for DevProd container images DEVPROD-34208 - #2801

Merged
mabaasit merged 4 commits into
mongodb-js:mainfrom
fetsko:DEVPROD-34208/use-devprod-platforms-ecr
Jul 31, 2026
Merged

chore(ci): use ECR for DevProd container images DEVPROD-34208#2801
mabaasit merged 4 commits into
mongodb-js:mainfrom
fetsko:DEVPROD-34208/use-devprod-platforms-ecr

Conversation

@fetsko

@fetsko fetsko commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

MongoDB Artifactory will be deprecated soon. This migrates from Artifactory to ECR for pulling the DevProd container images.

@mabaasit mabaasit changed the title ci: use ECR for DevProd container images DEVPROD-34208 chore(ci): use ECR for DevProd container images DEVPROD-34208 Jul 31, 2026
@mabaasit
mabaasit marked this pull request as ready for review July 31, 2026 16:16
@mabaasit
mabaasit requested a review from a team as a code owner July 31, 2026 16:16
@mabaasit
mabaasit requested review from Sgrinfy and Copilot July 31, 2026 16:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates Evergreen/CI usage of DevProd container images away from MongoDB Artifactory and toward AWS ECR, aligning the build/signing and SBOM-generation steps with the upcoming Artifactory deprecation.

Changes:

  • Bumps @mongodb-js/signing-utils in packages/build to a newer version likely supporting ECR-backed signing flows.
  • Updates Evergreen functions to assume an IAM role for pulling from DevProd Platforms ECR and to pass AWS credentials into relevant subprocesses.
  • Switches SBOM generation (silkbomb) image pulls/logins from Artifactory to ECR, and adds an ECR login password export for the non-Darwin signing path.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/build/package.json Updates @mongodb-js/signing-utils dependency version.
package-lock.json Locks updated @mongodb-js/signing-utils dependency resolution under the build workspace.
.evergreen/sign-packaged-artifact.sh Adds ECR login password retrieval/export for the non-Darwin signing path.
.evergreen/evergreen.yml.in Assumes ECR-readonly role and passes AWS creds into SBOM/signing subprocesses (template).
.evergreen/download-crypt-shared-and-generate-sbom.sh Moves silkbomb pull/login from Artifactory to ECR.
.evergreen.yml Generated Evergreen config reflecting the same ECR role + env var changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +47 to 49
ECR_LOGIN_PASSWORD="$(aws ecr get-login-password --region us-east-1)"
export ECR_LOGIN_PASSWORD
npm run evergreen-release sign

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this is probably not a terrible idea

Comment on lines 15 to 17
set +x
echo "${ARTIFACTORY_PASSWORD}" | docker login artifactory.corp.mongodb.com --username "${ARTIFACTORY_USERNAME}" --password-stdin
aws ecr get-login-password --region us-east-1 | docker login --username AWS --password-stdin "${ECR_HOST}"
set -x
@codeowners-service-app

Copy link
Copy Markdown

Assigned himanshusinghs for team mongosh-developers because Sgrinfy is out of office.

Comment on lines +47 to 49
ECR_LOGIN_PASSWORD="$(aws ecr get-login-password --region us-east-1)"
export ECR_LOGIN_PASSWORD
npm run evergreen-release sign

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this is probably not a terrible idea

@mabaasit
mabaasit merged commit 414956a into mongodb-js:main Jul 31, 2026
158 of 160 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants