Icinga / Icinga/icinga2

ITL: Update of ipmi-sensor arguments

Open
#10,172 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area/itl
Dominant language
C++
Stars
2.2k
Forks
616
Avg merge
5d 6h
Merged PRs (30d)
10

Description

Describe the bug

There were some arguments added to check_ipmi_sensors in version 3.14 and earlier, which are missing in the ITL definition. It would be great to get an update in the ITL.

The following arguments may be added to ITL (I stumbled about this issue missing the first two):

  • -O
  • --seloptions
  • --selonly
  • --seltail
  • -sx
  • -xx
  • --nosudo

Expected behavior

ITL gets an update when checks get an update with new arguments.

Your Environment

Include as many relevant details about the environment you experienced the problem in

  • Version used (icinga2 --version): 2.14.2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the ITL definition for check_ipmi_sensors and compare its accepted arguments with the list in this issue. Update the definition to include the listed arguments, then verify that the ITL exposes them consistently with the check; no specific file or test is named in the issue.

Written by the indexing model from the issue text.

Assessment

Domain
observability-sre
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.