Docs: Update debugging instructions for 2.11

Merge pull request #7480 from Icinga/feature/docs-windows-dev

Dev Docs: Update Windows to Visual Studio 2019

Docs: Update Windows for VS 2019, add CI details

Merge pull request #7491 from Icinga/bugfix/boost-asio-coroutine-exception-wrapper

Introduce IoEngine::SpawnCoroutine wrapping asio::spawn and Boost exceptions

Dev: Fix CMake generator platform for VS 2019 in PS scripts

Dev Docs: Update Windows to Visual Studio 2019

Introduce IoEngine::SpawnCoroutine wrapping asio::spawn and Boost exceptions

This is required to

- catch all exceptions and wrap them into Boost exceptions. They
are the only ones allowed with Boost.Coroutine.
- set a dedicated coroutine stack size for Windows.

refs #7431

Merge pull request #7490 from Icinga/bugfix/asio-deprecated-io-service

Quality: Replace deprecated Boost IO service code

Quality: Replace deprecated Boost IO service code

https://github.com/boostorg/asio/issues/110
https://www.boost.org/doc/libs/1_66_0/doc/html/boost_asio/example/cpp03/services/logger_service.hpp

Merge pull request #7487 from Icinga/bugfix/http-api-shutdown-avoid-io

API: Avoid I/O on shutdown

HttpServerConnection#ProcessMessages(): avoid I/O if shutting down

refs #7431

HttpServerConnection#ProcessMessages(): avoid I/O after boost::asio::error::operation_aborted

refs #7431

{HttpServer,JsonRpc}Connection#Disconnect(): cancel I/O ops ASAP

refs #7431

Merge pull request #7485 from Icinga/bugfix/api-disconnect-defer

Avoid the Defer-Disconnect destructor pattern with Boost.Coroutines

Merge pull request #7486 from Icinga/bugfix/http-header-error-handling

Rewrite error handling in HttpServerConnection#EnsureValidHeaders()

Rewrite error handling in HttpServerConnection#EnsureValidHeaders()

Throwing local exceptions unnecessarily pollutes the exception
stack with immediate unwinding. Avoid this pattern at all cost within
Boost.Coroutines. MSVC may handle exceptions differently and cause
problems with stack unwinding.

refs #7431
refs #7351

Avoid the Defer-Disconnect destructor pattern with Boost.Coroutines

Exceptions in Disconnect() might be thrown (this has been reworked
into error_code locally) which are swallowed inside the Destructor
for being dangerous. On the other hand, swallowing them may
corrupt the stack unwinding operation from the coroutine layer.

The best is to avoid Defer inside lib/remote and call Disconnect()
directly after breaking from other operations.

refs #7351
refs #7431

Merge pull request #7483 from Icinga/bugfix/jsonrpc-boost-asio-coroutine-exceptions

JsonRpcConnection: Don't swallow exceptions in Boost.Coroutine

Merge pull request #7481 from Icinga/feature/check-network-perfdata

Add summarized performance data to check_network

JsonRpcConnection: Don't swallow exceptions in Boost.Coroutine

refs #7351

Add summarized performance data to check_network

This adds two new performance data values to check_network, 'network_in'
and 'network_out'.

On systems with multiple network interfaces the 'network_in' value holds
the summarized input bytes and the 'network_out' value holds the
summarized output bytes.

Merge pull request #7479 from Icinga/feature/windows-dev

Build: Add Powershell script for development config for building Windows debug packages

Build: Add Powershell script for development config for building Windows debug packages

Merge pull request #7477 from Icinga/bugfix/api-boost-exceptions-errors

HttpServerConnection: Prefer error codes over Boost exceptions

HttpServerConnection: Prefer error codes over Boost exceptions

When run within a coroutine, exceptions on Windows may influence
bad behaviour here. Instead, we'll check for the error code
and extract the message from memory. In contrast to exceptions
which are stored on the stack frame and then return, this costs
a little more memory but simplifies the logic.

This doesn't fix the linked issue, but is related to the analysis.

refs #7431

Merge pull request #7476 from Icinga/bugfix/api-beast-buffer

Always reset Boost beast buffer in HttpServerConnection#ProcessMessages

Always reset Boost beast buffer in HttpServerConnection#ProcessMessages

refs #7431

Adjust code comment for Logger->Flush() on Windows

Ensure that issue templates are more readable

Merge pull request #7467 from Icinga/feature/docs-development-2-11

Docs: Update Linux and Windows Dev instructions for 2.11

Docs: Update dev chapter for Debian Buster

Docs: Update Boost setup instructions for Windows dev environment

Merge pull request #7464 from Icinga/feature/docs-windows-agent-versions

Docs: Update supported Windows agent versions

Docs: Sync Vagrant box dev instructions with development docs

Merge pull request #7463 from Icinga/feature/docs-powershell

Docs: Prefer the Powershell commands on Windows

Docs: Update supported Windows agent versions

Docs: Add Windows debugging

Docs: Prefer the Powershell commands on Windows

Merge pull request #7459 from Icinga/feature/expose-problem-handled-counts

Metrics: Expose problem/handled counts for hosts/services

Metrics: Expose problem/handled counts for hosts/services

With the addition of problem/handled as checkable runtime
attributes in #7096 we should also expose them via CIB
into

- icinga check
- /v1/status/CIB
- IcingaApplication runtime macros

This originates from a request by @phil-or at the recent
Icinga meetup in Linz.

Merge pull request #7356 from Icinga/bugfix/influxdbwriter-clone

InfluxdbWriter: clone less

Merge pull request #7451 from Icinga/bugfix/update-docs

Update docs: Fix online URL, cipher list

Merge pull request #7445 from Icinga/bugfix/ddos-reconnect-7444

JsonRpcConnection#Disconnect(): unregister the connection ASAP

Update default cipher_list in upgrading docs

Update doc URL for the API info handler at /v1

Update doc URL references for the DB IDO feature

Merge pull request #7450 from Icinga/feature/windows-exe-icon

Windows binary: Also update the exe icon

Windows binary: Also update the exe icon

A thousand places for everything.

Merge pull request #7449 from Icinga/feature/warn-set-global-var

DSL: warn on x=y if x is a global variable

Merge pull request #7346 from Icinga/feature/check-plugins-code-page-7110

DaemonCommand#Run(): set code page to UTF-8

Merge pull request #7447 from Icinga/bugfix/nscp-json

check_nscp_api: read HTTP response via Boost.Beast

ITL: set globals.NscpPath explicitly

DSL: warn on x=y if x is a global variable

Merge pull request #7434 from Icinga/bugfix/cmake-aix

CMake: Fix CXX flags for AIX

Merge pull request #7435 from Icinga/bugfix/cmake-lto

CMake: Fix LTO flags for shared linking

Merge pull request #7437 from m4k5ym/check_uptime

Add check_uptime to ITL

check_nscp_api: fix missing HTTP response reason

check_nscp_api: read HTTP response via Boost.Beast

Adding check_uptime to itl

JsonRpcConnection#Disconnect(): unregister the connection ASAP

refs #7444

CMake: Fix LTO flags for shared linking

CMake: Fix CXX flags for AIX

Merge pull request #7428 from Icinga/feature/itl-sar-perf

ITL: Add sar-perf

ITL: Add sar-perf

Merge pull request #7422 from Icinga/bugfix/current_concurrent_checks-7416

Correct current_concurrent_checks to actually running checks

Correct current_concurrent_checks to actually running checks

refs #7416

Merge pull request #7421 from Icinga/feature/threadpool-metric

Expose metric current_pending_callbacks

Merge pull request #7423 from Icinga/bugfix/cluster-config-object-create-ignore-on-error

Fix and improve logging for runtime object sync

Fix and improve logging for runtime object sync

