)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"change_message_id":"5587daa03221663af77118369b46b51ad2d4c1a1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove List events API from Cisco N1kv Neutron"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Earlier Cisco was using list events api to poll policies from VSM."},{"line_number":10,"context_line":"It was inefficient and caused delay in processing."},{"line_number":11,"context_line":"So, now Cisco switched to list profiles to poll policies from VSM."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2FxPE%3D","line":9,"updated":"2014-03-13 22:45:02.000000000","message":"It should be Cisco N1kv plugin and not just Cisco.","commit_id":"a766fa37fad093590dae69563c535185745f6bf7"},{"author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"change_message_id":"5587daa03221663af77118369b46b51ad2d4c1a1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Earlier Cisco was using list events api to poll policies from VSM."},{"line_number":10,"context_line":"It was inefficient and caused delay in processing."},{"line_number":11,"context_line":"So, now Cisco switched to list profiles to poll policies from VSM."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia734735361dc3eaee8e276ada0c80045eaa9ef96"},{"line_number":14,"context_line":"Closes-Bug: #1292173"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2FxPA%3D","line":11,"updated":"2014-03-13 22:45:02.000000000","message":"Ditto","commit_id":"a766fa37fad093590dae69563c535185745f6bf7"}],"neutron/plugins/cisco/db/n1kv_db_v2.py":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"71df1bb9931a14aee46f2a339f6420a027d94467","unresolved":false,"context_lines":[{"line_number":939,"context_line":"        raise c_exc.PolicyProfileIdNotFound(profile_id\u003did)"},{"line_number":940,"context_line":""},{"line_number":941,"context_line":""},{"line_number":942,"context_line":"def _get_policy_profiles():"},{"line_number":943,"context_line":"    \"\"\"Retrieve all policy profiles.\"\"\""},{"line_number":944,"context_line":"    db_session \u003d db.get_session()"},{"line_number":945,"context_line":"    with db_session.begin(subtransactions\u003dTrue):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2FwSc%3D","line":942,"updated":"2014-03-14 01:47:34.000000000","message":"Nit: In Python convention, starting a function name with underscore is an \"internal use\" indicator. Since you are using it from outside, please remove the leading underscore.","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"69d6213df7f43e1d7cb1205e4d2feb5d2d039fc7","unresolved":false,"context_lines":[{"line_number":939,"context_line":"        raise c_exc.PolicyProfileIdNotFound(profile_id\u003did)"},{"line_number":940,"context_line":""},{"line_number":941,"context_line":""},{"line_number":942,"context_line":"def _get_policy_profiles():"},{"line_number":943,"context_line":"    \"\"\"Retrieve all policy profiles.\"\"\""},{"line_number":944,"context_line":"    db_session \u003d db.get_session()"},{"line_number":945,"context_line":"    with db_session.begin(subtransactions\u003dTrue):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9qHE%3D","line":942,"in_reply_to":"AAAAWH%2F%2FwSc%3D","updated":"2014-04-03 01:44:15.000000000","message":"Done","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"}],"neutron/plugins/cisco/n1kv/n1kv_neutron_plugin.py":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"71df1bb9931a14aee46f2a339f6420a027d94467","unresolved":false,"context_lines":[{"line_number":168,"context_line":"            # Fetch policy profiles from VSM"},{"line_number":169,"context_line":"            if policy_profiles:"},{"line_number":170,"context_line":"                for profile in policy_profiles[\u0027body\u0027][c_const.SET]:"},{"line_number":171,"context_line":"                    if c_const.ID and c_const.NAME in profile:"},{"line_number":172,"context_line":"                        \u0027vsm_profiles[profile[c_const.PROPERTIES][c_const.ID]]\u0027"},{"line_number":173,"context_line":"                        \u0027\u003d profile[c_const.PROPERTIES][c_const.NAME]\u0027"},{"line_number":174,"context_line":"                # Fetch policy profiles previously populated"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2FwKs%3D","line":171,"updated":"2014-03-14 01:47:34.000000000","message":"This if condition doesn\u0027t look right. See below.","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"69d6213df7f43e1d7cb1205e4d2feb5d2d039fc7","unresolved":false,"context_lines":[{"line_number":168,"context_line":"            # Fetch policy profiles from VSM"},{"line_number":169,"context_line":"            if policy_profiles:"},{"line_number":170,"context_line":"                for profile in policy_profiles[\u0027body\u0027][c_const.SET]:"},{"line_number":171,"context_line":"                    if c_const.ID and c_const.NAME in profile:"},{"line_number":172,"context_line":"                        \u0027vsm_profiles[profile[c_const.PROPERTIES][c_const.ID]]\u0027"},{"line_number":173,"context_line":"                        \u0027\u003d profile[c_const.PROPERTIES][c_const.NAME]\u0027"},{"line_number":174,"context_line":"                # Fetch policy profiles previously populated"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9qG8%3D","line":171,"in_reply_to":"AAAAWH%2F%2FwKs%3D","updated":"2014-04-03 01:44:15.000000000","message":"Done","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"71df1bb9931a14aee46f2a339f6420a027d94467","unresolved":false,"context_lines":[{"line_number":170,"context_line":"                for profile in policy_profiles[\u0027body\u0027][c_const.SET]:"},{"line_number":171,"context_line":"                    if c_const.ID and c_const.NAME in profile:"},{"line_number":172,"context_line":"                        \u0027vsm_profiles[profile[c_const.PROPERTIES][c_const.ID]]\u0027"},{"line_number":173,"context_line":"                        \u0027\u003d profile[c_const.PROPERTIES][c_const.NAME]\u0027"},{"line_number":174,"context_line":"                # Fetch policy profiles previously populated"},{"line_number":175,"context_line":"                for profile in n1kv_db_v2._get_policy_profiles():"},{"line_number":176,"context_line":"                    plugin_profiles[profile.id] \u003d profile.name"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2FwP0%3D","line":173,"updated":"2014-03-14 01:47:34.000000000","message":"Did you intend to use brackets to break the above into two lines? Because putting quotes around it just makes it a string and nothing is executed.\n\nI think that lines 171-172 should look something like this:\n\n    prof_id \u003d getattr(profile[c_const.PROPERTIES], c_const.ID)\n    name \u003d getattr(profile[c_const.PROPERTIES], c_const.NAME)\n    if prof_id and name:\n        vsm_profiles[prof_id] \u003d name","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"69d6213df7f43e1d7cb1205e4d2feb5d2d039fc7","unresolved":false,"context_lines":[{"line_number":170,"context_line":"                for profile in policy_profiles[\u0027body\u0027][c_const.SET]:"},{"line_number":171,"context_line":"                    if c_const.ID and c_const.NAME in profile:"},{"line_number":172,"context_line":"                        \u0027vsm_profiles[profile[c_const.PROPERTIES][c_const.ID]]\u0027"},{"line_number":173,"context_line":"                        \u0027\u003d profile[c_const.PROPERTIES][c_const.NAME]\u0027"},{"line_number":174,"context_line":"                # Fetch policy profiles previously populated"},{"line_number":175,"context_line":"                for profile in n1kv_db_v2._get_policy_profiles():"},{"line_number":176,"context_line":"                    plugin_profiles[profile.id] \u003d profile.name"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9qGk%3D","line":173,"in_reply_to":"AAAAWH%2F%2FwP0%3D","updated":"2014-04-03 01:44:15.000000000","message":"Since profile is a dict, getattr wont work. \nI have changed it to \nprofile[c_const.PROPERTIES].get(key, None) instead","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"change_message_id":"ddce901614b0a9b632e95b15a5ef6ecb047354b5","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                cisco_exceptions.VSMConnectionFailed):"},{"line_number":194,"context_line":"            LOG.warning(_(\u0027No policy profile populated from VSM\u0027))"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    def _poll_policies(self, event_type\u003dNone, epoch\u003dNone, tenant_id\u003dNone):"},{"line_number":197,"context_line":"        \"\"\""},{"line_number":198,"context_line":"        Poll for Policy Profiles from Cisco Nexus1000V for any update/delete."},{"line_number":199,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FREA%3D","line":196,"updated":"2014-04-22 21:53:56.000000000","message":"This method is no longer used. Please remove\nAlso list_events from the client file is not used anymore.","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"30ddf9117b7713033b48b585a0f08db7b9433201","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                cisco_exceptions.VSMConnectionFailed):"},{"line_number":194,"context_line":"            LOG.warning(_(\u0027No policy profile populated from VSM\u0027))"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    def _poll_policies(self, event_type\u003dNone, epoch\u003dNone, tenant_id\u003dNone):"},{"line_number":197,"context_line":"        \"\"\""},{"line_number":198,"context_line":"        Poll for Policy Profiles from Cisco Nexus1000V for any update/delete."},{"line_number":199,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FQ1U%3D","line":196,"in_reply_to":"AAAAXH%2F%2FREA%3D","updated":"2014-04-22 22:39:37.000000000","message":"Done","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"}],"neutron/tests/unit/cisco/n1kv/fake_client.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"881e5589f3713fd4176072551ddebfa64283affe","unresolved":false,"context_lines":[{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""},{"line_number":96,"context_line":"    for num in range(1, total_profiles + 1):"},{"line_number":97,"context_line":"        xml_body \u003d xml_body + \"\"\""},{"line_number":98,"context_line":"                   \u003cinstance name\u003d\"%s\" url\u003d\"/api/n1k/virtual-port-profile/%s\"\u003e"},{"line_number":99,"context_line":"                   \u003cproperties\u003e"},{"line_number":100,"context_line":"                   \u003cid\u003e00000000-0000-0000-0000-00000000000%s\u003c/id\u003e"},{"line_number":101,"context_line":"                   \u003cname\u003epp-%s\u003c/name\u003e"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F9UyQ%3D","line":98,"updated":"2014-04-07 03:20:01.000000000","message":"replace all%s with %(num)s","commit_id":"cb400f5bc083ef0d17df6860531f7333ee8e7b4a"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"eab19e2a005f171da90d3706af8c40e9d4a1a450","unresolved":false,"context_lines":[{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""},{"line_number":96,"context_line":"    for num in range(1, total_profiles + 1):"},{"line_number":97,"context_line":"        xml_body \u003d xml_body + \"\"\""},{"line_number":98,"context_line":"                   \u003cinstance name\u003d\"%s\" url\u003d\"/api/n1k/virtual-port-profile/%s\"\u003e"},{"line_number":99,"context_line":"                   \u003cproperties\u003e"},{"line_number":100,"context_line":"                   \u003cid\u003e00000000-0000-0000-0000-00000000000%s\u003c/id\u003e"},{"line_number":101,"context_line":"                   \u003cname\u003epp-%s\u003c/name\u003e"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F9OMY%3D","line":98,"in_reply_to":"AAAAWH%2F9UyQ%3D","updated":"2014-04-07 21:00:51.000000000","message":"Done","commit_id":"cb400f5bc083ef0d17df6860531f7333ee8e7b4a"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"881e5589f3713fd4176072551ddebfa64283affe","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                   \u003cname\u003epp-%s\u003c/name\u003e"},{"line_number":102,"context_line":"                   \u003c/properties\u003e"},{"line_number":103,"context_line":"                   \u003c/instance\u003e"},{"line_number":104,"context_line":"                   \"\"\" % (tuple([num] * 4))"},{"line_number":105,"context_line":"    return \"%s%s%s\" % (xml_prefix, xml_body, xml_suffix)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F9UyE%3D","line":104,"updated":"2014-04-07 03:20:01.000000000","message":"\"\"\" % {\u0027num\": num}","commit_id":"cb400f5bc083ef0d17df6860531f7333ee8e7b4a"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"eab19e2a005f171da90d3706af8c40e9d4a1a450","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                   \u003cname\u003epp-%s\u003c/name\u003e"},{"line_number":102,"context_line":"                   \u003c/properties\u003e"},{"line_number":103,"context_line":"                   \u003c/instance\u003e"},{"line_number":104,"context_line":"                   \"\"\" % (tuple([num] * 4))"},{"line_number":105,"context_line":"    return \"%s%s%s\" % (xml_prefix, xml_body, xml_suffix)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F9OMU%3D","line":104,"in_reply_to":"AAAAWH%2F9UyE%3D","updated":"2014-04-07 21:00:51.000000000","message":"Done","commit_id":"cb400f5bc083ef0d17df6860531f7333ee8e7b4a"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"db985345940a3b0e56eb9629f4e2358032a0178d","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    :param total_profiles: integer representing total number of profiles to"},{"line_number":90,"context_line":"                           return"},{"line_number":91,"context_line":"    \"\"\""},{"line_number":92,"context_line":"    xml_prefix \u003d \"\"\"\u003c?xml version\u003d\"1.0\" encoding\u003d\"utf-8\"?\u003e"},{"line_number":93,"context_line":"                    \u003cset name\u003d\"virtual_port_profile_set\"\u003e\"\"\""},{"line_number":94,"context_line":"    xml_body \u003d \"\""},{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FR6s%3D","line":92,"updated":"2014-04-22 19:32:37.000000000","message":"xml \u003d [\u003cprefix string\u003e]","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"30ddf9117b7713033b48b585a0f08db7b9433201","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    :param total_profiles: integer representing total number of profiles to"},{"line_number":90,"context_line":"                           return"},{"line_number":91,"context_line":"    \"\"\""},{"line_number":92,"context_line":"    xml_prefix \u003d \"\"\"\u003c?xml version\u003d\"1.0\" encoding\u003d\"utf-8\"?\u003e"},{"line_number":93,"context_line":"                    \u003cset name\u003d\"virtual_port_profile_set\"\u003e\"\"\""},{"line_number":94,"context_line":"    xml_body \u003d \"\""},{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FQ1A%3D","line":92,"in_reply_to":"AAAAXH%2F%2FR6s%3D","updated":"2014-04-22 22:39:37.000000000","message":"Done","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"db985345940a3b0e56eb9629f4e2358032a0178d","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    xml_body \u003d \"\""},{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""},{"line_number":96,"context_line":"    for num in range(1, total_profiles + 1):"},{"line_number":97,"context_line":"        xml_body \u003d xml_body + \"\"\""},{"line_number":98,"context_line":"                   \u003cinstance name\u003d\"%(num)s\""},{"line_number":99,"context_line":"                   url\u003d\"/api/n1k/virtual-port-profile/%(num)s\"\u003e"},{"line_number":100,"context_line":"                   \u003cproperties\u003e"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FR6g%3D","line":97,"updated":"2014-04-22 19:32:37.000000000","message":"template \u003d (\n      \u0027\u003cinstance name\u003d\"%(num)d\"\u0027\n      \u0027 url\u003d\"/api/n1k/virtual-port-profile/%(num)s\"\u003e\u0027\n      \u0027\u003cproperties\u003e\u0027\n      \u0027\u003cid\u003e00000000-0000-0000-0000-00000000000%(num)s\u003c/id\u003e\u0027\n      \u0027\u003cname\u003epp-%(num)s\u003c/name\u003e\u0027\n      \u0027\u003c/properties\u003e\u0027\n      \u0027\u003c/instance\u003e\u0027\n)\n\n    \nxml.extend(template % {\u0027num\u0027: n} for n in range(1, total_profiles+1))","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"30ddf9117b7713033b48b585a0f08db7b9433201","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    xml_body \u003d \"\""},{"line_number":95,"context_line":"    xml_suffix \u003d \"\u003c/set\u003e\""},{"line_number":96,"context_line":"    for num in range(1, total_profiles + 1):"},{"line_number":97,"context_line":"        xml_body \u003d xml_body + \"\"\""},{"line_number":98,"context_line":"                   \u003cinstance name\u003d\"%(num)s\""},{"line_number":99,"context_line":"                   url\u003d\"/api/n1k/virtual-port-profile/%(num)s\"\u003e"},{"line_number":100,"context_line":"                   \u003cproperties\u003e"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FQ08%3D","line":97,"in_reply_to":"AAAAXH%2F%2FR6g%3D","updated":"2014-04-22 22:39:37.000000000","message":"Done","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"db985345940a3b0e56eb9629f4e2358032a0178d","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                   \u003c/properties\u003e"},{"line_number":104,"context_line":"                   \u003c/instance\u003e"},{"line_number":105,"context_line":"                   \"\"\" % {\u0027num\u0027: num}"},{"line_number":106,"context_line":"    return \"%s%s%s\" % (xml_prefix, xml_body, xml_suffix)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FR6U%3D","line":106,"updated":"2014-04-22 19:32:37.000000000","message":"xml.append(\"\u003c/set\u003e\")\nreturn \u0027\u0027.join(xml)","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"30ddf9117b7713033b48b585a0f08db7b9433201","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                   \u003c/properties\u003e"},{"line_number":104,"context_line":"                   \u003c/instance\u003e"},{"line_number":105,"context_line":"                   \"\"\" % {\u0027num\u0027: num}"},{"line_number":106,"context_line":"    return \"%s%s%s\" % (xml_prefix, xml_body, xml_suffix)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FQ04%3D","line":106,"in_reply_to":"AAAAXH%2F%2FR6U%3D","updated":"2014-04-22 22:39:37.000000000","message":"Done","commit_id":"6cb4750305ce84cdcc90ddc46d0d0eccfcdd55f9"}],"neutron/tests/unit/cisco/n1kv/test_n1kv_plugin.py":[{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"change_message_id":"b222f25d10c0244cb66724dde2ae54487263fa6d","unresolved":false,"context_lines":[{"line_number":60,"context_line":"def _fake_setup_vsm(self):"},{"line_number":61,"context_line":"    \"\"\"Fake establish Communication with Cisco Nexus1000V VSM.\"\"\""},{"line_number":62,"context_line":"    self.agent_vsm \u003d True"},{"line_number":63,"context_line":"    self._populate_policy_profiles()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"class NetworkProfileTestExtensionManager(object):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2FwuE%3D","line":63,"updated":"2014-03-14 00:07:33.000000000","message":"This change result in testing all the conditional statements you added to _populate_policy_profiles()?","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"},{"author":{"_account_id":8940,"name":"Sourabh Patwardhan","email":"sopatwar@gmail.com","username":"sopatwar"},"change_message_id":"69d6213df7f43e1d7cb1205e4d2feb5d2d039fc7","unresolved":false,"context_lines":[{"line_number":60,"context_line":"def _fake_setup_vsm(self):"},{"line_number":61,"context_line":"    \"\"\"Fake establish Communication with Cisco Nexus1000V VSM.\"\"\""},{"line_number":62,"context_line":"    self.agent_vsm \u003d True"},{"line_number":63,"context_line":"    self._populate_policy_profiles()"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"class NetworkProfileTestExtensionManager(object):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9qGc%3D","line":63,"in_reply_to":"AAAAWH%2F%2FwuE%3D","updated":"2014-04-03 01:44:15.000000000","message":"Added unit test cases.","commit_id":"4beddf7754ec88f6eaf6ec15fb269cb5d5b0923d"}]}