Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SNMP] Add profile for IBM datapower gateway #15371

Merged
merged 4 commits into from
Jul 26, 2023
Merged

[SNMP] Add profile for IBM datapower gateway #15371

merged 4 commits into from
Jul 26, 2023

Conversation

TCheruy
Copy link
Contributor

@TCheruy TCheruy commented Jul 25, 2023

What does this PR do?

Add profile for IBM datapower gateway

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached
  • If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

@TCheruy TCheruy requested review from a team as code owners July 25, 2023 14:43
@ghost ghost added the integration/snmp label Jul 25, 2023
TCheruy
TCheruy commented Jul 25, 2023
View reviewed changes
snmp/metadata.csv
snmp.ibm.dpStatusConnectionsAcceptedtenMinutes,gauge,,,,"[DATAPOWER-STATUS-MIB] The number of connections accepted in the previous 10 minutes.",0,snmp,,
snmp.ibm.dpStatusConnectionsAcceptedtenSeconds,gauge,,,,"[DATAPOWER-STATUS-MIB] The number of connections accepted in the previous 10 seconds.",0,snmp,,
snmp.ibm.dpStatusEnvironmentalSensorschassis1rpm,gauge,,,,"[DATAPOWER-STATUS-MIB] No description yet.",0,snmp,,
snmp.ibm.dpStatusEnvironmentalSensorschassis2rpm,gauge,,,,"[DATAPOWER-STATUS-MIB] No description yet.",0,snmp,,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The MIB description...

@codecov
Copy link

codecov bot commented Jul 25, 2023 •
edited
Loading

Codecov Report

Merging #15371 (bd5968b) into master (86d50e6) will decrease coverage by 0.15%.
The diff coverage is 10.76%.

Flag Coverage Δ
snmp 58.38% <10.76%> (-1.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@github-actions
Copy link

Test Results

       6 files      6 suites   28m 3s ⏱️
   340 tests 340 ✔️     0 💤 0 ❌
1 503 runs  520 ✔️ 983 💤 0 ❌

Results for commit bd5968b.

@TCheruy TCheruy requested a review from AlexandreYang July 25, 2023 16:06
@TCheruy TCheruy mentioned this pull request Jul 25, 2023
Merged
5 tasks
@TCheruy TCheruy merged commit 49cde9f into master Jul 26, 2023
@TCheruy TCheruy deleted the ThibaudC/SNMP-profile-IBM-datapower-gateway branch July 26, 2023 08:28
NouhaManai96 added a commit that referenced this pull request Aug 24, 2023
@NouhaManai96
fix mapping #15371
6fa799a
NouhaManai96 added a commit that referenced this pull request Aug 31, 2023
@NouhaManai96 @AlexandreYang
Fix SNMP profiles metric tags mappings using MIBs (#15668)
a50ecaa 
* fix mapping for apc-netbotz #15136

* for mapping for aruba-wireless-controller #15128

* fix mapping #15482

* fix mapping #15158

* fix mapping #15414

* fix mapping #15312

* fix mapping #15343

* fix mapping #15374

* fix mapping #15354

* fix mapping #15370

* fix mapping #15464

* fix mapping #15326

* fix mapping #15368

* fix mapping #15368

* fix mapping #15396

* fix mapping #15386

* fix mapping #15469

* fix mappings #15265

* fix mapping #15349

* fix mapping #15356

* fix mapping #15371

* fix mapping #15322

* fix mapping #15397

* fix mappings #15466

* fix mapping #15153

* fix mapping cisco

* fix mapping

* fix mapping hp

* fix mapping poweredge

* fix mapping f5

* update servertech: metricnames + metadata +tests

* update apc netbotz tests

* update aruba wlc tests

* update citrix netscaler sdx test

* update dell os10 tests

* update test hpe nimble

* update tests huawei

* update infinera coriant groove tests

* update tests infoblox

* update tests kyocera

* update tests raritan

* update tests sinetica

* update tests sophos

* update test zyxel

* update tests avaya

* update tests cisco ucs

* update tests huawei routers

* update tests ibm datapower

* update tests tripplite ups

* update test avaya cajun

* update test barracuda cloudgen

* update test _cisco_generic

* update test cisco nexus

* update test assert_extend_cisco_generic

* update test dell

* update test hp compaq health

* update test f5-big-ip

* update metrics names

* address reviews

* add changelog

* lint

* update changelog

* fix mib name

* address reviews

* revert to readynasos mib mapping

* address review

* remove tag

* add spacing

* Fix changelog

* update changelog

* fix changelog

* remove unnecessary ip_address formatting

---------

Co-authored-by: Alexandre Yang <alexandre.yang@datadoghq.com>
github-actions bot pushed a commit that referenced this pull request Aug 31, 2023
@NouhaManai96
Fix SNMP profiles metric tags mappings using MIBs (#15668)
f130e03 
* fix mapping for apc-netbotz #15136

* for mapping for aruba-wireless-controller #15128

* fix mapping #15482

* fix mapping #15158

* fix mapping #15414

* fix mapping #15312

* fix mapping #15343

* fix mapping #15374

* fix mapping #15354

* fix mapping #15370

* fix mapping #15464

* fix mapping #15326

* fix mapping #15368

* fix mapping #15368

* fix mapping #15396

* fix mapping #15386

* fix mapping #15469

* fix mappings #15265

* fix mapping #15349

* fix mapping #15356

* fix mapping #15371

* fix mapping #15322

* fix mapping #15397

* fix mappings #15466

* fix mapping #15153

* fix mapping cisco

* fix mapping

* fix mapping hp

* fix mapping poweredge

* fix mapping f5

* update servertech: metricnames + metadata +tests

* update apc netbotz tests

* update aruba wlc tests

* update citrix netscaler sdx test

* update dell os10 tests

* update test hpe nimble

* update tests huawei

* update infinera coriant groove tests

* update tests infoblox

* update tests kyocera

* update tests raritan

* update tests sinetica

* update tests sophos

* update test zyxel

* update tests avaya

* update tests cisco ucs

* update tests huawei routers

* update tests ibm datapower

* update tests tripplite ups

* update test avaya cajun

* update test barracuda cloudgen

* update test _cisco_generic

* update test cisco nexus

* update test assert_extend_cisco_generic

* update test dell

* update test hp compaq health

* update test f5-big-ip

* update metrics names

* address reviews

* add changelog

* lint

* update changelog

* fix mib name

* address reviews

* revert to readynasos mib mapping

* address review

* remove tag

* add spacing

* Fix changelog

* update changelog

* fix changelog

* remove unnecessary ip_address formatting

---------

Co-authored-by: Alexandre Yang <alexandre.yang@datadoghq.com> a50ecaa
iliakur pushed a commit that referenced this pull request Sep 5, 2023
@NouhaManai96 @AlexandreYang @iliakur
Fix SNMP profiles metric tags mappings using MIBs (#15668)
0858837 
* fix mapping for apc-netbotz #15136

* for mapping for aruba-wireless-controller #15128

* fix mapping #15482

* fix mapping #15158

* fix mapping #15414

* fix mapping #15312

* fix mapping #15343

* fix mapping #15374

* fix mapping #15354

* fix mapping #15370

* fix mapping #15464

* fix mapping #15326

* fix mapping #15368

* fix mapping #15368

* fix mapping #15396

* fix mapping #15386

* fix mapping #15469

* fix mappings #15265

* fix mapping #15349

* fix mapping #15356

* fix mapping #15371

* fix mapping #15322

* fix mapping #15397

* fix mappings #15466

* fix mapping #15153

* fix mapping cisco

* fix mapping

* fix mapping hp

* fix mapping poweredge

* fix mapping f5

* update servertech: metricnames + metadata +tests

* update apc netbotz tests

* update aruba wlc tests

* update citrix netscaler sdx test

* update dell os10 tests

* update test hpe nimble

* update tests huawei

* update infinera coriant groove tests

* update tests infoblox

* update tests kyocera

* update tests raritan

* update tests sinetica

* update tests sophos

* update test zyxel

* update tests avaya

* update tests cisco ucs

* update tests huawei routers

* update tests ibm datapower

* update tests tripplite ups

* update test avaya cajun

* update test barracuda cloudgen

* update test _cisco_generic

* update test cisco nexus

* update test assert_extend_cisco_generic

* update test dell

* update test hp compaq health

* update test f5-big-ip

* update metrics names

* address reviews

* add changelog

* lint

* update changelog

* fix mib name

* address reviews

* revert to readynasos mib mapping

* address review

* remove tag

* add spacing

* Fix changelog

* update changelog

* fix changelog

* remove unnecessary ip_address formatting

---------

Co-authored-by: Alexandre Yang <alexandre.yang@datadoghq.com>
lizaerem added a commit that referenced this pull request Sep 20, 2023
@AlexandreYang @NouhaManai96
@lizaerem @jhgilbert
Add assert_all_profile_metrics_and_tags_covered & fix mappings in pro…
3632e04 
…files (#15720)

* fix mapping for apc-netbotz #15136

* for mapping for aruba-wireless-controller #15128

* fix mapping #15482

* fix mapping #15158

* fix mapping #15414

* fix mapping #15312

* fix mapping #15343

* fix mapping #15374

* fix mapping #15354

* fix mapping #15370

* fix mapping #15464

* fix mapping #15326

* fix mapping #15368

* fix mapping #15368

* fix mapping #15396

* fix mapping #15386

* fix mapping #15469

* fix mappings #15265

* fix mapping #15349

* fix mapping #15356

* fix mapping #15371

* fix mapping #15322

* fix mapping #15397

* fix mappings #15466

* fix mapping #15153

* fix mapping cisco

* fix mapping

* fix mapping hp

* fix mapping poweredge

* fix mapping f5

* update servertech: metricnames + metadata +tests

* update apc netbotz tests

* update aruba wlc tests

* update citrix netscaler sdx test

* update dell os10 tests

* update test hpe nimble

* update tests huawei

* update infinera coriant groove tests

* update tests infoblox

* update tests kyocera

* update tests raritan

* update tests sinetica

* update tests sophos

* update test zyxel

* update tests avaya

* update tests cisco ucs

* update tests huawei routers

* update tests ibm datapower

* update tests tripplite ups

* update test avaya cajun

* update test barracuda cloudgen

* update test _cisco_generic

* update test cisco nexus

* update test assert_extend_cisco_generic

* update test dell

* update test hp compaq health

* update test f5-big-ip

* update metrics names

* address reviews

* add changelog

* lint

* update changelog

* fix mib name

* address reviews

* revert to readynasos mib mapping

* address review

* remove tag

* add spacing

* Fix changelog

* update changelog

* fix changelog

* remove unnecessary ip_address formatting

* Add assert_all_profile_metrics_and_tags_covered

* fix empty space

* Add assert_all_profile_metrics_and_tags_covered

* Add to test_profile__checkpoint_firewall_cpu_memory.py

* Add assertions

* better msg

* format

* not use f string

* not use f string

* Add ASSERT_ALL_PROFILE_EXCLUDED_TAGS

* fix cisco profiles tags & snmprec & test

* fix dell-rac snmprec (tag) & test

* fix _generic-entity-sensor snmprec (tags + mapping) & test

* fix alcatel-lucent-ent snmprec (tags + mapping) & test

* fix alcatel-lucent-ind (tags + mapping) & test

* rename anue-packet-broker snmprec to anue

* better test output

* fix aruba-clearpass tags & snmprec & test

* fix aruba-wc snmprec & test

* fix bluecat-server snmprec & test

* fix checkpoint snmprec & test

* fix eaton-epdu profile & test

* fix hp-h3c-switch snmprec & test

* fix huawei-routers snmprec & test

* fix infinera-coriant-groove snmprec & test

* fix juniper-variation test

* fix juniper-pulse-secure tags & snmprec & test

* fix nasuni-filer tags & snmprec & test

* fix meraki-cloud-controller device tag name

* fix netgear-readynas tag & snmprec & test

* fix riverbed-interceptor (comment scalar metric)

* fix servertech-pdu4 snmprec & test

* fix sinetica-eagle-i snmprec & test

* fix western-digital snmprec & test

* revert netgear-readynas & resolve conflict

* Update western-digital-mycloud-ex2-ultra.snmprec

fix type in snmprec for western-digital

* Update infinera-coriant-groove.snmprec

fix type in snmprec for infinera-coriant

* Update netgear-readynas.snmprec

fix type in snmprec for netgear-readynas

* update test_profiles.py

* update core tests

* let arista-switch test pass & fix it separately

* minor changes to profiles

* update _cisco-generic with the tooling

* update alcatel-lucent-ent with the tooling

* update alcatel-lucent-ind with the tooling

* update alcatel-lucent-omni-access-wlc with the tooling

* update aruba-wireless-controller with the tooling

* fix tests for cisco profiles

* update test for arista after fixing extends in profiles

* add changelog

* (turns out we cannot fix previous changelogs)

* fix mapping #15322

* rebase on master

* update test barracuda cloudgen

* address reviews

* address reviews

* revert to readynasos mib mapping

* remove tag

* Add assert_all_profile_metrics_and_tags_covered

* fix empty space

* Add assert_all_profile_metrics_and_tags_covered

* Add to test_profile__checkpoint_firewall_cpu_memory.py

* Add assertions

* better msg

* format

* not use f string

* not use f string

* Add ASSERT_ALL_PROFILE_EXCLUDED_TAGS

* fix cisco profiles tags & snmprec & test

* fix dell-rac snmprec (tag) & test

* fix _generic-entity-sensor snmprec (tags + mapping) & test

* fix alcatel-lucent-ent snmprec (tags + mapping) & test

* fix alcatel-lucent-ind (tags + mapping) & test

* rename anue-packet-broker snmprec to anue

* better test output

* fix aruba-clearpass tags & snmprec & test

* fix aruba-wc snmprec & test

* fix bluecat-server snmprec & test

* fix checkpoint snmprec & test

* fix eaton-epdu profile & test

* fix hp-h3c-switch snmprec & test

* fix huawei-routers snmprec & test

* fix infinera-coriant-groove snmprec & test

* fix juniper-variation test

* fix juniper-pulse-secure tags & snmprec & test

* fix nasuni-filer tags & snmprec & test

* fix meraki-cloud-controller device tag name

* rebase

* fix servertech-pdu4 snmprec & test

* fix sinetica-eagle-i snmprec & test

* fix western-digital snmprec & test

* rebase

* Update western-digital-mycloud-ex2-ultra.snmprec

fix type in snmprec for western-digital

* Update infinera-coriant-groove.snmprec

fix type in snmprec for infinera-coriant

* rebase

* update test_profiles.py

* update core tests

* let arista-switch test pass & fix it separately

* minor changes to profiles

* update _cisco-generic with the tooling

* update alcatel-lucent-ent with the tooling

* update alcatel-lucent-ind with the tooling

* update alcatel-lucent-omni-access-wlc with the tooling

* fix tests for cisco profiles

* update test for arista after fixing extends in profiles

* add changelog

* (turns out we cannot fix previous changelogs)

* fix mapping #15322

* address reviews

* address reviews

* fix nasuni-filer tags & snmprec & test

* rebase

* rebase

* let arista-switch test pass & fix it separately

* minor changes to profiles

* update test for arista after fixing extends in profiles

* fix cisco_nexus mapping & test for netgear

* delete jupiter-variation.yaml

* Update changelog

Co-authored-by: jhgilbert <jen.gilbert@datadoghq.com>

* fix netgear-readynas

* unexpected failure due to diagnoses metadata

* diagnoses metadata

* diagnoses metadata

---------

Co-authored-by: NouhaManai <nouha.manai@yahoo.fr>
Co-authored-by: lizaerem <elizaveta.eremina@datadoghq.com>
Co-authored-by: Elizaveta Eremina <70374721+lizaerem@users.noreply.github.com>
Co-authored-by: jhgilbert <jen.gilbert@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexandreYang AlexandreYang AlexandreYang approved these changes

Assignees
No one assigned
Labels
integration/snmp
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TCheruy @AlexandreYang