search

centreon / centreon-gorgone

Daemon tasks handler
Other
6 stars 11 forks source link

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

Open garnier-quentin opened 1 year ago

itoussies commented 1 year ago

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 commented 1 year ago

You're right. it's done

sonarqube-decoration[bot] commented 1 year ago

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 commented 1 year ago

migrated to https://github.com/centreon/centreon/pull/594