config::UpdateObject would create a new object, but this may
have been silently ignored with 'ignore_on_error' - downtimes, etc.
Since we cannot simply fetch the error from inside the config compiler,
we'd just check whether there's a config object created at this stage.
This happens synchronously, and once there is, log something.

The previous code always logged the creation, even if the downtime
was ignored, e.g. when the first master sent one for local host objects.

This commit also adds more details: identity, endpoint, zone to extract
the MessageOrigin details into log messages for better troubleshooting
and debugging.

refs #7198

Expose metric current_pending_callbacks

Introduce ThreadPool#GetPending()

Merge pull request #7420 from Icinga/feature/docs-tech-cluster-routing

Docs: Add cluster message routing details

Docs: Add cluster message routing details

refs #7198

Merge pull request #7419 from Icinga/bugfix/downtime-loop-activate-origin

Fix object create/delete config object cluster loop (missing message origin)

Merge pull request #7418 from Icinga/bugfix/icinga-check-version-compare

Fix minimum version parsing in the 'icinga' check

Properly parse the Icinga version for minimum version in the 'icinga' check

fixes #7415

Add unit test for Utility::CompareVersion

Add Utility::ParseVersion() and unit tests

This now uses a regex to extract the short version
similar to how Icinga Web 2 does it.

Additional unit tests prove the rule.

Fix cookie with ActivateItems

Fix message origin for runtime created config object (create/delete events)

Docs: Fix URL in Windows setup wizard

Merge pull request #7409 from Icinga/bugfix/cmake-target-arch

CMake: If target arch cannot be detected, don't throw an error

CMake: If target arch cannot be detected, don't throw an error

This only happens when ccache is improperly in use and -dumpmachine
doesn't work. Not on our ARM build platform.

Merge pull request #7406 from dgoetz/bugfix/doc_itl_icinga

Docs: Add a note to icinga check about failed reload detection

Docs: Add a note to icinga check about failed reload detection

Merge pull request #7404 from Icinga/bugfix/livestatus-interval

Explicitly write the 1.x legacy attribute 'interval_length' for Livestatus

Merge pull request #7353 from Icinga/bugfix/influxdbwriter-skip-tags-7341

InfluxdbWriter: skip non-resolvable tags

Merge pull request #7403 from Icinga/feature/docs-agents

Enhance Agent best practices throughout the documentation

Enhance Agent best practices throughout the documentation

- Highlight the Icinga agent
- Prefer SSH as fallback and add more detailed setup instructions
- Improve SNMP and add traps right after
- Explain nscp and wmi with URLs
- Drop discouraged nrpe instructions (checks and PNP customizations)
- Update Dependency examples with the Icinga Agent

This sources from recent discussions on community.icinga.com
and follows the updates with the distributed monitoring chapter for 2.11.

Explicitly write the 1.x legacy attribute 'interval_length' for Livestatus

fixes #7402

Merge pull request #7397 from Icinga/bugfix/umbrella-logging-systemd-7394

icinga2 daemon --close-stdio: keep console log open during first config validation

Log a hint after failed config validation for systemd users

Merge pull request #7400 from Icinga/bugfix/cib-stats-concurrent-checks

Fix missing stats for current_concurrent_checks via REST API

Merge pull request #7399 from Icinga/feature/log-downtimes-acks

Improve logging for downtime/ack events (add, remove, expire)

Fix missing stats for current_concurrent_checks via REST API

It was added inside the 'icinga' check task, but not for the REST API.
Thanks for asking, @Thomas-Gelf

Improve logging for downtime/ack events (add, remove, expire)

fixes #7374

icinga2 daemon --close-stdio: keep console log open during first config validation

refs #7394

Merge pull request #7391 from Icinga/feature/cluster-sync-avoid-non-utf-8

Cluster sync: Only sync valid UTF8 content (text config, no binaries)

Merge pull request #7396 from Icinga/bugfix/ido-drop-index-if-exists

DB IDO: Fix upgrade script for 2.11.0 (drop index only if existing)

DB IDO: Add upgrading note