)]}'
{"networking_odl/tests/unit/journal/test_dependency_validations.py":[{"author":{"_account_id":7921,"name":"Mike Kolesnik","email":"mkolesni@redhat.com","username":"mkolesni"},"change_message_id":"4bc22a657b38b45d6108d53b8f4fd33762ff7f40","unresolved":false,"context_lines":[{"line_number":33,"context_line":"_PORT_ID \u003d \u0027PORT_ID\u0027"},{"line_number":34,"context_line":"_PORT_DATA \u003d {\u0027network_id\u0027: _NET_ID,"},{"line_number":35,"context_line":"              \u0027fixed_ips\u0027: [{\u0027subnet_id\u0027: _SUBNET_ID}]}"},{"line_number":36,"context_line":"_PORT_DATA_DUPLICATE_SUBNET \u003d {"},{"line_number":37,"context_line":"    \u0027network_id\u0027: _NET_ID,"},{"line_number":38,"context_line":"    \u0027fixed_ips\u0027: [{\u0027subnet_id\u0027: _SUBNET_ID},"},{"line_number":39,"context_line":"                  {\u0027subnet_id\u0027: _SUBNET_ID}]"},{"line_number":40,"context_line":"}"},{"line_number":41,"context_line":"_ROUTER_ID \u003d \u0027ROUTER_ID\u0027"},{"line_number":42,"context_line":"_ROUTER_DATA \u003d {\u0027id\u0027: \u0027ROUTER_ID\u0027,"},{"line_number":43,"context_line":"                \u0027gw_port_id\u0027: \u0027GW_PORT_ID\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_23a38c51","line":40,"range":{"start_line":36,"start_character":0,"end_line":40,"end_character":1},"updated":"2018-01-15 09:24:35.000000000","message":"Wouldn\u0027t it be simpler that this was simply _PORT_DATA?\n\nOtherwise this introduces some unnecessary complexity IMHO","commit_id":"3a5d32f3f63206921cca3456ab91808eba3db598"},{"author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"change_message_id":"68424267f164c0e6501605a36fb24d3f54395dd0","unresolved":false,"context_lines":[{"line_number":33,"context_line":"_PORT_ID \u003d \u0027PORT_ID\u0027"},{"line_number":34,"context_line":"_PORT_DATA \u003d {\u0027network_id\u0027: _NET_ID,"},{"line_number":35,"context_line":"              \u0027fixed_ips\u0027: [{\u0027subnet_id\u0027: _SUBNET_ID}]}"},{"line_number":36,"context_line":"_PORT_DATA_DUPLICATE_SUBNET \u003d {"},{"line_number":37,"context_line":"    \u0027network_id\u0027: _NET_ID,"},{"line_number":38,"context_line":"    \u0027fixed_ips\u0027: [{\u0027subnet_id\u0027: _SUBNET_ID},"},{"line_number":39,"context_line":"                  {\u0027subnet_id\u0027: _SUBNET_ID}]"},{"line_number":40,"context_line":"}"},{"line_number":41,"context_line":"_ROUTER_ID \u003d \u0027ROUTER_ID\u0027"},{"line_number":42,"context_line":"_ROUTER_DATA \u003d {\u0027id\u0027: \u0027ROUTER_ID\u0027,"},{"line_number":43,"context_line":"                \u0027gw_port_id\u0027: \u0027GW_PORT_ID\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_eee26b01","line":40,"range":{"start_line":36,"start_character":0,"end_line":40,"end_character":1},"in_reply_to":"7f96bb07_23a38c51","updated":"2018-01-15 09:40:35.000000000","message":"I thought about it. I went with this route because by doing so we are testing for two different scenarios which is more comprehensive and declarative about the behavior we want.","commit_id":"3a5d32f3f63206921cca3456ab91808eba3db598"}]}