search

Icinga / icingaweb2

A lightweight and extensible web interface to keep an eye on your environment. Analyse problems and act on them.
https://icinga.com/get-started/
GNU General Public License v2.0
806 stars 279 forks source link

add MariaDB to requirements #5246

Closed robertsilen closed 1 month ago

robertsilen commented 1 month ago

add MariaDB to requirements, as discussed in https://github.com/Icinga/icingaweb2/issues/5237

cla-bot[bot] commented 1 month ago

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @robertsilen

- If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please [contact us](https://icinga.com/company/contact/) if you think this is the case. - If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.
bobapple commented 1 month ago

@cla-bot check

nilmerg commented 1 month ago

Thank you :D Though, I'm not sure why you picked such an old state. Please adjust this based on main.

robertsilen commented 1 month ago

The 2.8 installation doc implicitly mentions MySQL in the requirements, so I was trying to add MariaDB there, but looks like this PR is going into main now.

For main's current installation doc I suggest this PR: https://github.com/Icinga/icingaweb2/pull/5247

Sorry, not sure how to best merge or replace these PRs. This PR can be closed if you are not accepting it into 2.8?

nilmerg commented 1 month ago

2.8 is not supported anymore and as such, changes won't be released nor be included in our official documentation.