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

enh(gorgone) Update config form to manage gorgone options #8197

Merged
merged 26 commits into from
Dec 17, 2019
Merged

enh(gorgone) Update config form to manage gorgone options #8197

merged 26 commits into from
Dec 17, 2019

Conversation

loiclau
Copy link
Contributor

@loiclau loiclau commented Dec 16, 2019 •
edited

Description

Update admin > parameters form to manage gorgone options

continuation of https://github.com/centreon/centreon/pull/8184 for others form

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 ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

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)

loiclau and others added 8 commits December 10, 2019 17:13
@loiclau
fix(js): fix check type of broker connections count (#8185)
e5ff5f7
@callapa
fea(api): Added API to find the configuration of pollers (#8186)
58c9679 
Added API to find the configuration of pollers
@vhr
[BE]Update APIs related with BAM to use context instead DataRepresent…
918e072 
…ers (#8107)

* feat(base): add AnnotationRegistry::registerLoader to register the annotations in entities
* feat(base): add interface for serializer contexts
* feat(base): extend Webservice abstract class and separate duplicated code in trait related with dependencies
* feat(base): extend pagination service to work with context files
* feat(base): add Serializer groups to entities related with BAM and make their setters and getters to accept null values
* feat(api): update NagiosServer entity and repository to provide list of entities by IDs
* feat(api): use context for AclGroup webservice
* feat(api): use context for Image webservice
* fix(base): add docblock
* feat(api): use context in COntactGroup webservice
* feat(api): use context in Command webservice
* feat(api): use context in Escalation webservice
* feat(api): use context in Timeperiod webservice
* fix(test): improve phpunit tests
* fix(test): reduce the amount of exceptions in codestyle
* chore(dependencies): update composer dependencies
* feat(test): cover APIs with acceptance tests
* feat(test): add context information when CentreonPaginationServiceMock prepare response
* fix(test): upgrade unit tests to use WebServiceExecuteTestTrait and Interfaces for authentication
* fix(test): adjust unit texts fixtures
* fix(test): codestyle
* feat(test): extend test bootstrap to provide Serializer service from DI
* feat(base): extend Serializer service with ArrayDenormalizer
* feat(test): extend WebServiceExecuteTestTrait to mock the repositories for paginations
* feat(test): use mock of CentreonDbManager service from DI
* feat(di): separate the serialize service and object normalizer service
* fix(api): change return type of id getter of entities to be integer
* fix(test): add getters getId
* fix(test): wrong namespaces
* fix(test): improve test bootstrap to work with phing
* fix(test): adjust acceptance tests related with APIv1
* chore(dependency): update composer lock
* fix(test): remove old API tests
* chore(dependency): update composer lock
* feat(api): add alias with GET method to object centreon_configuration_remote with action list to postGetRemotesList
* chore(api): improve docblocks
* chore(api): change return type of ContactGroup::getCgActivate
* chore(style): review feedback
* chore(style): make the code to be compatible with PSR-12 too
* fix(test): reduce warnings
* chore(style): improve docblocks and codestyle
* fix(test): bot feedback

Resolve BAM-808
@kduret
fix(ui): check format of top counter's endpoints return data (#8187)
4c87741 
Refs: MON-4590
@kduret
fix(ui): display top counter even if monitoring is off (#8193)
03629f6 
Refs: MON-4590
@loiclau
update centcore config form to gorgone
db30892
@loiclau
update admin-param-centcore form
b33304e
@loiclau
update debug form
14a7160
@loiclau loiclau changed the title update centcore config form to gorgone enh(gorgone) Update config form to manage gorgone options Dec 16, 2019
loiclau and others added 3 commits December 17, 2019 16:56
@loiclau @cgagnaire
Update www/include/Administration/parameters/api/api.php
b9582f1 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
@loiclau @cgagnaire
Update www/include/Administration/parameters/api/help.php
b2bbf8e 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
@loiclau @cgagnaire
Update www/include/Administration/parameters/api/help.php
c393f27 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
cgagnaire
cgagnaire reviewed Dec 17, 2019
View reviewed changes
Copy link
Contributor

@cgagnaire cgagnaire left a comment

Choose a reason for hiding this comment

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

perfdata_sync and logs_sync are not used anymore.
But maybe it should be removed from PR https://github.com/centreon/centreon/pull/8172

loiclau and others added 3 commits December 17, 2019 17:45
@loiclau @cgagnaire
Update www/include/Administration/parameters/gorgone/help.php
7486a0e 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
@loiclau @cgagnaire
Update www/include/Administration/parameters/gorgone/help.php
02e4f3c 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
@loiclau @cgagnaire
Update www/include/Administration/parameters/gorgone/help.php
1e4152f 
Co-Authored-By: Colin Gagnaire <cgagnaire@centreon.com>
@loiclau loiclau changed the base branch from master to MON-gorgone-integration December 17, 2019 16:51
@loiclau loiclau merged commit 05394a1 into MON-gorgone-integration Dec 17, 2019
@loiclau loiclau deleted the MON-4509-other-form branch December 17, 2019 16:54
sc979 added a commit that referenced this pull request Mar 18, 2020
@sc979 @loiclau
@cgagnaire @callapa @bdauria @lpinsivy @kduret
enh(core): replace centcore with gorgoned (#8414)
985c8ba 
* enh(core): generate centreon gorgone configuration file (#8160)

* enh(core): add gorgone configuration file generation

* fix: folder paths

* fix(core): naming of file and field type

* enh(core): update configuration file to take in account last modif

* fix(core): correct path to private key

* enh(core): add nodes module

* enh(core): store stats in cache dir

* enh(core): enable broker stats collection

* enh(poller) Update poller form to manage gorgone (#8184)

* enh(gorgone) Update config form to manage gorgone options  (#8197)

* enh(chore) replace centcore by gorgone (#8172)

* remove centcore perl script (#8209)

* fix(UI): do not display gorgone options on central's form (#8206)

* fix(UI): do not display gorgone options on central

* remove duplicated code (#8215)

* enh(admin): adapt broker stats page to new folder tree (#8216)

* enh(core): update script to manage gorgone configuration file (#8229)

* enh: exception management

* enh(update): generate gorgone conf file on update

* fix(translation): avoid empty gettext translation

* enh(core): Add a yaml configuration loader (#8270)

* MON-4737 enh(core): split gorgone configuration in different files (#8276)

* enh(core): adapt upgrade script to latest modifications (#8284)

* adapt update script

* enh: add more check brefore trying to create the conf files

* fix: correct path in the template

* enh(core): update gorgone configuration files (#8293)

* update configuration location

* enh(gorgone): Added API to communicate with the Gorgone server (#8294)

* enh(install): fix default keys pathes (#8302)

* fix db name configuration template

* replace remote data folder

* fix rest api test

* enh(UI): Add nowrap style to badge class - from aladdinchan (#8314)

* fix(ui): Add hash:8 suffix to font files to avoid asset conflicts and optimize Webpack config (#8305)

* Mon 4640 remove deprecated engine fields (#8233)

* fix(CI): add new tests to clapi diff

* fix(core): source install (#8396)

* enh(core): add gorgone user and copy ssh keys

* fix(api): fix options retrieval issue (#8412)

* fix(core): manage the gorgone configuration files (#8400)

* fix(core): duplicate SSH key on source upgrade (#8421)

Co-authored-by: loiclau <loic.lau@gmail.com>
Co-authored-by: Colin Gagnaire <cgagnaire@centreon.com>
Co-authored-by: Laurent Calvet <lcalvet@centreon.com>
Co-authored-by: Bruno d'Auria <bdauria@centreon.com>
Co-authored-by: Matthieu Kermagoret <mkermagoret@centreon.com>
Co-authored-by: Laurent Pinsivy <lpinsivy@centreon.com>
Co-authored-by: Kevin Duret <duret.kevin@gmail.com>
Co-authored-by: Laurent Calvet <laurent@callapa.com>
Co-authored-by: Kevin Duret <kduret@centreon.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cgagnaire cgagnaire cgagnaire requested changes

@sc979 sc979 sc979 approved these changes

Assignees
No one assigned
Labels
area/administration kind/feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@loiclau @cgagnaire @sc979 @callapa @vhr @kduret