Skip to content
This repository has been archived by the owner on Jan 15, 2023. It is now read-only.

Compilation of etcd-aws-cfn fails #16

Open
chriskite opened this issue Feb 2, 2017 · 1 comment
Open

Compilation of etcd-aws-cfn fails #16

chriskite opened this issue Feb 2, 2017 · 1 comment

Comments

@chriskite
Copy link

Per the documentation, I ran "go install ./..." in the project directory to install the etcd-aws-cfn tool. I got this error:

go install ./...
# github.com/crewjam/etcd-aws/aws
aws/health.go:24: cannot use cloudformation.String("1") (type *cloudformation.St
ringExpr) as type *cloudformation.IntegerExpr in field value
aws/health.go:33: cannot use cloudformation.String("60") (type *cloudformation.S
tringExpr) as type *cloudformation.IntegerExpr in field value
aws/health.go:54: cannot use cloudformation.String("1") (type *cloudformation.St
ringExpr) as type *cloudformation.IntegerExpr in field value
aws/health.go:63: cannot use cloudformation.String("300") (type *cloudformation.StringExpr) as type *cloudformation.IntegerExpr in field value

I replaced cfn.String with cfn.Integer in health.go, and compilation succeeded. I am going to test to make sure the tool works as expected, then I'll submit a pull request.

@crewjam
Copy link
Owner

crewjam commented Feb 10, 2017

Excellent, thanks!

This is due to upstream changes in go-cloudformation which is in turn due to upstream changes AWS have made in their definition of the CFN schema, so you're on the right track.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants