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

provider/openstack: openstack_fw_policy_v1 defaulting shared results in 400 response #9829

Closed
fatmcgav opened this issue Nov 3, 2016 · 4 comments · Fixed by #9830
Closed
Labels

Comments

@fatmcgav
Copy link
Contributor

fatmcgav commented Nov 3, 2016

The openstack_fw_policy_v1 resource defaults shared to false.
However shared is an 'admin-only' field, as described here: openstack-api-ref/networking/v2/fwaas#create-firewall-policy

Attempting to specify shared with a non-admin accounts results in a HTTP 400 response from Neutron.

{"NeutronError": "Attribute 'shared' not allowed in POST"}

Terraform Version

0.7.8-dev

Affected Resource(s)

-openstack_fw_policy_v1

Terraform Configuration Files

resource "openstack_fw_policy_v1" "policy_1" {
  region = "uk-1"
  name = "policy_1"
  rules = [
    "${openstack_fw_rule_v1.allow_icmp.id}",
    "${openstack_fw_rule_v1.allow_ssh.id}"
  ]
}

Pull request incoming to resolve.

fatmcgav added a commit to fatmcgav/terraform that referenced this issue Nov 3, 2016
fatmcgav added a commit to fatmcgav/terraform that referenced this issue Nov 3, 2016
… specified.

Documentation updated to reflect removal of default value.
Fixes hashicorp#9829
fatmcgav added a commit to fatmcgav/terraform that referenced this issue Nov 3, 2016
… specified.

Documentation updated to reflect removal of default value.
Fixes hashicorp#9829
fatmcgav added a commit to fatmcgav/terraform that referenced this issue Nov 3, 2016
@jtopjian
Copy link
Contributor

jtopjian commented Nov 3, 2016

@fatmcgav passing shared: False as a non-admin user works with the default FWaaS driver. Do you know what FWaaS driver your cloud is using? I'd just like to know for reference.

The patch you made looks great, so I'm going to merge it. :)

@fatmcgav
Copy link
Contributor Author

fatmcgav commented Nov 3, 2016

@jtopjian I don't, but I can try and find-out... I doubt it's default though, as they've made lots of 'tweaks' to stuff...

@jtopjian
Copy link
Contributor

jtopjian commented Nov 3, 2016

@fatmcgav sounds good! Definitely interested if you can find out :)

gusmat pushed a commit to gusmat/terraform that referenced this issue Dec 6, 2016
… specified.

Documentation updated to reflect removal of default value.
Fixes hashicorp#9829
@ghost
Copy link

ghost commented Apr 20, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants