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.

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

Update mib_source_url to a Datadog fork of mibs.snmplabs.com #9952

Merged
merged 3 commits into from
Sep 9, 2021

Conversation

ian28223
Copy link
Contributor

@ian28223 ian28223 commented Aug 23, 2021

What does this PR do?

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

@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #9952 (b8fe7d4) into master (3b4eba3) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

Flag Coverage Δ
datadog_checks_dev 78.73% <100.00%> (-0.69%) ⬇️

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

@ian28223 ian28223 changed the title Update mib_source_url to our own fork Update mib_source_url to our own fork of mibs.snmplabs.com Aug 23, 2021
AlexandreYang
AlexandreYang previously approved these changes Aug 24, 2021
gmarabout
gmarabout previously approved these changes Sep 8, 2021
@pducolin
Copy link
Contributor

pducolin commented Sep 8, 2021

Fixed conflict and pushed, thank you for creating a public repo with all the mibs ❤️ ❤️ ❤️

pducolin
pducolin previously approved these changes Sep 8, 2021
AlexandreYang
AlexandreYang previously approved these changes Sep 9, 2021
@AlexandreYang AlexandreYang dismissed stale reviews from pducolin and themself via b8fe7d4 September 9, 2021 06:32
@pducolin pducolin merged commit 24b4bcc into master Sep 9, 2021
@pducolin pducolin deleted the ian.bucad/update_mib_source branch September 9, 2021 08:53
github-actions bot pushed a commit that referenced this pull request Sep 9, 2021
* Update mib_source_url
https://github.com/projx/snmp-mibs has been disabled

* code review

* Fix imports order

Co-authored-by: Alexandre Yang <[email protected]> 24b4bcc
@ChristineTChen ChristineTChen changed the title Update mib_source_url to our own fork of mibs.snmplabs.com Update mib_source_url to a Datadog fork of mibs.snmplabs.com Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants