Skip to content

[patch] Added appOMUpgradeMode in Facilities#2150

Merged
terc1997 merged 12 commits into
masterfrom
masref-4140
Mar 11, 2026
Merged

[patch] Added appOMUpgradeMode in Facilities#2150
terc1997 merged 12 commits into
masterfrom
masref-4140

Conversation

@sdoshi96
Copy link
Copy Markdown
Contributor

@sdoshi96 sdoshi96 commented Mar 9, 2026

1. Description

Added a new field for Facilities - appOMUpgradeMode

2. Dependencies

Test screenshots are attached in ibm-mas/cli#2101


⚠️ Notes for Reviewers

  • Ensure you have understood the PR guidelines in the Playbook before proceeding with a review.
  • Ensure all sections in the PR template are appropriately completed.

@sdoshi96 sdoshi96 marked this pull request as ready for review March 9, 2026 14:32
@sdoshi96 sdoshi96 requested a review from a team as a code owner March 9, 2026 14:32
Copy link
Copy Markdown
Contributor

@terc1997 terc1997 left a comment

Choose a reason for hiding this comment

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

How important is this flag for Facilities ? I'm asking if it's something that the customer needs to be aware and it's highly relevante please add here as well -> https://github.com/ibm-mas/ansible-devops/blob/master/ibm/mas_devops/playbooks/mas_add_facilities.yml

@sdoshi96
Copy link
Copy Markdown
Contributor Author

sdoshi96 commented Mar 9, 2026

Based on the flag, it will deploy the OM package in MREF. By default, this flag will be set to manual. The warning is just to inform them when they explictly change that flag's value to load-only or automatic, which will overwrite their changes.

By default, we always want them to use manual in production cases. The other values are used only in selected development environments.

@sdoshi96 sdoshi96 self-assigned this Mar 10, 2026
@sdoshi96 sdoshi96 requested a review from terc1997 March 10, 2026 09:58
Copy link
Copy Markdown
Contributor

@terc1997 terc1997 left a comment

Choose a reason for hiding this comment

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

LGTM

@terc1997 terc1997 added this pull request to the merge queue Mar 11, 2026
Merged via the queue into master with commit eec7d00 Mar 11, 2026
5 checks passed
@terc1997 terc1997 deleted the masref-4140 branch March 11, 2026 12:34
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.

2 participants