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 Icinga DB related docs #9423

Merged
merged 14 commits into from
Jun 29, 2022
Merged

Update Icinga DB related docs #9423

merged 14 commits into from
Jun 29, 2022

Conversation

julianbrost
Copy link
Contributor

@julianbrost julianbrost commented Jun 29, 2022

Merge conflicts were resolved by picking the version from the master. Therefore this also backports #9152. There are also some small differences with heading levels when comparing the two PRs, but the result looks fine (but feel free to double check).

backport of #9387

@cla-bot cla-bot bot added the cla/signed label Jun 29, 2022
@icinga-probot icinga-probot bot added this to the 2.13.4 milestone Jun 29, 2022
@icinga-probot icinga-probot bot added the area/documentation End-user or developer help label Jun 29, 2022
@julianbrost julianbrost marked this pull request as ready for review June 29, 2022 15:01
lippserd
lippserd previously approved these changes Jun 29, 2022
lrupp and others added 14 commits June 29, 2022 17:34
We are using a central DB-Cluster, therefor Icinga2 got a database on this remote cluster.
We used the icinga DB user, created during the step before, to import the DB schema.

For this, the additional GRANT options `ALTER` and `CREATE` where needed to install the schema successfully via:
```bash
mysql -u icinga -p icinga < /usr/share/icinga2-ido-mysql/schema/mysql.sql
```

Someone may find this a security problem. But I see two benefits:
* these GRANT options are only set for the icinga.* database
* this setup allows to execute also all later UPGRADE commands as the database user `icinga` - and not as the DB Administrator.

Two additional notes:
* I also included a small warning about the used default DB password
* sadly, I have no way to change this also in the provided mariadb-centos7.png image, so this is left unchanged at the moment
Almost every Icinga 2 node setup requires the Icinga 2 API, as Icinga
Web connects to it, Icinga DB requires it, and although it's not clearly
mentioned anywhere in the documentation at the moment, it enables the
cluster communication functionality.
There is a separate chapter, so why draw attention to installing even
more things.
The Icinga DB docs will link to installing Icinga DB Web.
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

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

OK, based on the PR-to-PR diff you sent me per PM:

  • IDO MySQL stuff: should now be gone as you've explicitly backported the commit.
  • AL2/CentOS headings "symmetric shifts": ah, screw it. It's 19:00+ and we have to release Icinga DB.

@Al2Klimov Al2Klimov merged commit 3ba7c08 into support/2.13 Jun 29, 2022
@icinga-probot icinga-probot bot deleted the icingadb-docs-2.13 branch June 29, 2022 17:11
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 cla/signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants