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

monitoring: remove old MIB #44796

Merged
merged 1 commit into from Feb 4, 2022
Merged

monitoring: remove old MIB #44796

merged 1 commit into from Feb 4, 2022

Conversation

pcuzner
Copy link
Contributor

@pcuzner pcuzner commented Jan 26, 2022 •
edited by epuertat

The MIB file that matches the OID definitions in the alerts is
CEPH-MIB.txt. The old MIB from the original SuSE snmp
gateway work, therefore needs to be removed to avoid
confusion.

Fixes: tracker.ceph.com/issues/54031
Signed-off-by: Paul Cuzner pcuzner@redhat.com

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@pcuzner
monitoring: remove old MIB
cbeab5c 
The MIB file that matches the OID definitions in the alerts is
CEPH-MIB.txt. The old MIB from the original SuSE snmp
gateway work, therefore needs to be removed to avoid
confusion.

Signed-off-by: Paul Cuzner <pcuzner@redhat.com>
@pcuzner pcuzner requested review from a team, epuertat and Sarthak0702 and removed request for a team January 26, 2022 22:46
@nizamial09 nizamial09 added this to In progress in Dashboard via automation Jan 27, 2022
Dashboard automation moved this from In progress to Reviewer approved Jan 27, 2022
@nizamial09 nizamial09 mentioned this pull request Jan 28, 2022
Closed
14 tasks
@epuertat epuertat moved this from Reviewer approved to Ready-to-merge in Dashboard Jan 31, 2022
@epuertat epuertat merged commit 6a4b1e1 into ceph:master Feb 4, 2022
Dashboard automation moved this from Ready-to-merge to Done Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alfonsomthd alfonsomthd alfonsomthd approved these changes

@avanthakkar avanthakkar avanthakkar approved these changes

@pereman2 pereman2 pereman2 approved these changes

@nizamial09 nizamial09 nizamial09 approved these changes

@epuertat epuertat Awaiting requested review from epuertat epuertat was automatically assigned from ceph/dashboard

@Sarthak0702 Sarthak0702 Awaiting requested review from Sarthak0702 Sarthak0702 was automatically assigned from ceph/dashboard

@aaSharma14 aaSharma14 Awaiting requested review from aaSharma14

Assignees
No one assigned
Labels
dashboard monitoring
Projects
Ceph-Dashboard
Archived in project
Dashboard
  
Done
Milestone
No milestone
6 participants
@pcuzner @alfonsomthd @avanthakkar @pereman2 @nizamial09 @epuertat