cloudfoundry / bosh-aws-cpi-release

BOSH AWS CPI
Apache License 2.0
29 stars 62 forks source link

Resource tagging can produce an endless loop #50

Closed beyhan closed 7 years ago

beyhan commented 7 years ago

When working for the pull request https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/pull/49 I noticed that the tags method in the tag_manager can produce an endless loop. If the taggable resource is not found for what ever reason (infrastructure issue) tagging will be retried forever. Does it make sense to add a max number of retries here?

cppforlife commented 7 years ago

@beyhan talking about retry (https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/blob/d4eb7d46af19a7e5d1700d53d6d2dd4df103083f/src/bosh_aws_cpi/lib/cloud/aws/tag_manager.rb#L26) i assume? probably makes sense to use bounded retry logic.

beyhan commented 7 years ago

@cppforlife yes

cppforlife commented 7 years ago

sure does in our ci =)

cdutra commented 7 years ago

@cppforlife @beyhan We have a story in our icebox for that: https://www.pivotaltracker.com/story/show/144225287

cppforlife commented 7 years ago

fixed in v65.