)]}'
{"nodepool/driver/kubernetes/provider.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"3977aa41d917c407da3e042072a541b677c8b7f2","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        except FileNotFoundError:"},{"line_number":55,"context_line":"            self.log.debug(\"Kubernetes config file not found, attempting \""},{"line_number":56,"context_line":"                           \"to load in-cluster configs\")"},{"line_number":57,"context_line":"            conf \u003d config.load_incluster_config()"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        return ("},{"line_number":60,"context_line":"            k8s_client.CoreV1Api(conf),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_171f526c","line":57,"updated":"2019-06-28 13:14:11.000000000","message":"Another approach would be to make this an explicit action using a config option. I went with a fallback method to keep backward compatibility and avoid introducing new config options.","commit_id":"e819f4b4b1bfe35cbb18b1a0e0d21579c8e06d71"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"93870cc1c4842f832be4ce94853d89ebbb88680e","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        except FileNotFoundError:"},{"line_number":55,"context_line":"            self.log.debug(\"Kubernetes config file not found, attempting \""},{"line_number":56,"context_line":"                           \"to load in-cluster configs\")"},{"line_number":57,"context_line":"            conf \u003d config.load_incluster_config()"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        return ("},{"line_number":60,"context_line":"            k8s_client.CoreV1Api(conf),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_45996b6a","line":57,"in_reply_to":"9fb8cfa7_171f526c","updated":"2019-07-02 12:09:25.000000000","message":"This debug message could get annoying for folks running in k8s using this method. Since this is how you\u0027re running, keep an eye on it - maybe the debug log line should go away. But maybe it\u0027s fine.","commit_id":"e819f4b4b1bfe35cbb18b1a0e0d21579c8e06d71"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"f2bfc0322c123f539e98785851ed0ec9e8762b6f","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        except FileNotFoundError:"},{"line_number":55,"context_line":"            self.log.debug(\"Kubernetes config file not found, attempting \""},{"line_number":56,"context_line":"                           \"to load in-cluster configs\")"},{"line_number":57,"context_line":"            conf \u003d config.load_incluster_config()"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        return ("},{"line_number":60,"context_line":"            k8s_client.CoreV1Api(conf),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_a51e27b8","line":57,"in_reply_to":"9fb8cfa7_45996b6a","updated":"2019-07-02 12:22:38.000000000","message":"yeah. That\u0027s why I made it `debug` instead of info. I could see why some folks could be annoyed by it but I wanted to make sure we\u0027d highlight that the fallback strategy is being attempted because of the missing file","commit_id":"e819f4b4b1bfe35cbb18b1a0e0d21579c8e06d71"}]}
