Skip to content

feat(autoscaling): enable group metrics collections#7432

Merged
NetaNir merged 16 commits intomasterfrom
neta-eli/autoscaling-metrics-collection
Jul 20, 2020
Merged

feat(autoscaling): enable group metrics collections#7432
NetaNir merged 16 commits intomasterfrom
neta-eli/autoscaling-metrics-collection

Conversation

@NetaNir
Copy link
Copy Markdown
Contributor

@NetaNir NetaNir commented Apr 19, 2020

Add support for MetricsCollection.
See README for details about the feature and implementation

closes #6453


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@NetaNir NetaNir requested a review from iliapolo April 19, 2020 07:16
@mergify mergify bot added contribution/core This is a PR that came from AWS. labels Apr 19, 2020
@NetaNir NetaNir added the pr/do-not-merge This PR should not be merged at this time. label Apr 19, 2020
Copy link
Copy Markdown
Contributor

@iliapolo iliapolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NetaNir Sweet!

What about adding a metrics property to the constructor as well?

Copy link
Copy Markdown
Contributor

@nija-at nija-at left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README needs to be updated as well.

@NetaNir NetaNir requested review from iliapolo and nija-at June 27, 2020 06:29
@NetaNir NetaNir removed the pr/do-not-merge This PR should not be merged at this time. label Jun 27, 2020
@NetaNir NetaNir changed the title feat(autoscaling): add metrics collections feat(autoscaling): add support for group metrics collections Jun 27, 2020
@nija-at nija-at changed the title feat(autoscaling): add support for group metrics collections feat(autoscaling): group metrics collections Jul 2, 2020
@NetaNir NetaNir requested a review from iliapolo July 20, 2020 01:18
@NetaNir NetaNir changed the title feat(autoscaling): group metrics collections feat(autoscaling): enable group metrics collections Jul 20, 2020
@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
  • Commit ID: ab84a7a
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@NetaNir NetaNir merged commit 9867555 into master Jul 20, 2020
@NetaNir NetaNir deleted the neta-eli/autoscaling-metrics-collection branch July 20, 2020 14:59
Chriscbr pushed a commit to Chriscbr/aws-cdk that referenced this pull request Jul 23, 2020
* feat(autoscaling): add metrics collections

Co-authored-by: Neta Nir <neta@amazon.com>
Co-authored-by: Eli Polonsky <epolon@amazon.com>
curtiseppel pushed a commit to curtiseppel/aws-cdk that referenced this pull request Aug 11, 2020
* feat(autoscaling): add metrics collections

Co-authored-by: Neta Nir <neta@amazon.com>
Co-authored-by: Eli Polonsky <epolon@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to switch on metric collections for autoscaling group

4 participants