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

Fix certificate paths for installers

refs #5450

Merge pull request #5606 from Icinga/feature/remove-bottom-up

Remove bottom-up client mode

Drop satellite.conf reference for bottom-up

refs #4799

Drop repository.d from configuration chapter in the docs

refs #4799

Drop repository from Windows installer

refs #4799

Remove reference to bottom-up in troubleshooting docs

refs #4799

Update CLI commands documentation

refs #4799

Drop unused code from NodeUtility class

refs #4799

Add upgrading docs for removed bottom-up client mode

refs #4799

Drop unused bottom-up images from the docs

refs #4799

Remove bottom-up repository.d references

refs #4799

Remove bottom-up from docs and add hints for Director, Puppet, etc.

refs #4799

Remove bottom-up cluster messages

refs #4799

Remove bottom-up CLI commands

refs #4799

Docs: Clarify on configuration and packages, thanks @dgoetz

refs #5604
refs #5301

Merge pull request #5604 from Icinga/feature/remove-deprecated-classicui-pkg

Remove the icinga2-classicui-package and update documentation

Docs: Add a note for upgrading with removed classicui-config package

refs #5301

Remove Classic UI references from docs

refs #5301

Remove deprecated Classic UI configuration package

refs #5301

Add process elevation & log message if user does not have enough privileges

refs #5515

Merge pull request #5601 from Icinga/fix/empty-cache-dir-on-pkg-upgrade

Ensure that the cache directory always is set and add a note to upgrading docs

Ensure that the cache directory always is set and add a note to upgrading docs

Thanks @dgoetz

Merge branch 'support/2.7'

Release version 2.7.1

Release version 2.7.1

changelog.py: Add "backported" to the list of ignored labels

refs #5599

Merge pull request #5599 from Icinga/fix/changelog-labels

changelog.py: Add "backported" to the list of ignored labels

changelog.py: Add "backported" to the list of ignored labels

Merge pull request #5587 from Icinga/feature/custom-syslog-facility-option-on-sysloglogger-object-3964

SyslogLogger: Implement option to set syslog facility

Docs: Enhance certificate and configuration troubleshooting chapter

refs #5594

Merge pull request #5594 from Icinga/feature/docs-troubleshooting-certificates

Docs: Enhance certificate and configuration troubleshooting chapter

Docs: Enhance certificate and configuration troubleshooting chapter

Docs: Add a note for upgrading to 2.7

refs #5593

Merge pull request #5593 from Icinga/feature/docs-upgrading

Docs: Add a note for upgrading to 2.7

Docs: Add a note for upgrading to 2.7

Merge pull request #5547 from ryanohnemus/feature/allow_package_upload_without_restart

Add optional reload parameter to config stage upload

API: Add optional reload parameter to config stage upload

You can now specify a boolean `reload` attribute that
will allow you to skip the icinga2 reload after config
validation. By default this is set to true.

The response text has been updated to show if icinga2
will reload or if it was requested to be skipped.

fixes #4769

SyslogLogger: Implement option to set syslog facility

fixes #3964