Eugene Cheung

Results 12 issues of Eugene Cheung

### Feature scope AWS Secrets Manager ### Describe your suggested feature Announced in https://aws.amazon.com/about-aws/whats-new/2022/05/aws-secrets-manager-publishes-usage-metrics-to-amazon-cloudwatch/ Docs: https://docs.aws.amazon.com/secretsmanager/latest/userguide/monitoring-cloudwatch.html

good first issue
feature-request
effort: small

### Describe the bug Originally posted in https://github.com/aws/jsii/issues/3410#issuecomment-1097127406 In module A: ```ts export type MetricWithAlarmSupport = Metric | MathExpression; export class MetricFactory { adaptMetric(metric: MetricWithAlarmSupport): MetricWithAlarmSupport { ... } }...

bug
p1

By default, JavaScript projects: - Run a workflow to upgrade dependencies [daily](https://github.com/projen/projen/blob/f90f012ef7aa212ed82aecb9b5b608f9894447ab/src/javascript/upgrade-dependencies.ts#L386) - Publish libraries on pushes to the main branch This combination seems to be creating huge amounts of...

### Discussed in https://github.com/cdklabs/cdk-monitoring-constructs/discussions/450 Originally posted by **ericxinzhang** November 2, 2023 My code looks as following. I can specify `height` in `CustomMonitoringProps`. I am wondering why `CustomMetricGroup` does not have...

enhancement
feature-request

### Feature scope All ### Describe your suggested feature From https://aws.amazon.com/about-aws/whats-new/2023/06/amazon-cloudwatch-dashboard-variables/ : --- We are excited to announce Amazon CloudWatch dashboard variables, a new experience that makes it easier for...

feature-request

Leveraging https://github.com/aws/aws-cdk/pull/26819 --- _By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license_

do-not-merge

This is a continuation of #534, but it includes breaking changes to some existing metric factories' constructor API contracts. Also includes some minor private method renaming for consistency and a...

do-not-merge

1. Throw an error at build time if we're missing the prop when it's needed. 2. Automatically get the region when needed when using `monitorScope`. --- _By submitting this pull...

do-not-merge

This allows users to target different regions/account for specific things within a facade rather than relying on 1 facade per region/account with the facade's global settings, so something like: ```ts...

do-not-merge

Avoids potentially referring to an API ID that ends up being a token. --- _By submitting this pull request, I confirm that my contribution is made under the terms of...

do-not-merge