)]}'
{"devstack/plugin.sh":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"11809f3ab61fe55ed5e95b122dfed57b0c3488d7","unresolved":false,"context_lines":[{"line_number":1002,"context_line":"    if [[ \"$KURYR_K8S_CONTAINERIZED_DEPLOYMENT\" \u003d\u003d \"True\" ]]; then"},{"line_number":1003,"context_line":"        iniset $TEMPEST_CONFIG kuryr_kubernetes containerized True"},{"line_number":1004,"context_line":"    fi"},{"line_number":1005,"context_line":"    if [[ \"$KURYR_SG_DRIVER\" \u003d\u003d \"namespace\" ]] \u0026\u0026 [[ \"$KURYR_SUBNET_DRIVER\" \u003d\u003d \"namespace\" ]]; then"},{"line_number":1006,"context_line":"        iniset $TEMPEST_CONFIG kuryr_kubernetes namespace_enabled True"},{"line_number":1007,"context_line":"    elif [[ \"$KURYR_SG_DRIVER\" \u003d\u003d \"policy\" ]]; then"},{"line_number":1008,"context_line":"        iniset $TEMPEST_CONFIG kuryr_kubernetes namespace_enabled True"},{"line_number":1009,"context_line":"    fi"},{"line_number":1010,"context_line":"    if [[ \"$KURYR_K8S_SERIAL_TESTS\" \u003d\u003d \"True\" ]]; then"},{"line_number":1011,"context_line":"        iniset $TEMPEST_CONFIG kuryr_kubernetes run_tests_serial True"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3fa7e38b_3c014c06","line":1008,"range":{"start_line":1005,"start_character":0,"end_line":1008,"end_character":70},"updated":"2019-09-25 06:48:46.000000000","message":"I think this should be changed to just one if, checking if KURYR_SUBNET_DRIVER \u003d\u003d namespace, and then set the var to true. Also, perhaps work to change the var name. Instead of \u0027namespace_enabled\u0027 to \u0027subnet_per_namespace\u0027 to make it more clear","commit_id":"b1e84fb3154762e1b5d8e98e2f7f6a9dc538e5f4"}]}