Skip to content

Commit 723e439

Browse files
authored
Fix for wrong repository name for ops-manager in build_info.json (#405)
# Summary I've notices that we had a wrong repository name for ops-manager in `build_info.json` -> `mongodb-enterprise-ops-manager` instead of `mongodb-enterprise-ops-manager-ubi`. If we would run the OM release it would be pushed to the wrong repository where only context files reside. ## Proof of Work Passing CI. ## 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
1 parent 85d4a02 commit 723e439

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

build_info.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -287,23 +287,23 @@
287287
"dockerfile-path": "docker/mongodb-enterprise-ops-manager/Dockerfile.atomic",
288288
"patch": {
289289
"version": "om-version-from-release.json",
290-
"repositories": ["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager"],
290+
"repositories": ["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager-ubi"],
291291
"platforms": [
292292
"linux/amd64"
293293
]
294294
},
295295
"staging": {
296296
"version": "om-version-from-release.json",
297297
"sign": true,
298-
"repositories": ["268558157000.dkr.ecr.us-east-1.amazonaws.com/staging/mongodb-enterprise-ops-manager"],
298+
"repositories": ["268558157000.dkr.ecr.us-east-1.amazonaws.com/staging/mongodb-enterprise-ops-manager-ubi"],
299299
"platforms": [
300300
"linux/amd64"
301301
]
302302
},
303303
"manual_release": {
304304
"sign": true,
305305
"olm-tag": true,
306-
"repositories": ["quay.io/mongodb/mongodb-enterprise-ops-manager"],
306+
"repositories": ["quay.io/mongodb/mongodb-enterprise-ops-manager-ubi"],
307307
"platforms": [
308308
"linux/amd64"
309309
]

scripts/release/tests/build_info_test.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ def test_load_build_info_development(git_repo: Repo):
8686
dockerfile_path="docker/mongodb-agent/Dockerfile.atomic",
8787
),
8888
"ops-manager": ImageInfo(
89-
repositories=["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager"],
89+
repositories=["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager-ubi"],
9090
platforms=["linux/amd64"],
9191
version="om-version-from-release.json",
9292
dockerfile_path="docker/mongodb-enterprise-ops-manager/Dockerfile.atomic",
@@ -187,7 +187,7 @@ def test_load_build_info_patch(git_repo: Repo):
187187
dockerfile_path="docker/mongodb-agent/Dockerfile.atomic",
188188
),
189189
"ops-manager": ImageInfo(
190-
repositories=["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager"],
190+
repositories=["268558157000.dkr.ecr.us-east-1.amazonaws.com/dev/mongodb-enterprise-ops-manager-ubi"],
191191
platforms=["linux/amd64"],
192192
version="om-version-from-release.json",
193193
dockerfile_path="docker/mongodb-enterprise-ops-manager/Dockerfile.atomic",
@@ -306,7 +306,9 @@ def test_load_build_info_staging(git_repo: Repo):
306306
sign=True,
307307
),
308308
"ops-manager": ImageInfo(
309-
repositories=["268558157000.dkr.ecr.us-east-1.amazonaws.com/staging/mongodb-enterprise-ops-manager"],
309+
repositories=[
310+
"268558157000.dkr.ecr.us-east-1.amazonaws.com/staging/mongodb-enterprise-ops-manager-ubi"
311+
],
310312
platforms=["linux/amd64"],
311313
version="om-version-from-release.json",
312314
dockerfile_path="docker/mongodb-enterprise-ops-manager/Dockerfile.atomic",
@@ -437,7 +439,7 @@ def test_load_build_info_manual_release(git_repo: Repo):
437439
sign=True,
438440
),
439441
"ops-manager": ImageInfo(
440-
repositories=["quay.io/mongodb/mongodb-enterprise-ops-manager"],
442+
repositories=["quay.io/mongodb/mongodb-enterprise-ops-manager-ubi"],
441443
platforms=["linux/amd64"],
442444
version=None, # Version is None for manual_release scenario
443445
dockerfile_path="docker/mongodb-enterprise-ops-manager/Dockerfile.atomic",

0 commit comments

Comments
 (0)