{"action":"created","issue":{"url":"https://api.github.com/repos/airbnb/airflow/issues/9","labels_url":"https://api.github.com/repos/airbnb/airflow/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/airbnb/airflow/issues/9/comments","events_url":"https://api.github.com/repos/airbnb/airflow/issues/9/events","html_url":"https://github.com/airbnb/airflow/issues/9","id":85787166,"number":9,"title":"How do you feel about putting hook configuration in airflow.cfg?","user":{"login":"jbalogh","id":57258,"avatar_url":"https://avatars.githubusercontent.com/u/57258?v=3","gravatar_id":"","url":"https://api.github.com/users/jbalogh","html_url":"https://github.com/jbalogh","followers_url":"https://api.github.com/users/jbalogh/followers","following_url":"https://api.github.com/users/jbalogh/following{/other_user}","gists_url":"https://api.github.com/users/jbalogh/gists{/gist_id}","starred_url":"https://api.github.com/users/jbalogh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbalogh/subscriptions","organizations_url":"https://api.github.com/users/jbalogh/orgs","repos_url":"https://api.github.com/users/jbalogh/repos","events_url":"https://api.github.com/users/jbalogh/events{/privacy}","received_events_url":"https://api.github.com/users/jbalogh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-06T17:52:16Z","updated_at":"2015-06-07T07:34:43Z","closed_at":null,"body":"We distribute our configs via text files managed by puppet so it would be awkward to put that into the db for airflow. Would you accept a patch that looks for `conn_id` in `settings.conf` before checking the database?"},"comment":{"url":"https://api.github.com/repos/airbnb/airflow/issues/comments/109713169","html_url":"https://github.com/airbnb/airflow/issues/9#issuecomment-109713169","issue_url":"https://api.github.com/repos/airbnb/airflow/issues/9","id":109713169,"user":{"login":"mistercrunch","id":487433,"avatar_url":"https://avatars.githubusercontent.com/u/487433?v=3","gravatar_id":"","url":"https://api.github.com/users/mistercrunch","html_url":"https://github.com/mistercrunch","followers_url":"https://api.github.com/users/mistercrunch/followers","following_url":"https://api.github.com/users/mistercrunch/following{/other_user}","gists_url":"https://api.github.com/users/mistercrunch/gists{/gist_id}","starred_url":"https://api.github.com/users/mistercrunch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mistercrunch/subscriptions","organizations_url":"https://api.github.com/users/mistercrunch/orgs","repos_url":"https://api.github.com/users/mistercrunch/repos","events_url":"https://api.github.com/users/mistercrunch/events{/privacy}","received_events_url":"https://api.github.com/users/mistercrunch/received_events","type":"User","site_admin":false},"created_at":"2015-06-07T07:34:43Z","updated_at":"2015-06-07T07:34:43Z","body":"Are you concerned about the security aspect of storing connection information in a database, or is it more about deployment and how you get the information in there?\r\n\r\nIf it's the later you could easily setup a simple script that populates the database. It would look like what's in `init_db` here https://github.com/mistercrunch/airflow/blob/master/airflow/utils.py#L79,  but with `session.merge` instead of `session.add`.\r\n\r\nIt's true that I'd also rather have connection information in source control rather than in the db. Having it in the db is an artifact of early development. But it does work. I believe all of the connection info fetching is done through here: https://github.com/mistercrunch/airflow/blob/master/airflow/utils.py#L79"}}