)]}'
{"lib/puppet/provider/neutron_router/neutron.rb":[{"author":{"_account_id":7604,"name":"Sergey Kolekonov","email":"skolekonov@mirantis.com","username":"skolekonov"},"change_message_id":"bc9e5a91037b6e3177ed34e68769ac042d97d3a1","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    end"},{"line_number":115,"context_line":"    @gateway_network_name"},{"line_number":116,"context_line":"  end"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"  def gateway_network_name\u003d(value)"},{"line_number":119,"context_line":"    if value \u003d\u003d \u0027\u0027"},{"line_number":120,"context_line":"      auth_neutron(\u0027router-gateway-clear\u0027, name)"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"7a740942_e82d01fe","line":117,"updated":"2015-12-10 16:10:04.000000000","message":"Please add a function to update router\u0027s type, smth like\n\ndef distributed\u003d(value)\n   auth_neutron(\u0027router-update\u0027, \"--distributed\u003d#{value}\", name)\nend\n\nBut please also note that to update a router you need to set admin_state to down, that update a router, and set it back to up","commit_id":"80eb4ba0d4c514c937becc902d711b8d69a69f57"},{"author":{"_account_id":7604,"name":"Sergey Kolekonov","email":"skolekonov@mirantis.com","username":"skolekonov"},"change_message_id":"c8d4fea1cf10d133846547c03a4fc8ff620c75f7","unresolved":false,"context_lines":[{"line_number":143,"context_line":"  def distributed\u003d(value)"},{"line_number":144,"context_line":"    results \u003d auth_neutron(\"router-show\", \u0027--format\u003dshell\u0027, resource[:name])"},{"line_number":145,"context_line":"    attrs \u003d self.class.parse_creation_output(results)"},{"line_number":146,"context_line":"    admin_state_up(False)"},{"line_number":147,"context_line":"    auth_neutron(\u0027router-update\u0027, \"--distributed\u003d#{value}\", name)"},{"line_number":148,"context_line":"    if attrs[\u0027admin_state_up\u0027] \u003d\u003d \u0027True\u0027"},{"line_number":149,"context_line":"      admin_state_up(True)"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"7a740942_e0f9584f","line":146,"updated":"2015-12-11 16:29:13.000000000","message":"You should not call this function like this, also \u0027false\u0027 and \u0027true\u0027 parameters in lower case should be used otherwise they will be interpreted incorrectly. Smth like this works:\n\n  def admin_state_up\u003d(value)\n    admin_state_up(value)\n  end\n\n  def admin_state_up(value)\n    auth_neutron(\u0027router-update\u0027, \"--admin-state-up\u003d#{value}\", name)\n  end\n\n  def distributed\u003d(value)\n    results \u003d auth_neutron(\"router-show\", \u0027--format\u003dshell\u0027, resource[:name])\n    attrs \u003d self.class.parse_creation_output(results)\n    admin_state_up(false)\n    auth_neutron(\u0027router-update\u0027, \"--distributed\u003d#{value}\", name)\n    if attrs[\u0027admin_state_up\u0027] \u003d\u003d \u0027True\u0027\n      admin_state_up(true)\n    end\n  end","commit_id":"4c8f826c20fe59b16a470723908281c002a5a645"}]}