Skip to content

Fix a bug in the script that updates latestOpsManagerAgentMapping in release.json#668

Merged
viveksinghggits merged 4 commits intomasterfrom
fix-update-latestom-agent-mapping
Jan 10, 2026
Merged

Fix a bug in the script that updates latestOpsManagerAgentMapping in release.json#668
viveksinghggits merged 4 commits intomasterfrom
fix-update-latestom-agent-mapping

Conversation

@viveksinghggits
Copy link
Collaborator

@viveksinghggits viveksinghggits commented Dec 30, 2025

Summary

We maintain a field in our release.json called latestOpsManagerAgentMapping that has mapping between OM and agent version of every highest major OM version. This list needs to get updated whenever a new version of OM is released and that why we had a script that would update this field whenever a new version of OM is released.
But that script had a bug because it assumed that OM versions are always going to be in increasing semver order, which is not true. OM team seems to maintain 3 branches (version 6, 7 and 8) and makes releases on them, so they can release 8.0.12 and then go ahead and release 7.0.20. Because of that bug if older versions of OM were released the field latestOpsManagerAgentMapping was not being updated successfully.

This PR fixes that by iterating over all the OM versions in release.json and then creating new latestOpsManagerAgentMapping instead of trying to update it based on latest release version.

Proof of Work

Manual.

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?

@github-actions
Copy link

github-actions bot commented Dec 30, 2025

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.6.2 Release Notes

Bug Fixes

  • Fixed an issue where monitoring agents would fail after disabling TLS on a MongoDB deployment.
  • Persistent Volume Claim resize fix: Fixed an issue where the Operator ignored namespaces when listing PVCs, causing conflicts with resizing PVCs of the same name. Now, PVCs are filtered by both name and namespace for accurate resizing.
  • Fixed a panic that occurred when the domain names for a horizon was empty. Now, if the domain names are not valid (RFC 1123), the validation will fail before reconciling.
  • Fixed an issue where the Operator could crash when TLS certificates are configured using the certificatesSecretsPrefix field without additional TLS settings.

@viveksinghggits viveksinghggits added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Dec 30, 2025
@viveksinghggits viveksinghggits marked this pull request as ready for review December 30, 2025 22:34
@viveksinghggits viveksinghggits requested a review from a team as a code owner December 30, 2025 22:34
Copy link
Collaborator

@nammn nammn left a comment

Choose a reason for hiding this comment

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

LGTM, i suggest some changes to readability and code structure

@viveksinghggits viveksinghggits merged commit 4692bd1 into master Jan 10, 2026
31 of 34 checks passed
@viveksinghggits viveksinghggits deleted the fix-update-latestom-agent-mapping branch January 10, 2026 12:09
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.

3 participants