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

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

Open
garnier-quentin wants to merge 3 commits into
developfrom
MON-16082-mbi-nopurge-options
Open

fix(mbi): add option --no-purge in script#275
garnier-quentin wants to merge 3 commits into
developfrom
MON-16082-mbi-nopurge-options

Conversation

@garnier-quentin

Copy link
Copy Markdown
Contributor

No description provided.

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

Copy link
Copy Markdown

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
Copy Markdown
Contributor Author

You're right. it's done

@sonarqube-decoration

Copy link
Copy Markdown

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

kduret commented Jan 2, 2023

Copy link
Copy Markdown
Contributor

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.

4 participants