tyler.durden
/
centreon-plugins
mirror of https://github.com/centreon/centreon-plugins.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #257 from bcournaud/master 

Fix github ticket count when number of tickets is greater than 100
This commit is contained in:
qgarnier 2015-12-23 16:10:39 +01:00
parent 2c96fe3550 346a677721
commit 813b9c6691
1 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
apps/github/mode/issues.pm
View File

@ -73,12 +73,10 @@ sub check_options {
$self->{output}->add_option_msg(short_msg => "Wrong critical threshold '" . $self->{option_results}->{critical} . "'.");
$self->{output}->option_exit();
}
$self->{option_results}->{url_path} = "/repos/" . $self->{option_results}->{owner} . "/" . $self->{option_results}->{repository} . "/issues";
$self->{option_results}->{url_path} = "/search/issues?q=state:open+repo:" . $self->{option_results}->{owner} . "/" . $self->{option_results}->{repository};
if (defined($self->{option_results}->{label}) && $self->{option_results}->{label} ne '') {
$self->{option_results}->{get_param} = ['state=open', 'labels=' . $self->{option_results}->{label}, 'per_page=1000'];
} else {
$self->{option_results}->{get_param} = ['state=open', 'per_page=1000'];
$self->{option_results}->{url_path} .= "+label:" . $self->{option_results}->{label};
}
$self->{http}->set_options(%{$self->{option_results}});
}
@ -86,6 +84,7 @@ sub check_options {
sub run {
my ($self, %options) = @_;
my $jsoncontent = $self->{http}->request();
my $json = JSON->new;
@ -98,9 +97,8 @@ sub run {
$self->{output}->add_option_msg(short_msg => "Cannot decode json response");
$self->{output}->option_exit();
}
# Number of issues is array length
my $nb_issues = @{$webcontent};
my $nb_issues = $webcontent->{total_count};
my $exit = $self->{perfdata}->threshold_check(value => $nb_issues, threshold => [ { label => 'critical', exit_litteral => 'critical' }, , { label => 'warning', exit_litteral => 'warning' } ]);