Skip to content

APPSINTG-31 - Deprecate Global Intelligence apps - #7125

Open
nykh wants to merge 4 commits into
mainfrom
nykh_APPSINTG-31_deprecate-gis-apps
Open

APPSINTG-31 - Deprecate Global Intelligence apps#7125
nykh wants to merge 4 commits into
mainfrom
nykh_APPSINTG-31_deprecate-gis-apps

Conversation

@nykh

@nykh nykh commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request removes the deprecated Global Intelligence (GIS) app documentation and all references to these apps across the site:

  • Removes doc pages for these GIS apps:
    • Global Intelligence for CloudTrail DevOps
    • GIS CloudTrail SecOps
    • GIS GuardDuty
  • Removes the Global Intelligence index page and AWS integrations index page
  • Cleans up sidebar entries, redirect rules, and cross-references in related docs (Security Quickstart, Terraform deploy guide, product list)

Should be similar to

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/APPSINTG-31

@nykh
nykh requested review from a team, JV0812, amee-sumo, kimsauce and mafsumo as code owners September 10, 2026 20:26
@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Sep 10, 2026
@nykh
nykh requested a lite review from Copilot September 10, 2026 20:26
@nykh nykh self-assigned this Sep 10, 2026

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.

🟡 Changes recommended

Legacy routes may return 404, and deprecated-app references remain in related documentation.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR removes deprecated Global Intelligence AWS app documentation and related site references.

Changes:

  • Deletes three deprecated app pages and navigation entries.
  • Removes catalog, account-plan, and cross-reference entries.
  • Updates Terraform documentation and redirect mappings.
File summaries
File Summary
sidebars.ts Removes deprecated app navigation entries.
docs/observability/aws/deploy-use-aws-observability/deploy-with-terraform.md Removes the obsolete deprecation note.
docs/integrations/product-list/product-list-a-l.md Removes deprecated app links.
docs/integrations/global-intelligence/index.md Removes deprecated app cards.
docs/integrations/amazon-aws/security-quickstart.md Updates app lists and counts.
docs/integrations/amazon-aws/index.md Removes deprecated app cards.
docs/integrations/amazon-aws/global-intelligence-guardduty.md Deletes deprecated documentation.
docs/integrations/amazon-aws/global-intelligence-cloudtrail-secops.md Deletes deprecated documentation.
docs/integrations/amazon-aws/global-intelligence-cloudtrail-devops.md Deletes deprecated documentation.
docs/integrations/account-plans.md Removes deprecated app rows.
cid-redirects.json Removes legacy URL mappings; affected routes need retargeting.
Review details

Suppressed comments (3)

cid-redirects.json:2661

  • These CID mappings preserve old Global Intelligence URLs. Removing them makes the published links return 404 after the app pages are deleted. Keep the mappings and point them to a surviving landing page instead.
  "/cid/6068": "/docs/integrations/global-intelligence",

cid-redirects.json:3735

  • This legacy documentation URL will return 404 after its target page is deleted. Keep the redirect and retarget it to the surviving AWS integrations index or another relevant landing page.
  "/Release-Notes/Cloud_SIEM_Enterprise_Release_Notes": "/release-notes-cse",

docs/integrations/global-intelligence/index.md:13

  • docs/security/index.md:40 still describes "Global Intelligence Service for GuardDuty and CloudTrail" as an available feature. Those are the deprecated apps removed here, so the security feature comparison will continue to advertise them after this PR. Update that row to the surviving capability or remove it.
<div className="box-wrapper" >
  • Files reviewed: 11/11 changed files
  • Comments generated: 3
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread cid-redirects.json
Comment thread cid-redirects.json
APPSINTG-31 Remove remaining GIS text references

Remove outdated "Global Intelligence Service for GuardDuty and CloudTrail"
parenthetical from the security feature table and the stale deprecation
notice from the CloudFormation deployment guide.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nykh
nykh force-pushed the nykh_APPSINTG-31_deprecate-gis-apps branch from 11f5655 to 79dca4c Compare September 10, 2026 20:43
@nykh
nykh requested a lite review from Copilot September 10, 2026 20:43

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.

🟡 Changes recommended

The deleted canonical app routes lack redirects to the AWS integrations index and may return 404s.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 13/13 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread cid-redirects.json
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

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.

🟢 Approval recommended

The reviewed documentation removals and reference updates are complete and ready for approval.

Review details
  • Files reviewed: 13/13 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

kimsauce

This comment was marked as duplicate.

@kimsauce
kimsauce self-requested a review September 11, 2026 01:57

@kimsauce kimsauce left a comment

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.

proposed changes #7127 @nykh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants