Warn but don't fail on limit-exceeded error

bboreham · bboreham · commit 30d2b5f71cbc · 2018-06-08T13:59:15.000Z
diff --git a/pkg/chunk/aws/dynamodb_table_client.go b/pkg/chunk/aws/dynamodb_table_client.go
@@ -353,7 +353,11 @@ func (d dynamoTableClient) UpdateTable(ctx context.Context, current, expected ch
 				return err
 			})
 		}); err != nil {
-			return err
+			if awsErr, ok := err.(awserr.Error); ok && awsErr.Code() == "LimitExceededException" {
+				level.Warn(util.Logger).Log("msg", "update limit exceeded", "err", err)
+			} else {
+				return err
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -353,7 +353,11 @@ func (d dynamoTableClient) UpdateTable(ctx context.Context, current, expected ch`
`353`	`353`	`return err`
`354`	`354`	`})`
`355`	`355`	`}); err != nil {`
`356`		`- return err`
	`356`	`+ if awsErr, ok := err.(awserr.Error); ok && awsErr.Code() == "LimitExceededException" {`
	`357`	`+ level.Warn(util.Logger).Log("msg", "update limit exceeded", "err", err)`
	`358`	`+ } else {`
	`359`	`+ return err`
	`360`	`+ }`
`357`	`361`	`}`
`358`	`362`	`}`
`359`	`363`