Skip to content

Conversation

@Ayush9026
Copy link

@Ayush9026 Ayush9026 commented Apr 12, 2025

/claim #3960
Closes #3960

📑 Description

Nagios Provider Implementation

This PR adds support for Nagios monitoring system integration with Keep. The implementation allows users to:

  • Fetch alerts from Nagios (both host and service status)
  • Acknowledge host and service problems
  • Enable/disable host and service checks
  • Map Nagios severity levels to Keep's alert severity system

Features

  • Full integration with Nagios Core's CGI API
  • Support for both host and service monitoring
  • Proper authentication handling
  • Comprehensive error handling
  • Follows Keep's provider implementation patterns

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

@vercel
Copy link

vercel bot commented Apr 12, 2025

@Ayush9026 is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. Feature A new feature Provider Providers related issues labels Apr 12, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Apr 12, 2025
@Ayush9026 Ayush9026 changed the title Fix/nagios provider 3960 fix: Nagios Provider #3960 Apr 12, 2025
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

  1. why did you delete IncidentActivityComment?
  2. Missing lot of files - docs, icon, etc. did you even try to run it?

@Ayush9026
Copy link
Author

@shahargl sir

Fixing now – thanks for catching that!

@talboren
Copy link
Member

@Ayush9026 are you on fixing the open items here?

we need:

  1. Icon
  2. documentation
  3. fixing non-relevant removed files

@talboren
Copy link
Member

Closing for now as this is stale.

@talboren talboren closed this Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🙋 Bounty claim Feature A new feature Provider Providers related issues size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🔌 Provider]: Nagios Provider

3 participants