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 docs #156

Merged
merged 6 commits into from
Feb 6, 2024
Merged

Update docs #156

merged 6 commits into from
Feb 6, 2024

Conversation

nook24
Copy link
Member

@nook24 nook24 commented Feb 5, 2024

This PR contains:

  • Fix the docs for CHANGE_CUSTOM_*_VAR external commands
  • Update *PROBLEMID and *NOTIFICATIONID macros to UUIDv4
  • Add new PROBLEM time macros

All example are shown with a leading underscore like "_SOMEVAR"
which does not work.

To change the value of the variable, the name does not have an underscore as prefix.

Signed-off-by: nook24 <[email protected]>
… now UUIDv4 strings instead of integer counter values

naemon/naemon-core#447

Signed-off-by: nook24 <[email protected]>
Copy link
Contributor

@sni sni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

found a type. Besides that it's fine. Thanks

</tr>
<tr>
<td class="MacroName"><a name="hostproblemdurationsec">$HOSTPROBLEMDURATIONSEC$</a></td>
<td class="MacroDescription">The duration in seconds bettwen $HOSTPROBLEMSTART$ and $HOSTPROBLEMEND$.</td>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

between

@nook24
Copy link
Member Author

nook24 commented Feb 6, 2024

  • Fixed between typo
  • Added Dockerfile + Readme to build and run locally using Docker

@sni Could you please check the macro availability of the new *PROBLEM macros? At the moment they are marked with yes in all cases.

@sni
Copy link
Contributor

sni commented Feb 6, 2024

that's right, they are available all time. Although they probably only make sense for eventhandler or notifictions.

@nook24 nook24 merged commit 51ff6c2 into naemon:master Feb 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants