InstanceType m4.large deploy fails #4

Closed
ahawkins opened this Issue Aug 9, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@ahawkins

ahawkins commented Aug 9, 2016

This fails because not every value listed in the allowed values for the instance type is listed in the AWSInstanceType2Arch mapping. It seems all the m4.* are unmapped.

@bchav

This comment has been minimized.

Show comment
Hide comment
@bchav

bchav Aug 9, 2016

Contributor

Good callout. This was fixed in our original repo here: nicolaka/ddc-aws#44

We'll open a PR against this repo, as well.

Contributor

bchav commented Aug 9, 2016

Good callout. This was fixed in our original repo here: nicolaka/ddc-aws#44

We'll open a PR against this repo, as well.

@ahawkins

This comment has been minimized.

Show comment
Hide comment
@ahawkins

ahawkins Aug 9, 2016

@bchav ah, so which repo is the source of truth for the template?

ahawkins commented Aug 9, 2016

@bchav ah, so which repo is the source of truth for the template?

@bchav

This comment has been minimized.

Show comment
Hide comment
@bchav

bchav Aug 9, 2016

Contributor

You should be able to use either. We've merged changes. This should be automated in the future.

Closing this, feel free to re-open if you have questions.

Contributor

bchav commented Aug 9, 2016

You should be able to use either. We've merged changes. This should be automated in the future.

Closing this, feel free to re-open if you have questions.

@bchav bchav closed this Aug 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment