cloudposse / helmfiles

Comprehensive Distribution of Helmfiles for Kubernetes
https://cloudposse.com/accelerate
Apache License 2.0
256 stars 80 forks source link

Use '%s' for accountId instead of '%v' #280

Closed sboardwell closed 3 years ago

sboardwell commented 3 years ago

what

why

From https://pkg.go.dev/fmt

%v the value in a default format

In the case of the AWS accountId the default format was a number meaning leading zeros are removed, meaning 0123456789 becomes 123456789 resulting in an incorrect role name.