Skip to content

Fix AppInspect: Remove index paths for Splunk Cloud compatibility#5

Merged
devinslick merged 2 commits intomainfrom
claude/comprehensive-repo-rewrite-ql8d5
Feb 4, 2026
Merged

Fix AppInspect: Remove index paths for Splunk Cloud compatibility#5
devinslick merged 2 commits intomainfrom
claude/comprehensive-repo-rewrite-ql8d5

Conversation

@devinslick
Copy link
Owner

  • Remove homePath, coldPath, thawedPath from indexes.conf
  • Remove frozenTimePeriodInSecs (admin-controlled setting)
  • Keep only datatype=metric for index definition
  • Add note about Splunk Cloud admin creating the index

https://claude.ai/code/session_014YNv3fKbpjQB5eMZCrHwzi

- Remove homePath, coldPath, thawedPath from indexes.conf
- Remove frozenTimePeriodInSecs (admin-controlled setting)
- Keep only datatype=metric for index definition
- Add note about Splunk Cloud admin creating the index

https://claude.ai/code/session_014YNv3fKbpjQB5eMZCrHwzi
Copilot AI review requested due to automatic review settings February 4, 2026 22:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request addresses Splunk AppInspect requirements by removing index path and retention settings from indexes.conf that are incompatible with Splunk Cloud deployments. The changes ensure the app can pass AppInspect validation for Splunk Cloud compatibility while maintaining functionality for Splunk Enterprise.

Changes:

  • Removed admin-controlled settings (homePath, coldPath, thawedPath, frozenTimePeriodInSecs) from indexes.conf
  • Added documentation comments explaining deployment differences between Splunk Cloud and Enterprise
  • Retained only the essential datatype = metric property for index definition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Remove indexes.conf entirely (Splunk Cloud apps cannot define indexes)
- Update README with manual index creation instructions
- Add instructions for both Enterprise and Cloud environments
- Update Initial Setup section to reflect manual index creation

https://claude.ai/code/session_014YNv3fKbpjQB5eMZCrHwzi
@devinslick devinslick merged commit 4a99f87 into main Feb 4, 2026
1 of 2 checks passed
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