width property for CustomMetricGroup

[ericxinzhang]

My code looks as following. I can specify height in CustomMonitoringProps. I am wondering why CustomMetricGroup does not have a width property?

I understand that for high level APIs (e.g. monitor<ResourceType>) it's easier for the library to calculate width dynamically. However for CustomMetricGroup may we have a width property?

const monitoring = new MonitoringFacade(this, 'MyDashboard');

monitoring.monitorCustom({
  height: 5,
  humanReadableName: 'API',
  metricGroups: [
    {
        title: 'API Errors',
        // I wish I could specify `width` here
        graphWidgetLegend: cw.LegendPosition.RIGHT,
        metrics:[
          new cloudwatch.Metric({...}),
          new cloudwatch.Metric({...}),
       ],
    },
    {
        title: 'API Latency',
        graphWidgetLegend: cw.LegendPosition.RIGHT,
        metrics:[
          new cloudwatch.Metric({...}),
          new cloudwatch.Metric({...}),
       ],
    }
  ],
});

[echeung-amzn]

This would be a feature request and is sort of captured as part of #66. I don't think there's a particular reason why it doesn't exist, although we may need to be careful in how to handle overflowing content (i.e. should we error out, or just let it happen, if that's allowed?).

[ericxinzhang]

Thanks Eugene! I tried editing the source of my dashboard, setting width of one widget from 6 to 8. CloudWatch just pushes the widget to next row and right aligned. IMO it's totally acceptable, and cdk-monitoring-constructs should just let it happen.

[echeung-amzn]

Moving to feature request: #487