)]}'
{"neutron/conf/common.py":[{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"b8cd6b30e21d29b7b5d8c626cd0db08e54a02048","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                       \u0027instances via DHCP and RA MTU options.\u0027)),"},{"line_number":134,"context_line":"    cfg.StrOpt(\u0027ipam_driver\u0027, default\u003d\u0027internal\u0027,"},{"line_number":135,"context_line":"               help\u003d_(\"Neutron IPAM (IP address management) driver to use. \""},{"line_number":136,"context_line":"                      \"If ipam_driver is not set, no IPAM driver is used. \""},{"line_number":137,"context_line":"                      \"By default, the reference implementation of Neutron \""},{"line_number":138,"context_line":"                      \"IPAM driver is used.\")),"},{"line_number":139,"context_line":"    cfg.BoolOpt(\u0027vlan_transparent\u0027, default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_019b0473","line":136,"range":{"start_line":136,"start_character":23,"end_line":136,"end_character":74},"updated":"2016-08-29 11:36:08.000000000","message":"Correct me, if I\u0027m wrong, but `default` means, that nothing is selected, then `internal` will be chosen.","commit_id":"6b96bc672afa9a4908115e618ece98fbee4b67f0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3990decd6c1e222bb1c9a115743e80c7c0a05e19","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                       \u0027instances via DHCP and RA MTU options.\u0027)),"},{"line_number":134,"context_line":"    cfg.StrOpt(\u0027ipam_driver\u0027, default\u003d\u0027internal\u0027,"},{"line_number":135,"context_line":"               help\u003d_(\"Neutron IPAM (IP address management) driver to use. \""},{"line_number":136,"context_line":"                      \"If ipam_driver is empty, no IPAM driver is used. \""},{"line_number":137,"context_line":"                      \"By default, the reference implementation of Neutron \""},{"line_number":138,"context_line":"                      \"IPAM driver is used.\")),"},{"line_number":139,"context_line":"    cfg.BoolOpt(\u0027vlan_transparent\u0027, default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_a9b4f381","line":136,"updated":"2016-08-29 14:51:59.000000000","message":"It doesn\u0027t seem like this sentence is necessary?","commit_id":"7fb0f63d6ac72f44f5605a51f40526f084289dae"},{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"3d70975a183291e1662445177b070fbbd73e2de4","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                       \u0027instances via DHCP and RA MTU options.\u0027)),"},{"line_number":134,"context_line":"    cfg.StrOpt(\u0027ipam_driver\u0027, default\u003d\u0027internal\u0027,"},{"line_number":135,"context_line":"               help\u003d_(\"Neutron IPAM (IP address management) driver to use. \""},{"line_number":136,"context_line":"                      \"If ipam_driver is empty, no IPAM driver is used. \""},{"line_number":137,"context_line":"                      \"By default, the reference implementation of Neutron \""},{"line_number":138,"context_line":"                      \"IPAM driver is used.\")),"},{"line_number":139,"context_line":"    cfg.BoolOpt(\u0027vlan_transparent\u0027, default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_c7f11b03","line":136,"in_reply_to":"fa7ab95a_a9b4f381","updated":"2016-08-29 16:07:23.000000000","message":"Yes, we should remove this. We actually should have removed it when we did the migration to pluggable IPAM because I\u0027d like to stop advertising it as an option at all. But, we could do it in a patch that will remove non-pluggable IPAM code altogether. I\u0027ve been meaning to post that patch and I can post it today.\n\nIf we do that, it kind of makes this patch pointless, to be honest. Should we take advantage of this opportunity to just remove that code in this patch?\n\nI guess this patch does have a use in that it ties up a loose end from the migration patch which has already merged. So, I\u0027ll go ahead and fix this one up, we can merge it, and then I\u0027ll post the patch to clean up non-pluggable code.","commit_id":"7fb0f63d6ac72f44f5605a51f40526f084289dae"}]}