Update upgrading docs for DB IDO schema update

Use the entry_time for icinga_comments updates

This follows all other queries in their where condition
using entry_time for comments/downtimes.

DB IDO: Force users to upgrade the schema to avoid workaround fixes/missing indexes

Remove unnecessary keys from where clauses

ido: Drop unused instance_id indices from comments, downtimes and their related history tables

Fix commenthistory

refs #5492

Alter DowntimeRemoval behaviour

This adds an additional index to the downtimehistory and updates the
dbevents in a way that it uses that index

Merge pull request #5719 from Icinga/feature/windows-wizard-2-8

Update Windows Wizard for 2.8 and new signing methods

Update documentation for Windows Wizard and On-Demand signing support

refs #5681

Windows Wizard: Finalize design for 2.8

refs #5681

Windows wizard: Sanitize user inputs from text forms

This avoids white space problems with tickets and host names.

refs #5681
refs #5705

Windows Wizard: Missing ticket should tell the user about the master signing

refs #5681

Node setup: ticket parameter is now optional

refs #5681

Windows Wizard: Always enable the endpoint connection form

We always have to tick them anyways, as otherwise the setup wizard
won't work with CSR signing.

refs #5681

Cleanup Windows wizard and prepare for enhanced CSR signing

refs #5681

Merge pull request #5568 from kaprizki/41614-submitting-extended-plugin-output-via-process-check-result-api-call

Add documentation for multi-line plugin output for API actions

Add documentation for multi-line plugin output for API actions

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Merge pull request #5678 from Tuxdiver/Tuxdiver-patch-ntp_peer

Added missing "-q" parameter to check_ntp_peer

ITL: Add "-q" parameter to ntp_peer CheckCommand

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Add missing image

refs #4982

Add additional tests for flapping

refs #4982

Merge pull request #5710 from mcktr/feature/add-global-zones-during-wizard

Include default global zones during node wizard/setup

Include default global zones during node wizard/setup

This changes the GenerateNodeMasterIcingaConfig and GenerateNodeIcingaConfig
functions inside the nodeutillity class to the effect, that the default global
zones global-templates and director-global will be written to the generated
zones.conf that is created during the node wizard/setup.

refs #5707

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Merge pull request #5708 from RobinOBrien/fix/small-documentation-fix

Fixed grammar and spelling mistakes

Merge branch 'master' into fix/small-documentation-fix

Merge pull request #5709 from bascarsija/fix/flapping-documentation-typo-5703

fixes minor typos in flapping detection documentation

Fixed grammer and spelling mistakes

fixes minor typos in flapping detection documentation

refs #5703

Merge pull request #5693 from Icinga/fix/flapping-old-4982

Re-implement flapping

fixes #4982

Make the 'deprecated' keyword work for all field types

Fix flapping

Re-implement flapping following the 'old way' of just observing the last
20 stage changes.

refs #4982

Merge pull request #5620 from Icinga/fix/api-staging-3668

WIP: Ensure that the REST API config package/stage creation is atomic

Merge pull request #5696 from Icinga/fix/crash-with-premature-eof-on-resource-limited-os-5617

Fix fork error handling

Process: Fix fork error handling

refs #5617

Process: Remove log message from child process

Update mkdocs.yaml

refs #5450

Merge pull request #5695 from Icinga/feature/docs-distributed-security

Enhance Security chapter for Distributed Monitoring documentation

Enhance Security chapter for Distributed Monitoring documentation

https://monitoring-portal.org/index.php?thread/42175-newbie-zones/

Merge pull request #5682 from Icinga/feature/cert-migration

Implement support for migrating certificates to /var/lib/icinga2/certs

Implement support for migrating certificates to /var/lib/icinga2/certs

This commit includes documentation too.

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Merge pull request #5691 from mcktr/fix/doc-formatting

Fixed doc formatting

Fixed doc formatting

This fixes a minor doc formatting bug, where one closing bracket is outside the code tag.

Merge pull request #5638 from strajansebastian/patch-1

Add support for check_address as default in database CheckCommand objects

Add support for check_address as default in database CheckCommand objects

1. set the default hostname to the "$check_address$" for multiple CheckCommands
2. add support for  "$check_address$" for ping4 and ping6 commands which don't import the ipv4-or-ipv6 template

This commit changes the default address for redis, mongodb and elasticsearch to
use check_address instead of localhost. Users who might not have set this explicitly
in their services should adopt this.

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>
Signed-off-by: Gunnar Beutner <gunnar.beutner@icinga.com>

Fix some formatting issues in the ITL documentation

Signed-off-by: Gunnar Beutner <gunnar.beutner@icinga.com>

Merge pull request #5687 from Icinga/fix/jsonrpc-error-message-#5686

Improve error message for unknown cluster message functions

Merge pull request #5688 from RobinOBrien/fix/template_library_documentation

Fixed typos and punctuation

Fixed typos and punctuation errors in the icinga template library documenation file. Very minor changes.

Improve error message for unknow functions

refs #5686

Merge branch 'mcktr-mcktr/feature/4128-win-edit-button'

fixes #4128

Replace space indents with tabs

refs #5545

Merge branch 'mcktr/feature/4128-win-edit-button' of https://github.com/mcktr/icinga2 into mcktr-mcktr/feature/4128-win-edit-button

Merge pull request #5677 from mcktr/fix/doc-win-threshold-syntax

Fix typo in threshold syntax documentation

Fix typo in threshold syntax documentation

Merge pull request #5602 from Icinga/fix/config-validation-fails-on-windows-with-unprivileged-account-5515

Add windows process elevation and log message if user does not have privileges to read/write files

Merge pull request #5647 from konstantin-kelemen/fix/itl-ipmi-sensor-missing-argument-5579

Allow to disable thresholds for ipmi CheckCommand

Allow to disable thresholds for ipmi CheckCommand

fixes #5579

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Merge pull request #5231 from Al2Klimov/bugfix/failure-to-kill-check-command-after-exceeding-timeout-is-not-reported-4981

Report failure to kill check command after exceeding timeout

Merge pull request #5672 from Tontonitch/feature/itl-snmp-service

add itl snmp-service for manubulon plugin check_snmp_win.pl

add itl snmp-service

add itl snmp service for manubulon plugin check_snmp_win.pl

Merge pull request #5640 from miken32/patch-1

ITL: Support weathermap data in snmp_interface CheckCommand

ITL: Support weathermap data in snmp_interface CheckCommand

See https://github.com/dnsmichi/manubulon-snmp/pull/26 for related commit.

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Merge pull request #5581 from Icinga/fix/api-crash-race-condition

Fix possible race condition in ApiListener locking

Merge pull request #5637 from Icinga/fix/influxdb-unnecessary-string-casts

Fix unnecessary String() casts in InfluxdbWriter

Merge pull request #5668 from Icinga/feature/docs-enhance-monitoring-basics

Enhance Monitoring Basics in the documentation

Enhance Monitoring Basics in the documentation

This commit adds documentation for

* custom attributes value types and nested dictionaries
* better introduction into apply rules and usage examples
* better apply for iterations and explanations
* notifications with users on the host/service objects explained in a dedicated chapter
* command arguments and set_if - which values are supported
* macro() as scoped function
* alternative host check command

This also changes all code and config snippets to use
three backticks instead of four spaces indent.

fixes #5663
fixes #5664
fixes #5665
fixes #5666
fixes #5667
fixes #5186
fixes #5060
fixes #5662

Merge pull request #5655 from Icinga/feature/docs-attribute-value-types

Enhance documentation with more details on value types for object attributes

Enhance documentation with more details on value types for object attributes

Main focus: Object attributes and their value types. Originates from the
API Event Streams patch to document their value types. Now applied to
the entire configuration objects chapter.

The API chapter and their value types has been changed to capitalized
type identifiers.

Value types are now explicitly told about inside monitoring basics. This
has been adopted into the Icinga 2 training material a long time ago
and finds its way into the docs too.

The CheckResult and PerfdataValue value types have been moved into the
advanced topics chapter. mkdocs doesn't like multiple `#` headings in one file
and did just hide it.

Also moved the Influxdb details into the features documentation where it
belongs.

Many small references and URLs to other chapters added.
E.g. dedicated chapters for object accessor examples, to better link against
time dependent thresholds.

Can serve as patch basis and reference for ITL documentation where the
value types are missing too.

This commit also fixes the non-requirement of "legacy-timeperiod" which
also is a default template import since 2.6.

This also fixes the state mapping issue.

fixes #4015

Merge pull request #5554 from Icinga/feature/cn-check-for-san

Add subjectAltName extension for all non-CA certificates

Merge pull request #5636 from Icinga/fix/db-ido-comments-downtimes-constraint-on-legacy-id-change

Fix unique constraint matching for UPDATE downtime/comment runtime tables in DB IDO

Merge pull request #5635 from Icinga/fix/match-with-empty-array

Fix match(), regex(), cidr_match() behaviour with MatchAll and empty arrays

Merge pull request #5652 from Icinga/fix/docs-cluster-zone

Docs: Fix missing default value for cluster-zone checks

Merge pull request #5653 from Icinga/fix/docs-manubulon-nocrypt-default-value

Docs: Fix default value for `snmp_nocrypt` for Manubulon CheckCommand definitions

Docs: Fix default value for `snmp_nocrypt` for Manubulon CheckCommand definitions

Docs: Fix missing default value for cluster-zone checks

Merge pull request #5643 from Icinga/fix/buildfix-apple-clang-9.0.0

Fix debug builds on Apple Clang 9.0.0 (macOS High Sierra)

Fix debug builds on Apple Clang 9.0.0 (macOS High Sierra)

Merge pull request #5624 from mcktr/mcktr/fix/missing-include-statements

Fixed missing include statement in unit test

Merge pull request #5641 from Icinga/fix/compiler-warnings-high-sierra

Fix compiler warnings on macOS 10.13

Fix compiler warnings on macOS 10.13

Merge pull request #5612 from Icinga/bugfix/systemd-tasksmax

Improve documentation and systemd config on TaskMax

Improve documentation and Systemd config for TasksMax setting

This also disables setting a value by default.

Packaging should decide to install the config file:
`/etc/systemd/system/icinga2.service.d`

fixes #5611

Signed-off-by: Michael Friedrich <michael.friedrich@icinga.com>

Fix unnecessary String() casts in InfluxdbWriter

fixes #5629

Fix unique constraint matching for UPDATE downtime/comment runtime tables in DB IDO

fixes #5623
fixes #5603

Fix match(), regex(), cidr_match() behaviour with MatchAll and empty arrays

fixes #5634

Merge pull request #5632 from Icinga/feature/docs-getting-started-selinux

Docs: Mention SELinux in Getting Started chapter

Docs: Mention SELinux in Getting Started chapter

refs icinga/icingaweb2#2973

Merge pull request #5258 from Icinga/fix/clang-pthread

Fix clang compiler detection on Fedora and macOS

changelog.py: Fix debugging leftover

refs #5600

Fix clang compiler detection on Fedora and macOS

fixes #5257

Fixed missing include statement in unit test

This adds the missing iostream include to the notification unit test.

refs #5613

Merge pull request #5619 from Icinga/fix/changelog-project-name-error

Exit early in changelog.py if GitHub API fetch fails

fixes #5600

Merge pull request #5578 from mcktr/mcktr/fix/5577-doc-ssl-check-command

ITL: Re-Add ssl_sni attribute for check_tcp

Exit early in changelog.py if GitHub API fetch fails

fixes #5600

Merge pull request #5616 from Icinga/fix/build-warning

Fix a build warning

Fix a build warning

refs #5443

Merge pull request #5614 from mcktr/mcktr/fix/5613-missing-include-statement

Fixed missing include statement in unit tests

fixes #5613

Fixed missing include statement in unit tests

refs #5613

ITL: Re-Add ssl_sni attribute for check_tcp

Merge pull request #5608 from Icinga/fix/certificate-directories

Fix certificate paths for installers