Skip to content

[patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766#2071

Merged
Alfishan merged 6 commits intomasterfrom
masr-6128-arcgis
Jan 21, 2026
Merged

[patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766#2071
Alfishan merged 6 commits intomasterfrom
masr-6128-arcgis

Conversation

@Alfishan
Copy link
Contributor

@Alfishan Alfishan commented Jan 20, 2026

Description

Adds detection logic to the ArcGIS role to identify if Manage (with Spatial) or Facilities applications are installed and ready. This ensures ArcGIS is installed as a bundled dependency with at least one supported application.

Changes

  • Added tasks to detect ManageApp and FacilitiesApp resources
  • Implemented validation to ensure ArcGIS is installed with at least one application
  • Added debug output showing detected applications and their ready status
  • Enforces valid installation combinations: ArcGIS+MREF, ArcGIS+Manage, or ArcGIS+MREF+Manage

Valid Installation Combinations

ArcGIS must be installed bundled with at least one application:

  • ArcGIS + MREF (Facilities)
  • ArcGIS + Manage (with Spatial component)
  • ArcGIS + MREF + Manage

If ArcGIS is not installed in a bundle, the installation will fail with a clear error message.

Test Results

Testing completed successfully as documented in MASCORE-10766:

  • ✅ Detection of Manage application works correctly
  • ✅ Detection of Facilities (MREF) application works correctly
  • ✅ Validation enforces bundled installation requirement
  • ✅ Debug output displays application detection status
  • ✅ Installation fails appropriately when no supported app is detected

Related Issues

  • MASCORE-10766: Update ansible-devops to detect Manage and MREF during installation of ArcGIS
  • MASCORE-10768: Update dependencies on Tekton for ArcGIS installation
  • MASCORE-10767: Update installation of ArcGIS on CLI

Related PRs

Files Modified

  • ibm/mas_devops/roles/arcgis/tasks/main.yml - Added application detection and validation logic

Alfishan and others added 5 commits December 31, 2025 18:35
@Alfishan Alfishan changed the title [patch] Detect Manage and Facilities for ArcGIS installation [patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766 Jan 20, 2026
@Alfishan Alfishan marked this pull request as ready for review January 20, 2026 08:48
@Alfishan Alfishan requested a review from a team as a code owner January 20, 2026 08:48
@Alfishan Alfishan requested a review from terc1997 January 20, 2026 08:48
durera
durera approved these changes Jan 21, 2026
View reviewed changes
Copy link
Contributor

@durera durera left a comment

Choose a reason for hiding this comment

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

Approved, this is a good step in the right direction, but it must be noted that there are still a couple of issues that we need to address in a follow up.

Code dupliation/non-standard install:

arcgis is implemented as a MAS application, therefore it should be installed using suite_app_install and suite_app_config roles

Use of banned cluster roles

Cluster roles for apps have always been forbidden, but from 9.2 will be blocked entirely, we can see two cases where the ansible code is creating cluster roles.

  • roles/arcgis/templates/ws-cluster-rbac.yml.j2
  • roles/arcgis/templates/ingress-cluster-rbac.yml.j2

@Alfishan Alfishan added this pull request to the merge queue Jan 21, 2026
Merged via the queue into master with commit 37bb5dc Jan 21, 2026
4 checks passed
@Alfishan Alfishan deleted the masr-6128-arcgis branch January 21, 2026 11:38
karol-czarnecki added a commit that referenced this pull request Jan 23, 2026
@karol-czarnecki
Squashed commit of the following:
62b3310 
commit 191ef3c
Author: JigarPatel2691 <66988540+JigarPatel2691@users.noreply.github.com>
Date:   Fri Jan 23 16:10:53 2026 +0530

    [patch] Simplify assertion for COS resource ID check (#2079)

commit 485e7a9
Author: Társis Augusto <64480693+terc1997@users.noreply.github.com>
Date:   Thu Jan 22 10:31:12 2026 -0300

    [patch] Fix broken conditionals (#2042)

commit 37bb5dc
Author: Alfishan Salar <Alfishan@gmail.com>
Date:   Wed Jan 21 17:06:55 2026 +0530

    [patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766 (#2071)

    Co-authored-by: Mahammad Alfishan Salar <alfishan@ibm.com>

commit cbebf54
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:43:38 2026 +0530

    [minor] Support catalog-controlled Db2-channel version fallback in Ansible (#2064)

commit 24541be
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:40:21 2026 +0530

    [patch] Updating schedule for Db2 rotate versions in release verification pipeline (#2072)

commit 1fd6364
Author: jainyjoseph <jainy587@gmail.com>
Date:   Fri Jan 16 22:42:20 2026 +0530

    [minor] Rotate Db2 versions in release verification pipeline (#2065)

commit b63f3b1
Author: David Parker <parkerda@uk.ibm.com>
Date:   Thu Jan 15 12:06:40 2026 +0000

    [doc] More documentation enhancements (#2059)

commit 57f3be5
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 21:40:39 2026 +0000

    [doc] Enhance Role Documentation - Phase 1 (#2054)

commit 27789b9
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 11:56:02 2026 +0000

    [doc] Overhaul role docs (#2053)
karol-czarnecki added a commit that referenced this pull request Jan 26, 2026
@karol-czarnecki
Squashed commit of the following:
463dc9f 
commit 35fbaaa
Author: MaxRobak <61717686+MaxRobak@users.noreply.github.com>
Date:   Fri Jan 23 14:53:38 2026 -0500

    [patch] Mongodb subnet nonetype error (#2081)

    Co-authored-by: Max Robakiewicz <Max.Robakiewicz@ibm.com>

commit 191ef3c
Author: JigarPatel2691 <66988540+JigarPatel2691@users.noreply.github.com>
Date:   Fri Jan 23 16:10:53 2026 +0530

    [patch] Simplify assertion for COS resource ID check (#2079)

commit 485e7a9
Author: Társis Augusto <64480693+terc1997@users.noreply.github.com>
Date:   Thu Jan 22 10:31:12 2026 -0300

    [patch] Fix broken conditionals (#2042)

commit 37bb5dc
Author: Alfishan Salar <Alfishan@gmail.com>
Date:   Wed Jan 21 17:06:55 2026 +0530

    [patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766 (#2071)

    Co-authored-by: Mahammad Alfishan Salar <alfishan@ibm.com>

commit cbebf54
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:43:38 2026 +0530

    [minor] Support catalog-controlled Db2-channel version fallback in Ansible (#2064)

commit 24541be
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:40:21 2026 +0530

    [patch] Updating schedule for Db2 rotate versions in release verification pipeline (#2072)

commit 1fd6364
Author: jainyjoseph <jainy587@gmail.com>
Date:   Fri Jan 16 22:42:20 2026 +0530

    [minor] Rotate Db2 versions in release verification pipeline (#2065)

commit b63f3b1
Author: David Parker <parkerda@uk.ibm.com>
Date:   Thu Jan 15 12:06:40 2026 +0000

    [doc] More documentation enhancements (#2059)

commit 57f3be5
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 21:40:39 2026 +0000

    [doc] Enhance Role Documentation - Phase 1 (#2054)

commit 27789b9
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 11:56:02 2026 +0000

    [doc] Overhaul role docs (#2053)
karol-czarnecki added a commit that referenced this pull request Jan 27, 2026
@karol-czarnecki
Squashed commit of the following:
d916962 
commit a984094
Author: mnivedithaa <mnivedithaa@gmail.com>
Date:   Tue Jan 27 15:28:28 2026 +0530

    [patch] Check for None in mongo and kafka roles (#2087)

    Co-authored-by: Nivedithaa Mahendran <nivedithaa.mahendran@ibm.com>

commit 678c277
Author: Andrew Whitfield <whitfiea@uk.ibm.com>
Date:   Mon Jan 26 14:55:36 2026 +0000

    [patch] Fix bool checks in backup_restore and ocs provision (#2083)

commit 88207b5
Author: Andrew Whitfield <whitfiea@uk.ibm.com>
Date:   Mon Jan 26 10:20:21 2026 +0000

    [skip ee] Fix skip ee condition (#2084)

commit 35fbaaa
Author: MaxRobak <61717686+MaxRobak@users.noreply.github.com>
Date:   Fri Jan 23 14:53:38 2026 -0500

    [patch] Mongodb subnet nonetype error (#2081)

    Co-authored-by: Max Robakiewicz <Max.Robakiewicz@ibm.com>

commit 191ef3c
Author: JigarPatel2691 <66988540+JigarPatel2691@users.noreply.github.com>
Date:   Fri Jan 23 16:10:53 2026 +0530

    [patch] Simplify assertion for COS resource ID check (#2079)

commit 485e7a9
Author: Társis Augusto <64480693+terc1997@users.noreply.github.com>
Date:   Thu Jan 22 10:31:12 2026 -0300

    [patch] Fix broken conditionals (#2042)

commit 37bb5dc
Author: Alfishan Salar <Alfishan@gmail.com>
Date:   Wed Jan 21 17:06:55 2026 +0530

    [patch] Detect Manage and Facilities for ArcGIS installation : MASCORE-10766 (#2071)

    Co-authored-by: Mahammad Alfishan Salar <alfishan@ibm.com>

commit cbebf54
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:43:38 2026 +0530

    [minor] Support catalog-controlled Db2-channel version fallback in Ansible (#2064)

commit 24541be
Author: jainyjoseph <jainy587@gmail.com>
Date:   Wed Jan 21 00:40:21 2026 +0530

    [patch] Updating schedule for Db2 rotate versions in release verification pipeline (#2072)

commit 1fd6364
Author: jainyjoseph <jainy587@gmail.com>
Date:   Fri Jan 16 22:42:20 2026 +0530

    [minor] Rotate Db2 versions in release verification pipeline (#2065)

commit b63f3b1
Author: David Parker <parkerda@uk.ibm.com>
Date:   Thu Jan 15 12:06:40 2026 +0000

    [doc] More documentation enhancements (#2059)

commit 57f3be5
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 21:40:39 2026 +0000

    [doc] Enhance Role Documentation - Phase 1 (#2054)

commit 27789b9
Author: David Parker <parkerda@uk.ibm.com>
Date:   Tue Jan 13 11:56:02 2026 +0000

    [doc] Overhaul role docs (#2053)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@durera durera durera approved these changes

@terc1997 terc1997 Awaiting requested review from terc1997

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Alfishan @durera