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

remove python.d/megacli #17417

Merged
merged 1 commit into from Apr 16, 2024
Merged

remove python.d/megacli #17417

merged 1 commit into from Apr 16, 2024

Conversation

ilyam8
Copy link
Member

@ilyam8 ilyam8 commented Apr 16, 2024

Summary
  • megacli collector was rewritten in go (rewrite megacli in go #17410)
  • python version is very problematic (doesn't work by default, requires reset CapabilityBoundingSet, adding "netdata" to sudoers, produces charts without labels with instances as dimensions, etc).
Test Plan

ci

Additional Information
For users: How does this change affect me?

@github-actions github-actions bot added area/docs area/health area/collectors Everything related to data collection area/build Build system (autotools and cmake). collectors/python.d area/metadata Integrations metadata labels Apr 16, 2024
@ilyam8 ilyam8 merged commit 6c72353 into netdata:master Apr 16, 2024
151 of 152 checks passed
@ilyam8 ilyam8 deleted the pyhton.d-remove-megacli branch April 16, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vkalintiris vkalintiris Awaiting requested review from vkalintiris vkalintiris is a code owner

@Ancairon Ancairon Awaiting requested review from Ancairon Ancairon is a code owner

@thiagoftsm thiagoftsm Awaiting requested review from thiagoftsm thiagoftsm is a code owner

Assignees
No one assigned
Labels
area/build Build system (autotools and cmake). area/collectors Everything related to data collection area/docs area/health area/metadata Integrations metadata collectors/python.d
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@ilyam8