)]}'
{"doc/source/plugins/neutron.rst":[{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"ab6eb45c4adb9372ec29ecb4c886eaa1ec357073","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"*****************"},{"line_number":17,"context_line":"Neutron Plugin Guide"},{"line_number":18,"context_line":"*****************"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_e2b494f3","line":16,"updated":"2016-02-22 05:54:53.000000000","message":"not long enough here:)","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"dce948f002e1e6187aea466f15ddcadad6f0d33b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"*****************"},{"line_number":17,"context_line":"Neutron Plugin Guide"},{"line_number":18,"context_line":"*****************"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_e61c61c3","line":16,"in_reply_to":"bae84128_e2b494f3","updated":"2016-02-22 17:48:09.000000000","message":":D  Will fix","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"ab6eb45c4adb9372ec29ecb4c886eaa1ec357073","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":".. note::"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    Restart the Neutron service and api (q-svc and q-api) after making changes."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"local.conf (devstack)"},{"line_number":79,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_f426ebb8","line":76,"updated":"2016-02-22 05:54:53.000000000","message":"q-svc is the api service for neutron, so no q-api.","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"dce948f002e1e6187aea466f15ddcadad6f0d33b","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":".. note::"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    Restart the Neutron service and api (q-svc and q-api) after making changes."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"local.conf (devstack)"},{"line_number":79,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_6687b1d7","line":76,"in_reply_to":"bae84128_f426ebb8","updated":"2016-02-22 17:48:09.000000000","message":"Yep, thanks, will correct","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"4b3f8cb0ae5aaaf57db61421e02b07d0e7d38164","unresolved":false,"context_lines":[{"line_number":43,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":44,"context_line":"::"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    [resource_plugin:os_neutron_network]"},{"line_number":47,"context_line":"    enabled \u003d true"},{"line_number":48,"context_line":"    index_name \u003d searchlight"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9aed3d3a_19e83b7c","line":46,"updated":"2016-02-25 18:32:42.000000000","message":"Please add to here.","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"bcee9748dcce3082b71810e019741079dcd19679","unresolved":false,"context_lines":[{"line_number":43,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":44,"context_line":"::"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    [resource_plugin:os_neutron_network]"},{"line_number":47,"context_line":"    enabled \u003d true"},{"line_number":48,"context_line":"    index_name \u003d searchlight"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"9aed3d3a_347426ce","line":46,"in_reply_to":"9aed3d3a_19e83b7c","updated":"2016-02-25 18:49:47.000000000","message":"Add what? :)  Corrected this to os_neutron_net","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"},{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"4b3f8cb0ae5aaaf57db61421e02b07d0e7d38164","unresolved":false,"context_lines":[{"line_number":52,"context_line":"::"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    [resource_plugin:os_neutron_port]"},{"line_number":55,"context_line":"    enabled \u003d true"},{"line_number":56,"context_line":"    index_name \u003d searchlight"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Neutron Configuration"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9aed3d3a_59548353","line":55,"updated":"2016-02-25 18:32:42.000000000","message":"Does it matter if true is upper or lower case?  We have it upper case in other examples.","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"bcee9748dcce3082b71810e019741079dcd19679","unresolved":false,"context_lines":[{"line_number":52,"context_line":"::"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    [resource_plugin:os_neutron_port]"},{"line_number":55,"context_line":"    enabled \u003d true"},{"line_number":56,"context_line":"    index_name \u003d searchlight"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Neutron Configuration"}],"source_content_type":"text/x-rst","patch_set":5,"id":"9aed3d3a_34ad4632","line":55,"in_reply_to":"9aed3d3a_59548353","updated":"2016-02-25 18:49:47.000000000","message":"It\u0027s in lower case in the designate one; i believe oslo config treates true\u003d\u003dTrue\u003d\u003d1","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":"    c) Copyright 2016 Hewlett-Packard Enterprise Development Company, L.P."},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7af24918_9a9be565","line":2,"updated":"2016-03-03 21:28:08.000000000","message":"This should be \"(c)\" instead of \"c)\". The other *.rst files are incorrect also. Maybe we should submit a new patch changing all of the other files.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":"    c) Copyright 2016 Hewlett-Packard Enterprise Development Company, L.P."},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7af24918_b79b4f65","line":2,"in_reply_to":"7af24918_9a9be565","updated":"2016-03-03 21:48:10.000000000","message":"Separate patch","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":93,"context_line":"-----------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"All provider:* properties of networks are exposed to administrators only."},{"line_number":96,"context_line":"All binding:* properties of ports are also protected."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Additional properties can be protected similarly with the `admin_only_fields`"},{"line_number":99,"context_line":"under each plugin\u0027s configuration section. Glob-like patterns are supported."}],"source_content_type":"text/x-rst","patch_set":6,"id":"7af24918_837c19ed","line":96,"updated":"2016-03-03 21:28:08.000000000","message":"Is it clear that \"also protected\" means administrators only?","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    [resource_plugin:os_neutron_network]"},{"line_number":103,"context_line":"    admin_only_fields\u003dadmin_state_up,status"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"See: ADMIN_ONLY_PROPERTIES in:"},{"line_number":106,"context_line":"* searchlight/elasticsearch/plugins/neutron/networks.py"},{"line_number":107,"context_line":"* searchlight/elasticsearch/plugins/neutron/ports.py"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7af24918_4377b101","line":105,"updated":"2016-03-03 21:28:08.000000000","message":"Change \"ADMIN_ONLY_PROPERTIES\" to \"ADMIN_ONLY_FIELDS\".","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    [resource_plugin:os_neutron_network]"},{"line_number":103,"context_line":"    admin_only_fields\u003dadmin_state_up,status"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"See: ADMIN_ONLY_PROPERTIES in:"},{"line_number":106,"context_line":"* searchlight/elasticsearch/plugins/neutron/networks.py"},{"line_number":107,"context_line":"* searchlight/elasticsearch/plugins/neutron/ports.py"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7af24918_17ce8362","line":105,"in_reply_to":"7af24918_4377b101","updated":"2016-03-03 21:48:10.000000000","message":"done","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"releasenotes/notes/bp-neutron-plugin-5c14dd556457c1dc.yaml":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"4b3f8cb0ae5aaaf57db61421e02b07d0e7d38164","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    or updated_at). created_at is left empty; updated_at"},{"line_number":7,"context_line":"    is taken from the server (and will be adjusted to"},{"line_number":8,"context_line":"    use the notification timestamp at a later time)"},{"line_number":9,"context_line":"  - Routers and subnets to follow"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9aed3d3a_597763d4","line":9,"updated":"2016-02-25 18:32:42.000000000","message":"Don\u0027t think you need that issue note about routers and subnets in here.","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"bcee9748dcce3082b71810e019741079dcd19679","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    or updated_at). created_at is left empty; updated_at"},{"line_number":7,"context_line":"    is taken from the server (and will be adjusted to"},{"line_number":8,"context_line":"    use the notification timestamp at a later time)"},{"line_number":9,"context_line":"  - Routers and subnets to follow"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9aed3d3a_74550e1c","line":9,"in_reply_to":"9aed3d3a_597763d4","updated":"2016-02-25 18:49:47.000000000","message":"Done","commit_id":"dc88d157fdc117925789140e0a1210c1c97af7e0"}],"searchlight/elasticsearch/plugins/neutron/__init__.py":[{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"ab6eb45c4adb9372ec29ecb4c886eaa1ec357073","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import copy"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"from searchlight.elasticsearch.plugins import utils"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_22967c4c","line":1,"updated":"2016-02-22 05:54:53.000000000","message":"no copyright head?","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"dce948f002e1e6187aea466f15ddcadad6f0d33b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import copy"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"from searchlight.elasticsearch.plugins import utils"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_e673c113","line":1,"in_reply_to":"bae84128_22967c4c","updated":"2016-02-22 17:48:09.000000000","message":"Good catch","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"}],"searchlight/elasticsearch/plugins/neutron/networks.py":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"949040ff2d2bc8d93ce821c2e8d002ade94fa2c5","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def get_document_type(self):"},{"line_number":32,"context_line":"        return \u0027OS::Neutron::Network\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def get_mapping(self):"},{"line_number":35,"context_line":"        return {"}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_06786529","line":32,"updated":"2016-02-22 17:42:02.000000000","message":"Looks like it should be OS::Neutron::Net\n\nhttp://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Neutron::Net","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"dce948f002e1e6187aea466f15ddcadad6f0d33b","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def get_document_type(self):"},{"line_number":32,"context_line":"        return \u0027OS::Neutron::Network\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def get_mapping(self):"},{"line_number":35,"context_line":"        return {"}],"source_content_type":"text/x-python","patch_set":2,"id":"bae84128_46c8ad22","line":32,"in_reply_to":"bae84128_06786529","updated":"2016-02-22 17:48:09.000000000","message":"Thanks, good catch","commit_id":"388824ef4d2607e754dbbc7dd2cf9b2c641129d7"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from searchlight.elasticsearch.plugins import openstack_clients"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"LIST_LIMIT \u003d 100"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class NetworkIndex(base.IndexBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_013921d7","line":22,"updated":"2016-03-03 21:28:08.000000000","message":"Since neutron_client.list_networks() is a generator that handles paging we do not need LIST_LIMIT.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from searchlight.elasticsearch.plugins import openstack_clients"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"LIST_LIMIT \u003d 100"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class NetworkIndex(base.IndexBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_57dbeb1a","line":22,"in_reply_to":"7af24918_013921d7","updated":"2016-03-03 21:48:10.000000000","message":"Done","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def get_document_type(self):"},{"line_number":32,"context_line":"        return \u0027OS::Neutron::Net\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def get_mapping(self):"},{"line_number":35,"context_line":"        return {"},{"line_number":36,"context_line":"            \u0027dynamic\u0027: \u0027false\u0027,"},{"line_number":37,"context_line":"            \u0027properties\u0027: {"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_869ee7b6","line":34,"updated":"2016-03-03 21:28:08.000000000","message":"Reminder. This will need to be changed to MappingType when that patch gets merged.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def get_document_type(self):"},{"line_number":32,"context_line":"        return \u0027OS::Neutron::Net\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def get_mapping(self):"},{"line_number":35,"context_line":"        return {"},{"line_number":36,"context_line":"            \u0027dynamic\u0027: \u0027false\u0027,"},{"line_number":37,"context_line":"            \u0027properties\u0027: {"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_972e13f9","line":34,"in_reply_to":"7af24918_869ee7b6","updated":"2016-03-03 21:48:10.000000000","message":"Thanks. I\u0027ll modify that patch (doc_values) once this one goes in","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":8959,"name":"Lakshmi N Sampath","email":"lakshmi.sampath@hp.com","username":"lakshmi"},"change_message_id":"cafddb01cb762eaedb772c6aa2cd53fdf5396986","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        query. Document type will be added to this list."},{"line_number":77,"context_line":"        \"\"\""},{"line_number":78,"context_line":"        return ["},{"line_number":79,"context_line":"            {\u0027term\u0027: {\u0027tenant_id\u0027: request_context.owner}}"},{"line_number":80,"context_line":"        ]"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def get_objects(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_c06c5a0a","line":79,"range":{"start_line":79,"start_character":56,"end_line":79,"end_character":58},"updated":"2016-03-03 19:25:11.000000000","message":"RBAC should add term for allowing \"shared\" and \"external\" networks to be accessible to non-tenant members.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"d577c7e5bf020b4b591fc11e35bf707559d8b1b1","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        query. Document type will be added to this list."},{"line_number":77,"context_line":"        \"\"\""},{"line_number":78,"context_line":"        return ["},{"line_number":79,"context_line":"            {\u0027term\u0027: {\u0027tenant_id\u0027: request_context.owner}}"},{"line_number":80,"context_line":"        ]"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def get_objects(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_edc22e84","line":79,"range":{"start_line":79,"start_character":56,"end_line":79,"end_character":58},"in_reply_to":"7af24918_c06c5a0a","updated":"2016-03-03 19:50:10.000000000","message":"Good catch, done","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"searchlight/elasticsearch/plugins/neutron/notification_handlers.py":[{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        }"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def create_or_update(self, payload):"},{"line_number":41,"context_line":"        network_id \u003d payload[\u0027network\u0027][\u0027id\u0027]"},{"line_number":42,"context_line":"        LOG.debug(\"Updating network information for %s\", network_id)"},{"line_number":43,"context_line":"        # TODO(sjmc7): Set updated_at from the notification timestamp"},{"line_number":44,"context_line":"        network \u003d serialize_network(payload[\u0027network\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_0662d709","line":41,"updated":"2016-03-03 21:28:08.000000000","message":"network_id is only used for the LOG message. It is not used by the serialize method. Do we really need it?","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        }"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def create_or_update(self, payload):"},{"line_number":41,"context_line":"        network_id \u003d payload[\u0027network\u0027][\u0027id\u0027]"},{"line_number":42,"context_line":"        LOG.debug(\"Updating network information for %s\", network_id)"},{"line_number":43,"context_line":"        # TODO(sjmc7): Set updated_at from the notification timestamp"},{"line_number":44,"context_line":"        network \u003d serialize_network(payload[\u0027network\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_172a03db","line":41,"in_reply_to":"7af24918_0662d709","updated":"2016-03-03 21:48:10.000000000","message":"Not really but the same could apply elsewhere. It\u0027s useful to have the log message. If we decide otherwise it can be done wholesale over the other handlers as well.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":55,"context_line":"            LOG.error(_LE("},{"line_number":56,"context_line":"                \u0027Error deleting network %(network_id)s \u0027"},{"line_number":57,"context_line":"                \u0027from index: %(exc)s\u0027) %"},{"line_number":58,"context_line":"                {\u0027network_id\u0027: network_id, \u0027exc\u0027: exc})"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"class PortHandler(base.NotificationBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_26ed53b7","line":58,"updated":"2016-03-03 21:28:08.000000000","message":"exc is not the index, it is the exception message. We need to change the logging message from \"from index:\" to \"exception:\".","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":55,"context_line":"            LOG.error(_LE("},{"line_number":56,"context_line":"                \u0027Error deleting network %(network_id)s \u0027"},{"line_number":57,"context_line":"                \u0027from index: %(exc)s\u0027) %"},{"line_number":58,"context_line":"                {\u0027network_id\u0027: network_id, \u0027exc\u0027: exc})"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"class PortHandler(base.NotificationBase):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_d759bb8f","line":58,"in_reply_to":"7af24918_26ed53b7","updated":"2016-03-03 21:48:10.000000000","message":"This is a phrasing problem rather than the wrong variable. Will make it clearer","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        }"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    def create_or_update(self, payload):"},{"line_number":74,"context_line":"        port_id \u003d payload[\u0027port\u0027][\u0027id\u0027]"},{"line_number":75,"context_line":"        LOG.debug(\"Updating port information for %s\", port_id)"},{"line_number":76,"context_line":"        # TODO(sjmc7): Set updated_at from the notification timestamp"},{"line_number":77,"context_line":"        port \u003d serialize_port(payload[\u0027port\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_a6468378","line":74,"updated":"2016-03-03 21:28:08.000000000","message":"port_id is only used for the LOG message. It is not used by the serialize method. Do we really need it?","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        }"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    def create_or_update(self, payload):"},{"line_number":74,"context_line":"        port_id \u003d payload[\u0027port\u0027][\u0027id\u0027]"},{"line_number":75,"context_line":"        LOG.debug(\"Updating port information for %s\", port_id)"},{"line_number":76,"context_line":"        # TODO(sjmc7): Set updated_at from the notification timestamp"},{"line_number":77,"context_line":"        port \u003d serialize_port(payload[\u0027port\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_b7d34fd4","line":74,"in_reply_to":"7af24918_a6468378","updated":"2016-03-03 21:48:10.000000000","message":"as previous","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        except Exception as exc:"},{"line_number":86,"context_line":"            LOG.error(_LE("},{"line_number":87,"context_line":"                \u0027Error deleting port %(port_id)s \u0027"},{"line_number":88,"context_line":"                \u0027from index: %(exc)s\u0027) %"},{"line_number":89,"context_line":"                {\u0027port_id\u0027: port_id, \u0027exc\u0027: exc})"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_0671370f","line":88,"updated":"2016-03-03 21:28:08.000000000","message":"Same comment about exc.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        except Exception as exc:"},{"line_number":86,"context_line":"            LOG.error(_LE("},{"line_number":87,"context_line":"                \u0027Error deleting port %(port_id)s \u0027"},{"line_number":88,"context_line":"                \u0027from index: %(exc)s\u0027) %"},{"line_number":89,"context_line":"                {\u0027port_id\u0027: port_id, \u0027exc\u0027: exc})"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_77cd4730","line":88,"in_reply_to":"7af24918_0671370f","updated":"2016-03-03 21:48:10.000000000","message":"as previous","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"searchlight/elasticsearch/plugins/neutron/ports.py":[{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def get_mapping(self):"},{"line_number":39,"context_line":"        string_not_analyzed \u003d {\u0027type\u0027: \u0027string\u0027, \u0027index\u0027: \u0027not_analyzed\u0027}"},{"line_number":40,"context_line":"        ordered_string \u003d {\u0027type\u0027: \u0027string\u0027,"},{"line_number":41,"context_line":"                          \u0027fields\u0027: {\u0027raw\u0027: string_not_analyzed}}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        # TODO(sjmc7): denormalize \u0027device\u0027 fields (instance_id, router_id)?"},{"line_number":44,"context_line":"        return {"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_e1f23505","line":41,"updated":"2016-03-03 21:28:08.000000000","message":"Reminder. This will need to be changed to MappingType when that patch gets merged.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":8959,"name":"Lakshmi N Sampath","email":"lakshmi.sampath@hp.com","username":"lakshmi"},"change_message_id":"cafddb01cb762eaedb772c6aa2cd53fdf5396986","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    def _get_rbac_field_filters(self, request_context):"},{"line_number":77,"context_line":"        return ["},{"line_number":78,"context_line":"            {\u0027term\u0027: {\u0027tenant_id\u0027: request_context.owner}}"},{"line_number":79,"context_line":"        ]"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    def get_objects(self):"},{"line_number":82,"context_line":"        neutron_client \u003d openstack_clients.get_neutronclient()"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_20537e3a","line":79,"range":{"start_line":79,"start_character":8,"end_line":79,"end_character":9},"updated":"2016-03-03 19:25:11.000000000","message":"Similar to comment on networks, the child port plugin should have has_parent filter added to show ports from external and shared networks","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"d577c7e5bf020b4b591fc11e35bf707559d8b1b1","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    def _get_rbac_field_filters(self, request_context):"},{"line_number":77,"context_line":"        return ["},{"line_number":78,"context_line":"            {\u0027term\u0027: {\u0027tenant_id\u0027: request_context.owner}}"},{"line_number":79,"context_line":"        ]"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    def get_objects(self):"},{"line_number":82,"context_line":"        neutron_client \u003d openstack_clients.get_neutronclient()"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_6d645eb8","line":79,"range":{"start_line":79,"start_character":8,"end_line":79,"end_character":9},"in_reply_to":"7af24918_20537e3a","updated":"2016-03-03 19:50:10.000000000","message":"I don\u0027t think this is the case. I\u0027ve created ports on a shared net and router:external net in one tenant and they\u0027re not visible in another.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"searchlight/elasticsearch/plugins/openstack_clients.py":[{"author":{"_account_id":8959,"name":"Lakshmi N Sampath","email":"lakshmi.sampath@hp.com","username":"lakshmi"},"change_message_id":"2cdc2c56eeb4ed058468c581e41326c496831e07","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    return _session"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"def get_glanceclient():"},{"line_number":61,"context_line":"    session \u003d _get_session()"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    return glance.Client("}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_9c98fa1d","line":60,"range":{"start_line":60,"start_character":20,"end_line":60,"end_character":23},"updated":"2016-03-03 21:28:35.000000000","message":"This is not supposed to be in this patch. I guess this is copied over from another patch on glance","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"9185691ac6fcc082b8d7ff61ae0567de9d6c08a4","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    return _session"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"def get_glanceclient():"},{"line_number":61,"context_line":"    session \u003d _get_session()"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    return glance.Client("}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_f7931759","line":60,"range":{"start_line":60,"start_character":20,"end_line":60,"end_character":23},"in_reply_to":"7af24918_9c98fa1d","updated":"2016-03-03 21:48:24.000000000","message":"I don\u0027t see any change here","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"}],"searchlight/tests/functional/test_glance_plugins.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"9185691ac6fcc082b8d7ff61ae0567de9d6c08a4","unresolved":false,"context_lines":[{"line_number":308,"context_line":"        osclient_patcher.start()"},{"line_number":309,"context_line":"        self.addCleanup(osclient_patcher.stop)"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"        self.image_events \u003d self._load_fixture_data(\u0027events/images.json\u0027)"},{"line_number":312,"context_line":"        self.metadef_events \u003d self._load_fixture_data(\u0027events/metadefs.json\u0027)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"        notification_plugins \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_b784ef0a","line":311,"range":{"start_line":311,"start_character":13,"end_line":311,"end_character":25},"updated":"2016-03-03 21:48:24.000000000","message":"Not sure either, removed","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"},{"author":{"_account_id":8959,"name":"Lakshmi N Sampath","email":"lakshmi.sampath@hp.com","username":"lakshmi"},"change_message_id":"2cdc2c56eeb4ed058468c581e41326c496831e07","unresolved":false,"context_lines":[{"line_number":327,"context_line":"        verification_keys \u003d [\u0027id\u0027, \u0027status\u0027]"},{"line_number":328,"context_line":"        self._verify_result(create_event, verification_keys, result)"},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"    def test_image_update_conflict(self):"},{"line_number":331,"context_line":"        \"\"\"Send a outdated image.update notification event to listener,"},{"line_number":332,"context_line":"           test if the document will be updated"},{"line_number":333,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_9c663a28","line":330,"updated":"2016-03-03 21:28:35.000000000","message":"Not sure again why this is in this patch","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"9185691ac6fcc082b8d7ff61ae0567de9d6c08a4","unresolved":false,"context_lines":[{"line_number":327,"context_line":"        verification_keys \u003d [\u0027id\u0027, \u0027status\u0027]"},{"line_number":328,"context_line":"        self._verify_result(create_event, verification_keys, result)"},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"    def test_image_update_conflict(self):"},{"line_number":331,"context_line":"        \"\"\"Send a outdated image.update notification event to listener,"},{"line_number":332,"context_line":"           test if the document will be updated"},{"line_number":333,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_778ee72e","line":330,"in_reply_to":"7af24918_9c663a28","updated":"2016-03-03 21:48:24.000000000","message":"I don\u0027t see a change","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"},{"author":{"_account_id":8959,"name":"Lakshmi N Sampath","email":"lakshmi.sampath@hp.com","username":"lakshmi"},"change_message_id":"2cdc2c56eeb4ed058468c581e41326c496831e07","unresolved":false,"context_lines":[{"line_number":447,"context_line":"                             \u0027protected\u0027]"},{"line_number":448,"context_line":"        self._verify_result(update_event, verification_keys, result)"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"    def test_md_namespace_update_conflict(self):"},{"line_number":451,"context_line":"        \"\"\"Send a outdated metadef_namespace.update notification event to"},{"line_number":452,"context_line":"        listener, test if the document will be updated"},{"line_number":453,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"7af24918_3c7c46b4","line":450,"range":{"start_line":450,"start_character":0,"end_line":450,"end_character":4},"updated":"2016-03-03 21:28:35.000000000","message":"Same as above","commit_id":"fca8bd11ac2eac13e28a4ccc3e959e03ffa5bba5"}],"searchlight/tests/functional/test_neutron_plugins.py":[{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"# This is in the load file"},{"line_number":27,"context_line":"TENANT1 \u003d \"8eaac046b2c44ab99246cb0850c7f06d\""},{"line_number":28,"context_line":"_now_str \u003d timeutils.isotime(datetime.datetime.utcnow())"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class TestNeutronPlugins(functional.FunctionalTest):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_9c291a8d","line":28,"updated":"2016-03-03 21:28:08.000000000","message":"Isn\u0027t this the reason we created the method plugins/utils.py::get_now_str()?\nThis should be:\n\n    from searchlight.elasticsearch.plugins.utils import get_now_str\n   _now_str \u003d get_now_str\".","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"# This is in the load file"},{"line_number":27,"context_line":"TENANT1 \u003d \"8eaac046b2c44ab99246cb0850c7f06d\""},{"line_number":28,"context_line":"_now_str \u003d timeutils.isotime(datetime.datetime.utcnow())"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class TestNeutronPlugins(functional.FunctionalTest):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_57fc8b3d","line":28,"in_reply_to":"7af24918_9c291a8d","updated":"2016-03-03 21:48:10.000000000","message":"Don\u0027t understand. I need to use a value to pass to mock out get_now_str to get predictable date values.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            \"port_security_enabled\": True,"},{"line_number":54,"context_line":"            \"provider:network_type\": \"vxlan\","},{"line_number":55,"context_line":"            \"provider:physical_network\": None,"},{"line_number":56,"context_line":"            \"provider:segmentation_id\": 1078,"},{"line_number":57,"context_line":"            \"router:external\": False,"},{"line_number":58,"context_line":"            \"shared\": False,"},{"line_number":59,"context_line":"            \"status\": \"ACTIVE\","}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_9c859a5b","line":56,"updated":"2016-03-03 21:28:08.000000000","message":"Aren\u0027t the provider:* fields admin only? They shouldn\u0027t appear in the search results.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            \"port_security_enabled\": True,"},{"line_number":54,"context_line":"            \"provider:network_type\": \"vxlan\","},{"line_number":55,"context_line":"            \"provider:physical_network\": None,"},{"line_number":56,"context_line":"            \"provider:segmentation_id\": 1078,"},{"line_number":57,"context_line":"            \"router:external\": False,"},{"line_number":58,"context_line":"            \"shared\": False,"},{"line_number":59,"context_line":"            \"status\": \"ACTIVE\","}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_f75cb74c","line":56,"in_reply_to":"7af24918_9c859a5b","updated":"2016-03-03 21:48:10.000000000","message":"Yeah. The test framework doesn\u0027t handle this properly at any rate and i didn\u0027t intend to test this functionality here. Restructured this a bit.","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"searchlight/tests/unit/test_neutron_network_plugin.py":[{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":25,"context_line":"import searchlight.tests.utils as test_utils"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"_now_str \u003d timeutils.isotime(datetime.datetime.utcnow())"},{"line_number":29,"context_line":"USER1 \u003d u\u002727f4d76b-be62-4e4e-aa33bb11cc55\u0027"},{"line_number":30,"context_line":"ID1 \u003d \"813dd936-663e-4e5b-877c-986021b73e2c\""},{"line_number":31,"context_line":"TENANT1 \u003d \"8eaac046b2c44ab99246cb0850c7f06d\""}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_1c2a2adb","line":28,"updated":"2016-03-03 21:28:08.000000000","message":"Isn\u0027t this the reason we created the method plugins/utils.py::get_now_str()?","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"766d83891ee56266b1f3952daea52d00412af28f","unresolved":false,"context_lines":[{"line_number":25,"context_line":"import searchlight.tests.utils as test_utils"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"_now_str \u003d timeutils.isotime(datetime.datetime.utcnow())"},{"line_number":29,"context_line":"USER1 \u003d u\u002727f4d76b-be62-4e4e-aa33bb11cc55\u0027"},{"line_number":30,"context_line":"ID1 \u003d \"813dd936-663e-4e5b-877c-986021b73e2c\""},{"line_number":31,"context_line":"TENANT1 \u003d \"8eaac046b2c44ab99246cb0850c7f06d\""}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_d73a5be6","line":28,"in_reply_to":"7af24918_1c2a2adb","updated":"2016-03-03 21:48:10.000000000","message":"No. It could be used instead though, but doesn\u0027t make much difference. I just need a value for mocking","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}],"searchlight/tests/unit/test_neutron_port_plugin.py":[{"author":{"_account_id":14680,"name":"Rick Aulino","email":"rick.aulino@hpe.com","username":"ricka"},"change_message_id":"edead03152fb4d9e65451f02991f9144f1df40be","unresolved":false,"context_lines":[{"line_number":27,"context_line":"ID1 \u003d u\u0027813dd936-663e-4e5b-877c-986021b73e2c\u0027"},{"line_number":28,"context_line":"TENANT1 \u003d u\u00278eaac046b2c44ab99246cb0850c7f06d\u0027"},{"line_number":29,"context_line":"NETWORK1 \u003d u\u0027bc0adf22-3aef-4e7b-8b99-12670b5a76b5\u0027"},{"line_number":30,"context_line":"_now_str \u003d timeutils.isotime(datetime.datetime.utcnow())"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"def _create_port_fixture(port_id, tenant_id, network_id, **kwargs):"}],"source_content_type":"text/x-python","patch_set":6,"id":"7af24918_3c49e6b7","line":30,"updated":"2016-03-03 21:28:08.000000000","message":"Isn\u0027t this the reason we created the method plugins/utils.py::get_now_str()?","commit_id":"c77155deac33e4f99be7cae0195c4cb85034c80e"}]}