)]}'
{"neutron/templates/bin/_neutron-linuxbridge-agent-init.sh.tpl":[{"author":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"change_message_id":"a8521a4606f1e8e62ac67c5207ae9e0dc1932569","unresolved":true,"context_lines":[{"line_number":46,"context_line":"fi"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# determine local-ip dynamically based on interface provided but only if tunnel_types is not null"},{"line_number":49,"context_line":"LOCAL_IP\u003d$(ip a s $tunnel_interface | grep \u0027inet \u0027 | grep -v 127.0.0.1 | awk \u0027{print $2}\u0027 | awk -F \"/\" \u0027{print $1}\u0027)"},{"line_number":50,"context_line":"if [ -z \"${LOCAL_IP}\" ] ; then"},{"line_number":51,"context_line":"  echo \"Var LOCAL_IP is empty\""},{"line_number":52,"context_line":"  exit 1"}],"source_content_type":"text/x-smarty","patch_set":4,"id":"3d68e4d4_de3296c9","line":49,"range":{"start_line":49,"start_character":0,"end_line":49,"end_character":116},"updated":"2021-08-16 16:33:29.000000000","message":"Does it require a similar change to OVS agent as well?","commit_id":"7972cfd44b6042ad5cd530fee60683a105e01304"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"d0c48730a83d63f01b0259141558cf06a7404071","unresolved":true,"context_lines":[{"line_number":46,"context_line":"fi"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# determine local-ip dynamically based on interface provided but only if tunnel_types is not null"},{"line_number":49,"context_line":"LOCAL_IP\u003d$(ip a s $tunnel_interface | grep \u0027inet \u0027 | grep -v 127.0.0.1 | awk \u0027{print $2}\u0027 | awk -F \"/\" \u0027{print $1}\u0027)"},{"line_number":50,"context_line":"if [ -z \"${LOCAL_IP}\" ] ; then"},{"line_number":51,"context_line":"  echo \"Var LOCAL_IP is empty\""},{"line_number":52,"context_line":"  exit 1"}],"source_content_type":"text/x-smarty","patch_set":4,"id":"843c4e93_9aab7a2d","line":49,"range":{"start_line":49,"start_character":3,"end_line":49,"end_character":116},"updated":"2021-08-20 15:16:11.000000000","message":"ip -json | jq ... \n\n?","commit_id":"7972cfd44b6042ad5cd530fee60683a105e01304"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"7e7deb226b42d302d0212b51ee5a0f4c82be12aa","unresolved":true,"context_lines":[{"line_number":46,"context_line":"fi"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# determine local-ip dynamically based on interface provided but only if tunnel_types is not null"},{"line_number":49,"context_line":"LOCAL_IP\u003d$(ip a s $tunnel_interface | grep \u0027inet \u0027 | grep -v 127.0.0.1 | awk \u0027{print $2}\u0027 | awk -F \"/\" \u0027{print $1}\u0027)"},{"line_number":50,"context_line":"if [ -z \"${LOCAL_IP}\" ] ; then"},{"line_number":51,"context_line":"  echo \"Var LOCAL_IP is empty\""},{"line_number":52,"context_line":"  exit 1"}],"source_content_type":"text/x-smarty","patch_set":4,"id":"9dc8fcd7_40f9afd0","line":49,"range":{"start_line":49,"start_character":0,"end_line":49,"end_character":116},"in_reply_to":"3d68e4d4_de3296c9","updated":"2021-08-16 16:48:11.000000000","message":"I am not familiar with the Open vSwitch agent (as Open vSwitch is not compatible with FRR based EVPN yet). \n\nI could add it to https://github.com/openstack/openstack-helm/blob/master/neutron/templates/bin/_neutron-openvswitch-agent-init.sh.tpl#L68 as that function is used here https://github.com/openstack/openstack-helm/blob/master/neutron/templates/bin/_neutron-openvswitch-agent-init.sh.tpl#L430 but I am not sure if it would bring unexpected consequences for other uses of that function.","commit_id":"7972cfd44b6042ad5cd530fee60683a105e01304"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"9e54893e706e21451ecf7f107e1b48b13738804b","unresolved":true,"context_lines":[{"line_number":46,"context_line":"fi"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# determine local-ip dynamically based on interface provided but only if tunnel_types is not null"},{"line_number":49,"context_line":"LOCAL_IP\u003d$(ip a s $tunnel_interface | grep \u0027inet \u0027 | grep -v 127.0.0.1 | awk \u0027{print $2}\u0027 | awk -F \"/\" \u0027{print $1}\u0027)"},{"line_number":50,"context_line":"if [ -z \"${LOCAL_IP}\" ] ; then"},{"line_number":51,"context_line":"  echo \"Var LOCAL_IP is empty\""},{"line_number":52,"context_line":"  exit 1"}],"source_content_type":"text/x-smarty","patch_set":4,"id":"b9e54beb_7e5df5ce","line":49,"range":{"start_line":49,"start_character":3,"end_line":49,"end_character":116},"in_reply_to":"843c4e93_9aab7a2d","updated":"2021-08-25 08:34:05.000000000","message":"What about something like `ip -json a s lo | jq -r \u0027.[0].addr_info | .[] | select(.family \u003d\u003d \"inet\" and .local !\u003d \"127.0.0.1\") | .local\u0027`, though it won\u0027t me much shorter","commit_id":"7972cfd44b6042ad5cd530fee60683a105e01304"}]}