[Bps-public-commit] app-aws-cloudwatch-monitor branch 0.02/split-metric-payload-upload updated. 7e4b3f1511c4a880c151b61900da1dd43fb72064
BPS Git Server
git at git.bestpractical.com
Wed May 11 23:23:30 UTC 2022
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "app-aws-cloudwatch-monitor".
The branch, 0.02/split-metric-payload-upload has been updated
via 7e4b3f1511c4a880c151b61900da1dd43fb72064 (commit)
via fdec15299258685a8afcc7ddd41c78b4188a1faf (commit)
from cad07ee4aa465ae11e1d2f17aca8d25452b9734d (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 7e4b3f1511c4a880c151b61900da1dd43fb72064
Author: Blaine Motsinger <blaine at bestpractical.com>
Date: Wed May 11 18:22:27 2022 -0500
Add POSIX to requirements
diff --git a/Makefile.PL b/Makefile.PL
index aa61497..2f156c1 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -48,6 +48,7 @@ WriteMakefile(
'Module::Loader' => 0,
'parent' => 0,
'Pod::Usage' => '1.67', # rewrite in 1.62, bugfixes in 1.67
+ 'POSIX' => 0,
'strict' => 0,
'Try::Tiny' => 0,
'URI::Escape' => 0,
commit fdec15299258685a8afcc7ddd41c78b4188a1faf
Author: Blaine Motsinger <blaine at bestpractical.com>
Date: Wed May 11 18:19:18 2022 -0500
Remove metric upload limitation from docs
diff --git a/README b/README
index cfbc7c6..e278fe6 100644
--- a/README
+++ b/README
@@ -60,30 +60,6 @@ CONFIGURATION
exists, "config.ini" will be loaded from there regardless of a config
file in "/etc/aws-cloudwatch-monitor/".
-KNOWN LIMITATIONS
- AWS CloudWatch limits each upload to no more than 20 different metrics
- AWS CloudWatch will return a 400 response if attempting to upload more
- than 20 different metrics at once.
-
- <https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API_Pu
- tMetricData.html>
-
- A metrics collection can quickly exceed 20 metrics since each check
- module gathers multiple metrics.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs --check Memory --check DiskSpace --check Inode --disk-path /
- Failed to call CloudWatch: HTTP 400. Message: The collection MetricData must not have a size greater than 20.
-
- Until this limitation is worked around in a future release of
- "App::AWS::CloudWatch::Monitor", splitting the checks into separate
- aws-cloudwatch-monitor commands allows the uploads to succeed.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
- aws-cloudwatch-monitor --check Memory --check DiskSpace --check Inode --disk-path /
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
BUGS AND ENHANCEMENTS
Please report any bugs or feature requests at rt.cpan.org
<https://rt.cpan.org/Public/Dist/Display.html?Name=App-AWS-CloudWatch-Mo
diff --git a/README.md b/README.md
index 5cea11c..691dede 100644
--- a/README.md
+++ b/README.md
@@ -58,27 +58,6 @@ After creating the file, edit and update the values accordingly.
**NOTE:** If the `$ENV{HOME}/.config/aws-cloudwatch-monitor/` directory exists, `config.ini` will be loaded from there regardless of a config file in `/etc/aws-cloudwatch-monitor/`.
-# KNOWN LIMITATIONS
-
-## AWS CloudWatch limits each upload to no more than 20 different metrics
-
-AWS CloudWatch will return a 400 response if attempting to upload more than 20 different metrics at once.
-
-[https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API\_PutMetricData.html](https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API_PutMetricData.html)
-
-A metrics collection can quickly exceed 20 metrics since each check module gathers multiple metrics.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs --check Memory --check DiskSpace --check Inode --disk-path /
- Failed to call CloudWatch: HTTP 400. Message: The collection MetricData must not have a size greater than 20.
-
-Until this limitation is worked around in a future release of `App::AWS::CloudWatch::Monitor`, splitting the checks into separate [aws-cloudwatch-monitor](https://metacpan.org/pod/aws-cloudwatch-monitor) commands allows the uploads to succeed.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
- aws-cloudwatch-monitor --check Memory --check DiskSpace --check Inode --disk-path /
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
# BUGS AND ENHANCEMENTS
Please report any bugs or feature requests at [rt.cpan.org](https://rt.cpan.org/Public/Dist/Display.html?Name=App-AWS-CloudWatch-Monitor).
diff --git a/lib/App/AWS/CloudWatch/Monitor.pm b/lib/App/AWS/CloudWatch/Monitor.pm
index 2914609..1ecf5a5 100644
--- a/lib/App/AWS/CloudWatch/Monitor.pm
+++ b/lib/App/AWS/CloudWatch/Monitor.pm
@@ -246,27 +246,6 @@ After creating the file, edit and update the values accordingly.
B<NOTE:> If the C<$ENV{HOME}/.config/aws-cloudwatch-monitor/> directory exists, C<config.ini> will be loaded from there regardless of a config file in C</etc/aws-cloudwatch-monitor/>.
-=head1 KNOWN LIMITATIONS
-
-=head2 AWS CloudWatch limits each upload to no more than 20 different metrics
-
-AWS CloudWatch will return a 400 response if attempting to upload more than 20 different metrics at once.
-
-L<https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API_PutMetricData.html>
-
-A metrics collection can quickly exceed 20 metrics since each check module gathers multiple metrics.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs --check Memory --check DiskSpace --check Inode --disk-path /
- Failed to call CloudWatch: HTTP 400. Message: The collection MetricData must not have a size greater than 20.
-
-Until this limitation is worked around in a future release of C<App::AWS::CloudWatch::Monitor>, splitting the checks into separate L<aws-cloudwatch-monitor> commands allows the uploads to succeed.
-
- aws-cloudwatch-monitor --check Process --process apache --process postgres --process master --process emacs
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
- aws-cloudwatch-monitor --check Memory --check DiskSpace --check Inode --disk-path /
- Successfully reported metrics to CloudWatch. Reference Id: <snip>
-
=head1 BUGS AND ENHANCEMENTS
Please report any bugs or feature requests at L<rt.cpan.org|https://rt.cpan.org/Public/Dist/Display.html?Name=App-AWS-CloudWatch-Monitor>.
-----------------------------------------------------------------------
Summary of changes:
Makefile.PL | 1 +
README | 24 ------------------------
README.md | 21 ---------------------
lib/App/AWS/CloudWatch/Monitor.pm | 21 ---------------------
4 files changed, 1 insertion(+), 66 deletions(-)
hooks/post-receive
--
app-aws-cloudwatch-monitor
More information about the Bps-public-commit
mailing list