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 documentation #10081

Open
wants to merge 25 commits into
base: support/2.14
Choose a base branch
from
Open

Update documentation #10081

wants to merge 25 commits into from

Conversation

@cla-bot cla-bot bot added the cla/signed label Jun 10, 2024
@icinga-probot icinga-probot bot added this to the 2.14.3 milestone Jun 10, 2024
@icinga-probot icinga-probot bot added area/documentation End-user or developer help area/setup Installation, systemd, sample files bug Something isn't working enhancement New feature or request labels Jun 10, 2024
@Al2Klimov Al2Klimov self-assigned this Jun 21, 2024
@Al2Klimov Al2Klimov marked this pull request as ready for review October 2, 2024 15:17
@Al2Klimov Al2Klimov marked this pull request as draft October 22, 2024 09:05
Josef-Friedrich and others added 18 commits October 24, 2024 11:49
The installation documentation currently implies that all commands are
being prompted by the root user or an user with root-like privileges.
This is now explicitly stated and, additionally, another if-guard was
added to not include the "Add Icinga Package Repository" section for
Windows, as it does not fit there.

Closes #9959.
just like on packages.icinga.com and for the same security reasons we don't
use apt-key add anymore for: https://blog.cloudflare.com/dont-use-apt-key

Our repos already reference https://packages.icinga.com/icinga.key
and both RPM distro families properly handle that.
The Backports Repository is required for Debian 9, but we don't build even v10 anymore.
Josef-Friedrich and others added 6 commits October 24, 2024 11:49
We instruct the users to build as root and chown just /usr/local/icinga2/var, but at least `icinga2 api setup` also needs to modify /usr/local/icinga2/etc.
feat: Add the `$` character to the escape sequences table.
@Al2Klimov Al2Klimov marked this pull request as ready for review October 24, 2024 13:24
@Al2Klimov Al2Klimov requested a review from oxzi October 29, 2024 12:12
Copy link
Member

@oxzi oxzi left a comment

Choose a reason for hiding this comment

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

Looks good, I guess. I don't know what you want me to review?

This is a good alternative to `icinga2 feature enable debuglog`:

* Object creation/deletion via API happens immediately and requires no restart
* Hence, the debug log is enabled exactly as long as desired

Co-authored-by: alvar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation End-user or developer help area/setup Installation, systemd, sample files bug Something isn't working cla/signed enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants