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

fix(config): fix gorgone config generation for poller not using remote as proxy #8450

Merged
merged 4 commits into from
Mar 25, 2020

Conversation

cgagnaire
Copy link

Description

Use the remote_server_use_as_proxy option to determine wether Central or Remote is the "parent" of the Poller.

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
  • 20.04.x (master)

How this pull request can be tested ?

  • Have a Central > Remote > Poller kinf of setup,
  • Use ZMQ communication type for Poller,
  • Set "Use the Remote Server as a proxy" to "Yes",
  • Generate Gorgone configuration, authorized_clients key should be Remote thumbprint,
  • Set "Use the Remote Server as a proxy" to "No",
  • Generate Gorgone configuration, authorized_clients key should be Central thumbprint,

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)

@cgagnaire cgagnaire added area/configuration kind/fix pr/internal-rd PR made by the Centreon R&D Team labels Mar 23, 2020
@cgagnaire cgagnaire requested review from sc979 and adr-mo March 24, 2020 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/configuration kind/fix pr/internal-rd PR made by the Centreon R&D Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants