{"action":"opened","issue":{"url":"https://api.github.com/repos/saltstack-formulas/salt-formula/issues/190","labels_url":"https://api.github.com/repos/saltstack-formulas/salt-formula/issues/190/labels{/name}","comments_url":"https://api.github.com/repos/saltstack-formulas/salt-formula/issues/190/comments","events_url":"https://api.github.com/repos/saltstack-formulas/salt-formula/issues/190/events","html_url":"https://github.com/saltstack-formulas/salt-formula/issues/190","id":118369991,"number":190,"title":"Error while trying to use salt.gitfs.gitpython","user":{"login":"johnccfm","id":8737019,"avatar_url":"https://avatars.githubusercontent.com/u/8737019?v=3","gravatar_id":"","url":"https://api.github.com/users/johnccfm","html_url":"https://github.com/johnccfm","followers_url":"https://api.github.com/users/johnccfm/followers","following_url":"https://api.github.com/users/johnccfm/following{/other_user}","gists_url":"https://api.github.com/users/johnccfm/gists{/gist_id}","starred_url":"https://api.github.com/users/johnccfm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnccfm/subscriptions","organizations_url":"https://api.github.com/users/johnccfm/orgs","repos_url":"https://api.github.com/users/johnccfm/repos","events_url":"https://api.github.com/users/johnccfm/events{/privacy}","received_events_url":"https://api.github.com/users/johnccfm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-11-23T12:24:23Z","updated_at":"2015-11-23T12:24:23Z","closed_at":null,"body":"I'm getting an error when trying use this state.\r\n\r\n```\r\njohncc@aws-dev-c-salt-1:~⟫ sudo salt-call -l debug state.sls salt.gitfs.gitpython                                                                                                                                                              \r\n[sudo] password for johncc: \r\n[DEBUG   ] Reading configuration from /etc/salt/minion\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/_schedule.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/_schedule.conf\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/f_defaults.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/f_defaults.conf\r\n[DEBUG   ] Configuration file path: /etc/salt/minion\r\n[DEBUG   ] Reading configuration from /etc/salt/minion\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/_schedule.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/_schedule.conf\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/f_defaults.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/f_defaults.conf\r\n[DEBUG   ] Initializing new SAuth for ('/etc/salt/pki/minion', 'aws-dev-c-salt-1', 'tcp://192.168.101.7:4506')\r\n[DEBUG   ] Decrypting the current master AES key\r\n[DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem\r\n[DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem\r\n[DEBUG   ] LazyLoaded jinja.render\r\n[DEBUG   ] LazyLoaded yaml.render\r\n[DEBUG   ] LazyLoaded state.sls\r\n[DEBUG   ] LazyLoaded saltutil.is_running\r\n[DEBUG   ] LazyLoaded grains.get\r\n[DEBUG   ] Re-using SAuth for ('/etc/salt/pki/minion', 'aws-dev-c-salt-1', 'tcp://192.168.101.7:4506')\r\n[DEBUG   ] Reading configuration from /etc/salt/minion\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/_schedule.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/_schedule.conf\r\n[DEBUG   ] Including configuration from '/etc/salt/minion.d/f_defaults.conf'\r\n[DEBUG   ] Reading configuration from /etc/salt/minion.d/f_defaults.conf\r\n[DEBUG   ] Re-using SAuth for ('/etc/salt/pki/minion', 'aws-dev-c-salt-1', 'tcp://192.168.101.7:4506')\r\n[DEBUG   ] Loaded minion key: /etc/salt/pki/minion/minion.pem\r\n[INFO    ] Loading fresh modules for state activity\r\n[DEBUG   ] LazyLoaded jinja.render\r\n[DEBUG   ] LazyLoaded yaml.render\r\n[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/gitfs/gitpython.sls'\r\n[DEBUG   ] LazyLoaded cmd.run\r\n[DEBUG   ] Jinja search path: ['/var/cache/salt/minion/files/base']\r\n[DEBUG   ] Re-using SAuth for ('/etc/salt/pki/minion', 'aws-dev-c-salt-1', 'tcp://192.168.101.7:4506')\r\n[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/map.jinja'\r\n[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/defaults.yaml'\r\n[DEBUG   ] LazyLoaded grains.filter_by\r\n[DEBUG   ] LazyLoaded pillar.get\r\n[CRITICAL] Rendering SLS 'base:salt.gitfs.gitpython' failed: Jinja variable 'dict object' has no attribute 'gitpython'\r\n[DEBUG   ] Re-using SAuth for ('/etc/salt/pki/minion', 'aws-dev-c-salt-1', 'tcp://192.168.101.7:4506')\r\n[DEBUG   ] LazyLoaded highstate.output\r\nlocal:\r\n    Data failed to compile:\r\n----------\r\n    Rendering SLS 'base:salt.gitfs.gitpython' failed: Jinja variable 'dict object' has no attribute 'gitpython'\r\n```\r\n\r\nIt was suggested in the IRC channel by AndreasLutro that the problem is caused by incorrect merging of the default and OS specific settings dictionaries.\r\n\r\nIn particular, it was suggested that this line is at fault https://github.com/saltstack-formulas/salt-formula/blob/master/salt/map.jinja#L79 and should be changed to do a deep merge instead of a shallow merge."}}