)]}'
{"cinder/volume/drivers/nfs.py":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"d105a1e04a60a9da57cdfb0300f20c3df8bf9ed2","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        package \u003d \u0027mount.nfs\u0027"},{"line_number":149,"context_line":"        try:"},{"line_number":150,"context_line":"            self._execute(package, check_exit_code\u003dFalse,"},{"line_number":151,"context_line":"                          run_as_root\u003dTrue)"},{"line_number":152,"context_line":"        except OSError as exc:"},{"line_number":153,"context_line":"            if exc.errno \u003d\u003d errno.ENOENT:"},{"line_number":154,"context_line":"                msg \u003d _(\u0027%s is not installed\u0027) % package"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_085f2042","line":151,"updated":"2016-04-18 06:37:06.000000000","message":"I realise that this is merged (and backported)  but how does this work?\n\nI don\u0027t see mount.nfs in etc/cinder/rootwrap.d/volume.filters ?","commit_id":"c811d98c8b3991a9c298fce914f091b2a9ac215f"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"d105a1e04a60a9da57cdfb0300f20c3df8bf9ed2","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        try:"},{"line_number":150,"context_line":"            self._execute(package, check_exit_code\u003dFalse,"},{"line_number":151,"context_line":"                          run_as_root\u003dTrue)"},{"line_number":152,"context_line":"        except OSError as exc:"},{"line_number":153,"context_line":"            if exc.errno \u003d\u003d errno.ENOENT:"},{"line_number":154,"context_line":"                msg \u003d _(\u0027%s is not installed\u0027) % package"},{"line_number":155,"context_line":"                raise exception.NfsException(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_6ba04ef5","line":152,"updated":"2016-04-18 06:37:06.000000000","message":"When you use run_as_root\u003dTrue putils.execute no longer returns OSError it returns ProcessExecutionError.\n\nhttp://docs.openstack.org/developer/oslo.concurrency/api/processutils.html#oslo_concurrency.processutils.execute\n\nThe upshot is init_host will catch the new exception and return early, but this code isn\u0027t ever executed (outside of the tests)","commit_id":"c811d98c8b3991a9c298fce914f091b2a9ac215f"}]}