)]}'
{"lib/neutron_plugins/services/l3":[{"author":{"_account_id":19307,"name":"Antonio Ojea","email":"antonio.ojea.garcia@gmail.com","username":"itsuugo"},"change_message_id":"20ffbd74c276b0740d5bc2a6c7fee1400f1283dd","unresolved":false,"context_lines":[{"line_number":103,"context_line":"default_v4_route_devs\u003d$(ip -4 route | grep ^default | awk \u0027{print $5}\u0027)"},{"line_number":104,"context_line":"die_if_not_set $LINENO default_v4_route_devs \"Failure retrieving default IPv4 route devices\""},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"default_v6_route_devs\u003d$(ip -6 route list match default table all | grep via | awk \u0027{print $5}\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"function _determine_config_l3 {"},{"line_number":109,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file $Q_L3_CONF_FILE\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3f79a3b5_929a0911","line":106,"range":{"start_line":106,"start_character":24,"end_line":106,"end_character":94},"updated":"2018-10-09 06:50:55.000000000","message":"OMG you are right, how could I overlook that? ... need more coffee and less rushing :)","commit_id":"866efef17af8f0d04240bf05714cb12452185822"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b5e73ea5a395dee1a8013509c574ed55327bdfa8","unresolved":false,"context_lines":[{"line_number":103,"context_line":"default_v4_route_devs\u003d$(ip -4 route | grep ^default | awk \u0027{print $5}\u0027)"},{"line_number":104,"context_line":"die_if_not_set $LINENO default_v4_route_devs \"Failure retrieving default IPv4 route devices\""},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"default_v6_route_devs\u003d$(ip -6 route list match default table all | grep via | awk \u0027{print $5}\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"function _determine_config_l3 {"},{"line_number":109,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file $Q_L3_CONF_FILE\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3f79a3b5_8aa67154","line":106,"updated":"2018-10-08 03:14:02.000000000","message":"looking at the example in the bug, won\u0027t this list \"ens3\" twice?  perhaps we should \"| sort | uniq\" this to filter it out?\n\nhowever, OTOH, this only seems to be used to set an idempotent sysctl which means it doesn\u0027t matter.","commit_id":"866efef17af8f0d04240bf05714cb12452185822"},{"author":{"_account_id":19307,"name":"Antonio Ojea","email":"antonio.ojea.garcia@gmail.com","username":"itsuugo"},"change_message_id":"631b903eed517a963713d759be5061d5da7eb2c8","unresolved":false,"context_lines":[{"line_number":103,"context_line":"default_v4_route_devs\u003d$(ip -4 route | grep ^default | awk \u0027{print $5}\u0027)"},{"line_number":104,"context_line":"die_if_not_set $LINENO default_v4_route_devs \"Failure retrieving default IPv4 route devices\""},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"default_v6_route_devs\u003d$(ip -6 route list match default table all | grep via | awk \u0027{print $5}\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"function _determine_config_l3 {"},{"line_number":109,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file $Q_L3_CONF_FILE\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3f79a3b5_be504b61","line":106,"in_reply_to":"3f79a3b5_8aa67154","updated":"2018-10-08 07:52:47.000000000","message":"yeah, I came to the same conclusion. I was looking how to remove duplicates in a string and seems that is overcomplicating too much the solution https://stackoverflow.com/a/13648438/7794348. On the other hand, as you point out, is not a big deal to set the sysctl several times :)","commit_id":"866efef17af8f0d04240bf05714cb12452185822"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"94b66d331865614ca6690a84df93771596d7a8f3","unresolved":false,"context_lines":[{"line_number":103,"context_line":"default_v4_route_devs\u003d$(ip -4 route | grep ^default | awk \u0027{print $5}\u0027)"},{"line_number":104,"context_line":"die_if_not_set $LINENO default_v4_route_devs \"Failure retrieving default IPv4 route devices\""},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"default_v6_route_devs\u003d$(ip -6 route list match default table all | grep via | awk \u0027{print $5}\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"function _determine_config_l3 {"},{"line_number":109,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file $Q_L3_CONF_FILE\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3f79a3b5_eb7b55bc","line":106,"in_reply_to":"3f79a3b5_be504b61","updated":"2018-10-08 21:15:17.000000000","message":"re your link; this isn\u0027t an array so \"sort -u\" would do it, in this case, if we need it","commit_id":"866efef17af8f0d04240bf05714cb12452185822"}]}