09-object-types: notifications: states filter ignored for Acknowledgements!

Merge pull request #6591 from Icinga/bugfix/lto-builds-static-initialize-namespaces

Fix static initializer priority for namespaces in LTO builds

Merge pull request #6590 from Icinga/fix/custom-var-workaround

Update workaround for custom vars

Fix static initializer priority for namespaces in LTO builds

fixes #6575

Merge pull request #6356 from sourcejedi/fix/systemd

Fix logging under systemd

Merge pull request #6568 from Icinga/feature/commit-order

Ensure that config object types are committed in dependent load order

Update workaround for custom vars

This updates #6572 which did not work

Update variable names

Ensure that config object types are committed in dependent load order

Merge pull request #6581 from Icinga/fix/speedup-config

Shuffle items before config validation

Shuffle items before config validation

Merge pull request #6588 from Icinga/bugfix/prepare-dirs-safe-reload-full-path

Fix using full path in prepare-dirs/safe-reload scripts

Fix using full path in prepare-dirs/safe-reload scripts

This won't be visible with packages and overridden paths,
just source builds which invoke the scripts manually.

refs #6506

Merge pull request #6583 from maxswjeon/master

Update PostgreSQL library path variable in INSTALL.md

Merge pull request #6586 from Icinga/bugfix/centos-7-buildfix-non-unity-builds

Fix non-unity builds on CentOS 7 with std::shared_ptr

Fix non-unity builds on CentOS 7 with std::shared_ptr

refs #6509

Merge pull request #6574 from gunnarbeutner/fix/move-downtime-constants

Move new downtime constants into the Icinga namespace

Edited INSTALL.md File

Edited the Mistake of the PostgreSQL_LIBRARY_DIR comment error.
#6582

Move new downtime constants into the Icinga namespace

Fix logging under systemd

icinga2.service used `-e ${ICINGA2_ERROR_LOG}`, but this is documented
as having no effect without `-d`.  Furthermore, icinga2 under systemd
unconditionally logged everything to the system log (but without setting
the log level etc), which contradicted the documentation.  (Issue #6339)

Stop icinga2 on systemd from logging to stdout - and hence the system log -
once it has finished starting up.  Just like when you start icinga2 from a
terminal using `-d`.  And just like -d, we stop logging fatal errors to
stderr, and instead write to the log file passed with `-e`.

As per docs, mainlog (icinga2.log) is already enabled by default.  And
pre-startup messages including config errors will still appear in the
system log.

This uses a new option --close-stdio, which has the same effect on logging as
--daemonize, but does not fork or call setsid().

For this purpose, I moved setsid() up and into Daemonize().

Consequence of that last point: if anyone is weird enough to specify a TTY
device file as the fatal error log (-e option), that will become icinga's
controlling terminal, which you generally don't want as a daemon.  This
makes it consistent with the existing behaviour for icinga mainlog.  For
this reason you're supposed to use O_NOCTTY in Linux daemons.  But I wasn't
sure where icinga would want to put the ugly `#ifdef _WIN32 ... #else ...`.

Daemonize(): use one error convention, not three

Standardize on exit() / _exit() (this depends whether we are considered
to be the "main" fork, which should run anything registered with atexit()).
Exclude `return false` and throwing exceptions.

This fixes the error path for fork().  Daemonize() would return false, but
the `return false` error convention was not tested in the caller.

It also fixes the error message for fork() to show the error code.
Everyone loves `strace`, but sysadmins should not have to rerun their
daemons under it just to see an error code.

Also in case an exception is thrown, show its diagnostic information
instead of dropping it on the floor.  In the log message, I mention why we
are uninitializing and then initializing the app at this point.

For the reader, it pushes all the weirdness into the error convention of
Daemonize().  This comes back to the exit() / _exit() distinction.  Once
we have forked, we technically don't want to allow the parent process to
exit(), so we don't want to return to the caller.

fix "Console" log to flush

It's called "Console", which would be line-buffered anyway.  But, it's
implemented as std::cout.  This might be piped to a logger, as in
daemontools or systemd.  In this case it will not be a TTY, and log lines
should be flushed without too much delay.  Let's just flush each message.

Let's not introduce a static instance of StreamLogger (flushed by interval
timer).  That's too stressful to read, because static instances are really
annoying to order.  Example citation: "Yay, our static destructors are
pretty much beyond repair at this point." -- application.cpp.

I don't know if there will be any need to optimize logging syscalls.  The
init script uses `--daemonize`.  I think the systemd service should also
avoid using the "Console" log after startup (see next commit).  The
documentation does not warn that the syslog feature is less efficient
in system calls than mainlog; deferred flusing does not seem to be a highly
prominent feature.  There's no cool comment in the code about how much the
syscalls were slowing down some use case (or qualifying that this
optimization can only eliminate syscalls on platforms with both mutexes and
clocks that can work without syscalls).

Merge pull request #6532 from Icinga/feature/allow-down-times-to-apply-on-child-hosts-3935

Add child_options to ScheduledDowntime

Add upgrading docs for API schedule-downtime 'child_options'

Merge pull request #6502 from jenslink/patch-2

Update 17-language-reference.md

Merge pull request #6501 from jenslink/patch-1

Update 03-monitoring-basics.md

Merge pull request #6510 from Icinga/feature/windows-build-scripts

Update Appveyor build scripts

Add child_options to ScheduledDowntime

refs #3935

Merge pull request #6509 from gunnarbeutner/feature/real-constants

Implement support for namespaces

Update upgrading docs for v2.10 and namespaces

Merge pull request #6572 from Icinga/bugfix/29X-custom-vars-upgrade

Add note about workaround for broken custom vars

Add note about workaround for broken custom vars

Merge pull request #6570 from Icinga/bugfix/tls-anonymous-clients-limit

Increase limit for simultaneously connected anonymous TLS clients

Increase limit for simultaneously connected anonymous TLS clients

Merge pull request #6567 from Icinga/bugfix/env-api-port-number

ApiListener: Dump the state file port detail as number

Merge pull request #6556 from Icinga/feature/installer-msi-suppress

windows: Allow suppression of extra actions in the MSI package

ApiListener: Dump the state file port detail as number

refs #6511

Merge pull request #6561 from miso231/feature_itl_ceph

[Feature] Ceph health CheckCommand

Merge pull request #6469 from Icinga/fix/windows-resize-behavior

Fix Windows Agent resize behavior

Merge pull request #6563 from miso231/feature/itl-cloudera-check

[Feature] Cloudera service health CheckCommand

Merge pull request #6544 from gunnarbeutner/fix/deprecated-strstream-header

Remove #include for deprecated header file

Add cloudera plugin to itl

Fix value attributes for ceph plugin

Fix table format for ceph check doc

Add ceph plugin to itl

windows: Allow suppression of extra actions in the MSI package

By setting SUPPRESS_XTRA

Update documentation

Refactor Application::*Const()

Remove #include for deprecated header file

Implement namespace support for the keys() function

Implement support for the namespace and using keywords

Move constants and functions into different namespaces

Implement support for namespaces

Implement support for modifying frozen attributes

Refactor REGISTER_SCRIPTFUNCTION_* macros

Merge pull request #6512 from Icinga/feature/sni-environment

Refactor environment for API connections

Re-add environment to IcingaApplication feature stats

Update documentation and add upgrading notes

Introduce IcingaApplication#environment

Precedence as follows:

- DEnvironment=...
- const Environment = ...
- object IcingaApplication "app" { environment = "..." }

The wrapped script constant handling is required
since we cannot directly link from libremote (SNI handling)
to libicinga where the object resides. Instead we'll
use the Application class helpers for hiding the ScriptGlobal
calls.

Mkclass: Only render setter if not declared pure virtual

Merge pull request #6539 from Icinga/bugfix/centos-7-non-unity-build-fix

Build fix for CentOS 7 and non-unity builds

Build fix for CentOS 7 and non-unity builds

Rename to Environment constant

Update doc anchors

Refactor environment for API connections

* Const renamed to `ApiEnvironment`
* Handling moved to ApiListener
* Now a property of ApiListener

Merge pull request #6531 from Icinga/feature/zone-all_parents

Expose Zone#all_parents via API

Merge pull request #6511 from Icinga/feature/apilistener-dynamic-port

ApiListener: Add support for dynamic port handling

Update documentation for new constants

Implement ApiListener status file removal on shutdown

ApiListener: Add support for dynamic port handling

Merge pull request #6530 from Icinga/bugfix/query-was-empty

IDO/MySQL: avoid empty queries

Expose Zone#all_parents via API

Rename Zone#GetAllParents() to Zone#GetAllParentsRaw()

IDO/MySQL: avoid empty queries

Merge pull request #6506 from Icinga/feature/improved-paths

Improve path handling in cmake and daemon

Update output for `--version`

This moves the system and build information before
the application paths which are normally not that important.

Take a note why the explicit configDir variable is needed on Windows

Update INSTALL.md a bit and kick RHEL 5 away

Refine log messages and code comments

Docs: Add a section to the 'upgrading' chapter

Docs: Improve the global constants section

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

Merge pull request #6528 from Icinga/bugfix/doc-user-enable_notifications

Document default of User#enable_notifications

Improve path handling in cmake and daemon

travis: Stop when one of the commands fail

cmake: Improve InstallConfig for stable path handling

* similar to install() a non absolute path is prefixed with CMAKE_INSTALL_PREFIX on runtime
* in case of CPack path with be prefixed with share/skel/
* DESTDIR is prefixed as well

Also see https://cmake.org/cmake/help/latest/command/install.html

Document default of User#enable_notifications

Merge pull request #6521 from gunnarbeutner/feature/references

Implement references

Merge pull request #6526 from Icinga/bugfix/wpessimizing-move-6524

icinga::PackObject(): shorten conversion to string

icinga::PackObject(): shorten conversion to string

refs #6524

Implement references

Update windows build scripts

Merge pull request #6427 from gunnarbeutner/fix/recursive-serialize

Improve error message for serializing objects with recursive references

Improve error message for serializing objects with recursive references

Merge pull request #6504 from roccolord/fix/itl-exlcude-squashfs

squashfs ignored

Merge pull request #6508 from gunnarbeutner/feature/dictionary-clear

Implement the Dictionary#clear script function

Implement the Dictionary#clear script function

Merge pull request #6505 from schinken/fix-clusterzonecheck-lag

Fix clusterzonecheck if not connected

Only check lag if connected

Invert connected check

squashfs ignored

Add squashfs to check_disk ignore list, used on Ubuntu for snaps.