Skip to content

adding netscaler traps#877

Merged
i3149 merged 2 commits into
mainfrom
adding-ns-traps
Jun 4, 2026
Merged

adding netscaler traps#877
i3149 merged 2 commits into
mainfrom
adding-ns-traps

Conversation

@i3149

@i3149 i3149 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Closes #876

@personal-git-pavlch, is this what you are thinking here?

Copilot AI review requested due to automatic review settings June 3, 2026 22:59

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.

Pull request overview

Adds an initial SNMP trap mapping for Citrix ADC/NetScaler (NS-ROOT-MIB) into the existing NetScaler polling profile, aiming to address missing trap visibility referenced in issue #876.

Changes:

  • Introduces a new traps: section in the NetScaler profile.
  • Defines a trap entry intended to map SSL certificate/key-related varbinds to human-friendly field names.

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


# NS-ROOT-MIB traps
traps:
- trap_oid: .1.3.6.1.4.1.5951.4.1.1.56.1
Comment on lines +881 to +883
- trap_oid: .1.3.6.1.4.1.5951.4.1.1.56.1
trap_name: sslCertKeyTable
drop_undefined: true
Comment on lines +879 to +880
# NS-ROOT-MIB traps
traps:
@personal-git-pavlch

Copy link
Copy Markdown
Contributor

Hi @i3149 , thank you, this is exactly what I was looking for. Just remove dot before - trap_oid: 1.3.6.1.4.1.5951.4.1.1.56.1.

@personal-git-pavlch

Copy link
Copy Markdown
Contributor

Thanks for handling this.

@i3149 i3149 merged commit dc800cd into main Jun 4, 2026
1 check passed
@i3149 i3149 deleted the adding-ns-traps branch June 4, 2026 17:57
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.

Missing Citrix ADC/NetScaler traps

3 participants