Skip to content

Conversation

@Kesavaraja67
Copy link

@Kesavaraja67 Kesavaraja67 commented Dec 24, 2025

Description

This PR implements the SNMP Provider, allowing Keep to fetch metrics from SNMP-enabled devices using GET requests.

Commands

/claim #2112

Proof of Work

snmp_provider.py.-.keep.-.Visual.Studio.Code.2025-12-24.13-09-43.mp4

Results

✅ Logic Verified: {'1.3.6.1.2.1.1.1.0': 'Keep-Verified-SNMP-Device'}

Linked Issue

Closes #2112

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 24, 2025

Skipped: This PR does not target one of your configured branches: (refactor/2512-nextjs-15)

@vercel
Copy link

vercel bot commented Dec 24, 2025

Someone 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 the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 24, 2025
@CLAassistant
Copy link

CLAassistant commented Dec 24, 2025

CLA assistant check
All committers have signed the CLA.

@dosubot dosubot bot added the Feature A new feature label Dec 24, 2025
@Kesavaraja67 Kesavaraja67 force-pushed the feat/issue-2112-snmp-provider branch from ab6e7d6 to b8948dd Compare December 24, 2025 08:22
@Kesavaraja67
Copy link
Author

I have updated the commit author email to match my GitHub account, signed the CLA, and linked the issue. All technical checks are now passing! The logic verification video is attached in the description. Ready for review @shahargl.

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 size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🔌 Provider]: SNMP provider

2 participants