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

check_mk_agent for freebsd : optional use freeipmi if ipmitool is not… #655

Closed
wants to merge 0 commits into from

Conversation

zerwes
Copy link

@zerwes zerwes commented Dec 15, 2023

With the latest updates of opnsense based on FreebSD 13.2, the ipmitool package is not available anymore.

This patch enables the check_mk_agent for freebsd to use the ipmi-sensors command from the freeipmi package.

The patch is hardly derived from the ipmi_sensors section of the linux agent.

Currently the ipmi-sensors command will only be run if ipmitoolis not available, but if desired, I can change this, so both sections can be used in parallel.

Copy link

github-actions bot commented Dec 15, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@zerwes
Copy link
Author

zerwes commented Dec 15, 2023

I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA.

@TimotheusBachinger
Copy link
Contributor

Dear Checkmk Contributor! Unfortunately, we had to re-write our git-repo history, rendering your PR auto-closed. We will therefore rebase your PR onto the current master and reopen it again. Sorry for the inconvenience.

@TimotheusBachinger
Copy link
Contributor

Dear Contributor. Unfortunately, we learned that re-opening a PR which was force-rebased, is not possible (see isaacs/github#361). Therefore we kindly ask you to create a new PR for your change. We apologize for the circumstances and will implement technical measures to prevent such incidents in the future.

@zerwes zerwes mentioned this pull request Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants