{"action":"opened","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/32801","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/32801/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/32801/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/32801/events","html_url":"https://github.com/saltstack/salt/issues/32801","id":150581994,"number":32801,"title":"archive.extracted now has unexpected/wanted behavior after 66ba35261cb4775e196ef738979914f8aaaddf4f","user":{"login":"sjorge","id":379665,"avatar_url":"https://avatars.githubusercontent.com/u/379665?v=3","gravatar_id":"","url":"https://api.github.com/users/sjorge","html_url":"https://github.com/sjorge","followers_url":"https://api.github.com/users/sjorge/followers","following_url":"https://api.github.com/users/sjorge/following{/other_user}","gists_url":"https://api.github.com/users/sjorge/gists{/gist_id}","starred_url":"https://api.github.com/users/sjorge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sjorge/subscriptions","organizations_url":"https://api.github.com/users/sjorge/orgs","repos_url":"https://api.github.com/users/sjorge/repos","events_url":"https://api.github.com/users/sjorge/events{/privacy}","received_events_url":"https://api.github.com/users/sjorge/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-04-23T18:19:40Z","updated_at":"2016-04-23T18:19:40Z","closed_at":null,"body":"### Description of Issue/Question\r\nThis seems to have resulted in 'if_missing' to be creates as a directory if something is provided.\r\n\r\n```yaml\r\naccount.profile.base::{{ login }}:\r\n  archive.extracted:\r\n    - name: /home/{{ login }}\r\n    - if_missing: /home/{{ login }}/.salt-profile-base\r\n    - archive_format: tar\r\n    - source:\r\n        - salt://common/account/_files/profile-base.tar.gz.{{ salt.grains.get('os')|lower }}\r\n        - salt://common/account/_files/profile-fallback.tar.gz\r\n    - user: {{ login }}\r\n    - group: {{ group }}\r\n    - require:\r\n        - user: account.user::{{ login }}\r\n  file.managed:\r\n    - name: /home/{{ login }}/.salt-profile-base\r\n    - user: {{ login }}\r\n    - group: {{ group }}\r\n    - contents: '## salt marker file ##'\r\n    - watch:\r\n        - archive: account.profile.base::{{ login }}\r\n```\r\n\r\nThe above snippet extracts the archive and used to create .salt-profile-base as a marker file.\r\nSo get a update profile the user just has to remove the marker and it will be updated next profile run.\r\n\r\nHowever after this commit it seems to archive.extracted seems to create the location specified in if_missing as a directory.\r\n\r\nEven when removing the file.managed one, a directory with the path of if_missing shows up. (I checked that it was not in the archive. This is also easily verified by changing the if_missing to something else to test.\r\n\r\nThis new odd behavior seems to come form this commit as my minions running a pure 2016.3rc2 do not suffer from this. Only the ones I updated today tracking the 2016.3 branch.\r\n\r\nAlso assuming that if_missing is always a directory is probably not desired.\r\n\r\n### Setup\r\nSee above for an example snippet.\r\n\r\n### Steps to Reproduce Issue\r\nPut a random path in if_missing and it will end up as a directory.\r\n\r\n### Versions Report\r\n2016.3 as of 2016/04/23 @ ~20:00\r\n```yaml\r\nSalt Version:\r\n           Salt: 2016.3.0\r\n\r\nDependency Versions:\r\n           cffi: Not Installed\r\n       cherrypy: 3.8.0\r\n       dateutil: 2.4.0\r\n          gitdb: 0.6.4\r\n      gitpython: 2.0.0\r\n          ioflo: 1.5.1\r\n         Jinja2: 2.8\r\n        libgit2: Not Installed\r\n        libnacl: 1.4.4\r\n       M2Crypto: 0.22\r\n           Mako: Not Installed\r\n   msgpack-pure: Not Installed\r\n msgpack-python: 0.4.7\r\n   mysql-python: Not Installed\r\n      pycparser: Not Installed\r\n       pycrypto: 2.6.1\r\n         pygit2: Not Installed\r\n         Python: 2.7.11 (default, Mar 18 2016, 13:38:08)\r\n   python-gnupg: 2.0.2\r\n         PyYAML: 3.11\r\n          PyZMQ: 14.4.1\r\n           RAET: 0.6.5\r\n          smmap: 0.9.0\r\n        timelib: 0.2.4\r\n        Tornado: 4.3\r\n            ZMQ: 4.1.3\r\n\r\nSystem Versions:\r\n           dist:\r\n        machine: i86pc\r\n        release: 5.11\r\n         system: SunOS\r\n        version: Not Installed\r\n```"}}