Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

fix(mbi): add option --no-purge in script #275

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

garnier-quentin
Copy link
Contributor

No description provided.

@garnier-quentin garnier-quentin added the bug Something isn't working label Dec 13, 2022
@itoussies
Copy link

Not related to eventStatisticsBuilder.pl but the help message for perfdataStatisticsBuilder.pl says that --no-purge exists :

[root@ito-mbi ~]# /usr/share/centreon-bi/etl/perfdataStatisticsBuilder.pl --help
Usage:
    perfdataStatisticsBuilder.pl [options]

Options:
    --url   Specify the api url (default: 'http://127.0.0.1:8085').

    --severity
            Set the script log severity (default: 'info').

    --help  Print a brief help message and exits.

        Rebuild options:
            [-r | --rebuild] [-s|--start] <YYYY-MM-DD> [-e|--end] <YYYY-MM-DD> [--no-purge] [--month-only] [--centile-only] [--no-centile]
        Daily run options:
            [-d | --daily]

but it doesn't :

[root@ito-mbi ~]# /usr/share/centreon-bi/etl/perfdataStatisticsBuilder.pl -r --no-purge -s 2022-12-11 -e 2022-12-12
Unknown option: no-purge
Error: Command line error at /usr/share/perl5/vendor_perl/gorgone/class/script.pm line 129.

This option exists and works for the legacy script.

@garnier-quentin
Copy link
Contributor Author

You're right. it's done

@sonarqube-decoration
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kduret
Copy link
Contributor

kduret commented Jan 2, 2023

migrated to centreon/centreon#594

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants