)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"93a79945bd4796856d2bacb444e40fdf7df51bc1","unresolved":false,"context_lines":[{"line_number":169,"context_line":"]"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"def driver_dict_from_config(named_driver_config, *args, **kwargs):"},{"line_number":173,"context_line":"    driver_registry \u003d dict()"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_aab16491","line":172,"updated":"2016-11-09 14:20:52.000000000","message":"The next step would be to use something like stevedore I guess.","commit_id":"bd5c54bbd3c2d425e336abc7d230faa97f1c5c72"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"c39d649027cb9647b52c3062aeac7efb18586ddd","unresolved":false,"context_lines":[{"line_number":169,"context_line":"]"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"def driver_dict_from_config(named_driver_config, *args, **kwargs):"},{"line_number":173,"context_line":"    driver_registry \u003d dict()"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_d61b9a71","line":172,"in_reply_to":"9a629dbe_3be4bfdf","updated":"2016-11-09 17:02:30.000000000","message":"This is a left-over from the days when we got this from nova.conf. I agree though, we don\u0027t need stevedore here as this is explicitly *not* something we want to be user configurable.\n\nWe should just move all this stuff into volume.py and load just the class we name, at time of first use. That\u0027s certainly not for this patch though.","commit_id":"bd5c54bbd3c2d425e336abc7d230faa97f1c5c72"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"667e07d38b81a4494ee00fb819ba4559966576eb","unresolved":false,"context_lines":[{"line_number":169,"context_line":"]"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"def driver_dict_from_config(named_driver_config, *args, **kwargs):"},{"line_number":173,"context_line":"    driver_registry \u003d dict()"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_3be4bfdf","line":172,"in_reply_to":"9a629dbe_aab16491","updated":"2016-11-09 16:49:34.000000000","message":"That seems a bit heavyweight to me and these aren\u0027t really plugins/extensions. I\u0027m not entirely sure why we load these all up on init anyway, we could just load them on demand when we get a volume that actually uses a particular type.","commit_id":"bd5c54bbd3c2d425e336abc7d230faa97f1c5c72"}]}