)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"078970ad455562596c4b8005108258cdbf6d76ae","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Omer Anson \u003comer.anson@toganetworks.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-07 09:45:38 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Prepare (refactor) trunk for MACVLAN/IPVLAN support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I4350e618bf047e95538207071c649ba7f49f33aa"},{"line_number":10,"context_line":"Related-Bug: #1720734"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_b2f6aed6","line":7,"updated":"2017-12-09 21:33:28.000000000","message":"nit: maybe small explanation about why mac-vald need diffrent id , and why port status check is different then normal trunk.","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"a7779341439169e75299895d8ca9b7537dede1e1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Omer Anson \u003comer.anson@toganetworks.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-07 09:45:38 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Prepare (refactor) trunk for MACVLAN/IPVLAN support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I4350e618bf047e95538207071c649ba7f49f33aa"},{"line_number":10,"context_line":"Related-Bug: #1720734"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df87a7cf_838c7ad7","line":7,"in_reply_to":"df87a7cf_b2f6aed6","updated":"2017-12-10 10:17:23.000000000","message":"If I respin, will do.","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"}],"dragonflow/neutron/services/trunk/driver.py":[{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"078970ad455562596c4b8005108258cdbf6d76ae","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from dragonflow.neutron.services import mixins"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def get_child_port_status(parent_port):"},{"line_number":32,"context_line":"    new_status \u003d n_constants.PORT_STATUS_ACTIVE"},{"line_number":33,"context_line":"    if parent_port[\u0027status\u0027] !\u003d n_constants.PORT_STATUS_ACTIVE:"},{"line_number":34,"context_line":"        new_status \u003d n_constants.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_1208bad7","line":31,"updated":"2017-12-09 21:33:28.000000000","message":"nit : maybe _get_child_port_status as it used only in the driver?","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"},{"author":{"_account_id":17880,"name":"Shachar Snapiri","email":"shachar.snapiri@toganetworks.com","username":"snapiri"},"change_message_id":"334597af732214ae788d8f0ff3a8a68dac96730e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from dragonflow.neutron.services import mixins"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def get_child_port_status(parent_port):"},{"line_number":32,"context_line":"    new_status \u003d n_constants.PORT_STATUS_ACTIVE"},{"line_number":33,"context_line":"    if parent_port[\u0027status\u0027] !\u003d n_constants.PORT_STATUS_ACTIVE:"},{"line_number":34,"context_line":"        new_status \u003d n_constants.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_c301223a","line":31,"in_reply_to":"df87a7cf_1208bad7","updated":"2017-12-10 10:12:06.000000000","message":"+1","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"},{"author":{"_account_id":17880,"name":"Shachar Snapiri","email":"shachar.snapiri@toganetworks.com","username":"snapiri"},"change_message_id":"bd7d24b7832b1536a860c9780ad289bf61fc2e7b","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from dragonflow.neutron.services import mixins"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def get_child_port_status(parent_port):"},{"line_number":32,"context_line":"    new_status \u003d n_constants.PORT_STATUS_ACTIVE"},{"line_number":33,"context_line":"    if parent_port[\u0027status\u0027] !\u003d n_constants.PORT_STATUS_ACTIVE:"},{"line_number":34,"context_line":"        new_status \u003d n_constants.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_5eef29e8","line":31,"in_reply_to":"df87a7cf_639136f1","updated":"2017-12-10 12:03:15.000000000","message":"OK","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"a7779341439169e75299895d8ca9b7537dede1e1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from dragonflow.neutron.services import mixins"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"def get_child_port_status(parent_port):"},{"line_number":32,"context_line":"    new_status \u003d n_constants.PORT_STATUS_ACTIVE"},{"line_number":33,"context_line":"    if parent_port[\u0027status\u0027] !\u003d n_constants.PORT_STATUS_ACTIVE:"},{"line_number":34,"context_line":"        new_status \u003d n_constants.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_639136f1","line":31,"in_reply_to":"df87a7cf_c301223a","updated":"2017-12-10 10:17:23.000000000","message":"Actually, the error is in the followup patch, which should use this function rather than a copy of it.","commit_id":"8e68534427fe8578104ad2064838fb5e109b9f60"}]}