)]}'
{"magnum/common/octavia.py":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"a5615a6471c705782426c8fec182654fc5d6043d","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    The load balancers created for kubernetes services and ingresses are"},{"line_number":63,"context_line":"    deleted."},{"line_number":64,"context_line":"    \"\"\""},{"line_number":65,"context_line":"    pattern \u003d (r\u0027Kubernetes .+ from cluster %s\u0027 % cluster.uuid)"},{"line_number":66,"context_line":"    valid_status \u003d [\"ACTIVE\", \"ERROR\", \"PENDING_DELETE\", \"DELETED\"]"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_f07b3777","line":65,"updated":"2020-02-04 02:21:17.000000000","message":"Hi Lingxian, why do we need this end of string/line char now? Is it because the description changed recently?","commit_id":"84803e3f0323ddbdc6e656fc7adb3cf000f7d595"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"83897e9f958555b238f826d98861d4613fdb5fae","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    The load balancers created for kubernetes services and ingresses are"},{"line_number":63,"context_line":"    deleted."},{"line_number":64,"context_line":"    \"\"\""},{"line_number":65,"context_line":"    pattern \u003d (r\u0027Kubernetes .+ from cluster %s\u0027 % cluster.uuid)"},{"line_number":66,"context_line":"    valid_status \u003d [\"ACTIVE\", \"ERROR\", \"PENDING_DELETE\", \"DELETED\"]"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_b0693f8e","line":65,"in_reply_to":"3fa7e38b_f07b3777","updated":"2020-02-04 02:35:05.000000000","message":"We DO NOT need it. Please see the example in the story https://storyboard.openstack.org/#!/story/2007254, there is resource version information in the load balancer description. No change happened recently. I think we only have considered the Service load balancer in the beginning when we added support to remove load balancers from Octavia before deleting cluster.","commit_id":"84803e3f0323ddbdc6e656fc7adb3cf000f7d595"}]}