fix alarm metric dimension CacheClusterId

[chelomontilla]

what

• Update CacheClusterId dimension of the alarms

why

• The current dimension doesn't match the right cluster id

references

• Closes Cloudwatch alarms insufficient data #56

[maximmi]

@chelomontilla please, try to provide CacheNodeId dimension as well as stated in documentation here and see if it will help.

[chelomontilla]

@maximmi I did an aws elasticache describe-cache-clusters and the output doesn't display any CacheNodeId.
Here the output:

{
   "CacheClusterId":"my-cluster-cache-001",
   "ReplicationGroupId":"my-cluster-cache",
   "CacheClusterStatus":"available",
   "SnapshotRetentionLimit":0,
   "ClientDownloadLandingPage":"https://console.aws.amazon.com/elasticache/home#client-download:",
   "PendingModifiedValues":{

   },
   "Engine":"redis",
   "CacheSecurityGroups":[

   ],
   "NumCacheNodes":1,
   "AuthTokenLastModifiedDate":"2020-02-24T08:35:42.379Z",
   "AutoMinorVersionUpgrade":true,
   "TransitEncryptionEnabled":true,
   "SecurityGroups":[
      {
         "Status":"active",
         "SecurityGroupId":"sg-xxxxxxxx"
      }
   ],
   "CacheNodeType":"cache.t2.small",
   "PreferredMaintenanceWindow":"wed:03:00-wed:04:00",
   "CacheSubnetGroupName":"my-cluster-cache",
   "AuthTokenEnabled":true,
   "AtRestEncryptionEnabled":false,
   "EngineVersion":"5.0.6",
   "CacheClusterCreateTime":"2020-01-15T11:55:02.665Z",
   "PreferredAvailabilityZone":"eu-west-1a",
   "SnapshotWindow":"06:30-07:30",
   "NotificationConfiguration":{
      "TopicStatus":"active",
      "TopicArn":"arn:aws:sns:eu-west-1:xxxxxxxxxxx:xxxxxxxxxxxxxxxxxxx"
   },
   "CacheParameterGroup":{
      "CacheNodeIdsToReboot":[

      ],
      "CacheParameterGroupName":"my-cluster-cache",
      "ParameterApplyStatus":"in-sync"
   }
}

[osterman]

/test all

[jamengual]

/test all

[jamengual]

/rebuild-readme

[jamengual]

@chelomontilla can you rebase from master, please?

[mergify]

This pull request is now in conflict. Could you fix it @chelomontilla? 🙏

This Pull Request has been updated, so we're dismissing all reviews.

[nitrocode]

Looks like this was fixed by a previous PR.