Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(autoscaling): enable group metrics collections #7432

Merged
merged 16 commits into from
Jul 20, 2020

Conversation

NetaNir
Copy link
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
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
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
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 <[email protected]>
Co-authored-by: Eli Polonsky <[email protected]>
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 <[email protected]>
Co-authored-by: Eli Polonsky <[email protected]>
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