Skip to content

Conversation

@mjwolf
Copy link
Contributor

@mjwolf mjwolf commented Oct 30, 2025

Proposed commit message

Update documentation for the fortinet_fortigate integration. The documentation now more complete, with expanded information on supported use-cases and data types that can be collected, more details on how to completely configure the integration, and more troubleshooting information.

This also includes a new packages/fortinet_fortigate/docs/knowledge_base/service_info.md file, which will be included in the built package, and can be used by other tools which use information from the package, such as LLM tools.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
    - [ ] I have verified that any added dashboard complies with Kibana's Dashboard good practices

Screenshots

Screenshot of the updated documentation as rendered in Kibana:
fortigate-docs

@mjwolf mjwolf requested a review from a team as a code owner October 30, 2025 22:30
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:fortinet_fortigate Fortinet FortiGate Firewall Logs Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Oct 31, 2025
@elasticmachine
Copy link

Pinging @elastic/integration-experience (Team:Integration-Experience)

@mjwolf mjwolf self-assigned this Oct 31, 2025
## Note
This integration is compatible with Elastic Stack version 8.11.0 or higher.

### How it works
Copy link
Contributor

Choose a reason for hiding this comment

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

@mjwolf should the titles be title case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, we didn't use title case in the template or the Palo Alto documentation

Copy link
Contributor

@qcorporation qcorporation left a comment

Choose a reason for hiding this comment

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

Overall - this is really good. A major step above the existing documentation! Bravo

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @mjwolf

@mjwolf mjwolf merged commit 7f44034 into elastic:main Nov 14, 2025
8 checks passed
@mjwolf mjwolf deleted the fortigate-update-docs branch November 14, 2025 21:48
@elastic-vault-github-plugin-prod

Package fortinet_fortigate - 1.36.2 containing this change is available at https://epr.elastic.co/package/fortinet_fortigate/1.36.2/

tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
Update documentation for the fortinet_fortigate integration. The documentation now more complete, with expanded information on supported use-cases and data types that can be collected, more details on how to completely configure the integration, and more troubleshooting information.

This also includes a new packages/fortinet_fortigate/docs/knowledge_base/service_info.md file, which will be included in the built package, and can be used by other tools which use information from the package, such as LLM tools.
@andrewkroh andrewkroh added the bugfix Pull request that fixes a bug issue label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:fortinet_fortigate Fortinet FortiGate Firewall Logs Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants