Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

add centreontrapd @HOSTID@ macro #7592

Merged
merged 1 commit into from
Jul 4, 2019
Merged

add centreontrapd @HOSTID@ macro #7592

merged 1 commit into from
Jul 4, 2019

Conversation

garnier-quentin
Copy link
Contributor

Pull Request Template

Description

Can @hostid@ in special commands (could be useful for centreon-dsm).

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 2.8.x
  • 18.10.x
  • 19.04.x
  • 19.10.x (master)

How this pull request can be tested ?

Use @hostid@ in special trap command definition. The variable should be subsitute by the host_id.

Checklist

Community contributors & Centreon team

  • I followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

Centreon team only

  • I have made sure that the unit tests related to the story are successful.
  • I have made sure that unit tests cover 80% of the code written for the story.
  • I have made sure that acceptance tests related to the story are successful (local and CI)

@lpinsivy
Copy link
Contributor

lpinsivy commented Jun 21, 2019

Good be good to add in documentation a description about which macros can be replaced:

@HOSTNAME@
@HOSTID@
@HOSTADDRESS@
@HOSTADDRESS2@
@SERVICEDESC@
@SERVICENOTES@
@TRAPOUTPUT@
@OUTPUT@
@STATUS@
@SEVERITYNAME@
@SEVERITYLEVEL@
@TIME@
@POLLERID@
@POLLERADDRESS@
@CMDFILE@
@CMDDIR@
@USERARG1@
@USERARG2@
@DAY@ => %Y%m%d

@lpinsivy lpinsivy added the status/in-backlog A dev will begin to work on your modifications soon label Jun 21, 2019
@lpinsivy lpinsivy merged commit 4633578 into centreon:master Jul 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/monitoring kind/enhancement status/in-backlog A dev will begin to work on your modifications soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants