Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: incorrect node_pools variable type #1424

Merged
merged 2 commits into from
Oct 26, 2022

Commits on Oct 12, 2022

  1. fix: incorrect node_pools variable type

    `node_pool` variables are a mix of string, bool, number
    therefore, it is better to use `map(any)` to indicate its type
    
    moreover, the use of `list(map(string))` is breaking
    the use of this module in `cdktf`.
    
    in the generated code it will produce
    `[]map[string]string` for `node_pools` option
    but we want `[]map[string]interface{}` in order to pass correct value,
    e.g. `initial_node_count` is a `number` not `string`
    michaellzc committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    7c253f8 View commit details
    Browse the repository at this point in the history

Commits on Oct 25, 2022

  1. Configuration menu
    Copy the full SHA
    dfee3cd View commit details
    Browse the repository at this point in the history