)]}'
{"neutron/objects/subnet.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3d5e325ac19fec5f5dc7e06f2dfce4425e907481","unresolved":false,"context_lines":[{"line_number":498,"context_line":"            query \u003d query.filter(~exists().where(and_("},{"line_number":499,"context_line":"                cls.db_model.id \u003d\u003d service_type_model.subnet_id,"},{"line_number":500,"context_line":"                service_type_model.service_type \u003d\u003d filtered_service_type)))"},{"line_number":501,"context_line":"        return [segment_id for (segment_id,) in query.all()]"},{"line_number":502,"context_line":""},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"@base.NeutronObjectRegistry.register"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_53b988c9","line":501,"updated":"2020-07-16 13:28:27.000000000","message":"If query is still returning segment_id\u003dNone, this is because, somehow, this network has some subnets without segment associated. In any case, although that could be filtered in the query, you can also remove those None values here:\n\nreturn [segment_id for (segment_id,) in query.all() if segment_id]","commit_id":"2b9033b5a9879e15db5a70297a3105222bcb2891"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"01178e2f7b0ce6a83384350750a486c6923a5037","unresolved":false,"context_lines":[{"line_number":498,"context_line":"            query \u003d query.filter(~exists().where(and_("},{"line_number":499,"context_line":"                cls.db_model.id \u003d\u003d service_type_model.subnet_id,"},{"line_number":500,"context_line":"                service_type_model.service_type \u003d\u003d filtered_service_type)))"},{"line_number":501,"context_line":"        return [segment_id for (segment_id,) in query.all()]"},{"line_number":502,"context_line":""},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"@base.NeutronObjectRegistry.register"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_73e76ce4","line":501,"in_reply_to":"bf51134e_53b988c9","updated":"2020-07-16 13:29:52.000000000","message":"BTW, try to reproduce it with UTs. This is the fastest way to properly test this query, that\u0027s why we have all the framework ready for this.","commit_id":"2b9033b5a9879e15db5a70297a3105222bcb2891"}]}
