Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(doc): change grant option for remote server database centreon user #7888

Merged
merged 1 commit into from
Oct 2, 2019

Conversation

cgagnaire
Copy link

Description

As Remote Server does not have remote database, the grant option must be done to the "localhost" user.

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 2.8.x
  • 18.10.x
  • 19.04.x
  • 19.10.x (master)

How this pull request can be tested ?

If the grant is done to the "server ip" user, the import will result with the following error :

SQLSTATE[28000]: Invalid authorization specification: 1045 Access denied for user 'centreon'@'localhost' (using password: YES)

Checklist

Community contributors & Centreon team

  • I followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

Centreon team only

  • I have made sure that the unit tests related to the story are successful.
  • I have made sure that unit tests cover 80% of the code written for the story.
  • I have made sure that acceptance tests related to the story are successful (local and CI)


Ajout des droits pour que l'utilisateur de base de données centreon puisse utiliser la commande **LOAD DATA INFILE**::

# mysql -u root -p
Copy link
Contributor

Choose a reason for hiding this comment

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

this won't work on a deported DB like MySQL8
you should restore the previous command

Copy link
Author

Choose a reason for hiding this comment

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

Remote Server is not supposed to have a deported database, at least for most installations.

@kduret kduret merged commit f68502b into master Oct 2, 2019
@kduret kduret deleted the remote-server-doc branch October 2, 2019 06:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants