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

bm provider: issue with creating route table without route rule #68

Closed
ttsharma opened this issue Apr 25, 2017 · 3 comments
Closed

bm provider: issue with creating route table without route rule #68

ttsharma opened this issue Apr 25, 2017 · 3 comments
Assignees
Labels

Comments

@ttsharma
Copy link

Want to create a route table with 0 route rule.

Such a route table can come handy to attach to a subnet which I am not sure if I want to open to public internet or not.

baremetal_core_route_table does not allow me to create such a route table, I can create one from console though.

In case I define route table as:
resource "baremetal_core_route_table" "tts_tf_apps_routetable" {
compartment_id = "${var.compartment_ocid}"
vcn_id = "${baremetal_core_virtual_network.tts_tf_vcn.id}"
display_name = "tts_tf_routetable"
}

It errors:

  • baremetal_core_route_table.tts_tf_apps_routetable: "route_rules": required field is not set

If I define route table as it crashes and creates a crash.log
resource "baremetal_core_route_table" "tts_tf_apps_routetable" {
compartment_id = "${var.compartment_ocid}"
vcn_id = "${baremetal_core_virtual_network.tts_tf_vcn.id}"
display_name = "tts_tf_routetable"
route_rules {}
}

@jmajumde
Copy link

We need address two issue here

  1. Ability to create just a route table with 'baremetal_core_route_table' resource specifying route_rules {}

  2. Create a new resource 'baremetal_core_route_rule' resource to have a flexibility formcreating rules later as the rest API supports

@codycushing
Copy link
Contributor

Based on how this resources is consuming route_rule blocks what we need to support is complete omission of a block resulting in an empty array being posted in the create/update call--both already supported by the api.

@codycushing
Copy link
Contributor

Fixed in v2.0.1

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

No branches or pull requests

4 participants