{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/24045","labels_url":"https://api.github.com/repos/saltstack/salt/issues/24045/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/24045/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/24045/events","html_url":"https://github.com/saltstack/salt/issues/24045","id":79130572,"number":24045,"title":"npm.bootstrap returns error, even though it eventually succeeds","user":{"login":"c10b10","id":306633,"avatar_url":"https://avatars.githubusercontent.com/u/306633?v=3","gravatar_id":"","url":"https://api.github.com/users/c10b10","html_url":"https://github.com/c10b10","followers_url":"https://api.github.com/users/c10b10/followers","following_url":"https://api.github.com/users/c10b10/following{/other_user}","gists_url":"https://api.github.com/users/c10b10/gists{/gist_id}","starred_url":"https://api.github.com/users/c10b10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/c10b10/subscriptions","organizations_url":"https://api.github.com/users/c10b10/orgs","repos_url":"https://api.github.com/users/c10b10/repos","events_url":"https://api.github.com/users/c10b10/events{/privacy}","received_events_url":"https://api.github.com/users/c10b10/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/saltstack/salt/labels/Info%20Needed","name":"Info Needed","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/saltstack/salt/milestones/37","html_url":"https://github.com/saltstack/salt/milestones/Blocked","labels_url":"https://api.github.com/repos/saltstack/salt/milestones/37/labels","id":343741,"number":37,"title":"Blocked","description":"Issues which are blocked in some way, whether they need further discussion, clarification, justification, or are blocked for some other reason.","creator":{"login":"basepi","id":702318,"avatar_url":"https://avatars.githubusercontent.com/u/702318?v=3","gravatar_id":"","url":"https://api.github.com/users/basepi","html_url":"https://github.com/basepi","followers_url":"https://api.github.com/users/basepi/followers","following_url":"https://api.github.com/users/basepi/following{/other_user}","gists_url":"https://api.github.com/users/basepi/gists{/gist_id}","starred_url":"https://api.github.com/users/basepi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/basepi/subscriptions","organizations_url":"https://api.github.com/users/basepi/orgs","repos_url":"https://api.github.com/users/basepi/repos","events_url":"https://api.github.com/users/basepi/events{/privacy}","received_events_url":"https://api.github.com/users/basepi/received_events","type":"User","site_admin":false},"open_issues":285,"closed_issues":638,"state":"open","created_at":"2013-05-28T22:24:12Z","updated_at":"2015-09-18T16:27:29Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2015-05-21T19:32:19Z","updated_at":"2015-09-18T19:07:20Z","closed_at":null,"body":"I've been running a salt sls using a salt-ssh and `npm.bootstrap` always errors out with:\r\n\r\n```sh\r\nError Bootstrapping '/var/www/...':\r\n```\r\n\r\nIf I check on the server, I see that the `npm.installed` succeeded.\r\n\r\nI tried debugging the issue by editing the `npm.py` module and state files, but it seems that salt-ssh acts as if I haven't changed anything (Is there some kind of cache active?)\r\n\r\nI'm proposing that this might be a bug.\r\n\r\nTried this on both v2014.7.1_1 and v2015.5.0."},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/141538006","html_url":"https://github.com/saltstack/salt/issues/24045#issuecomment-141538006","issue_url":"https://api.github.com/repos/saltstack/salt/issues/24045","id":141538006,"user":{"login":"jfindlay","id":702905,"avatar_url":"https://avatars.githubusercontent.com/u/702905?v=3","gravatar_id":"","url":"https://api.github.com/users/jfindlay","html_url":"https://github.com/jfindlay","followers_url":"https://api.github.com/users/jfindlay/followers","following_url":"https://api.github.com/users/jfindlay/following{/other_user}","gists_url":"https://api.github.com/users/jfindlay/gists{/gist_id}","starred_url":"https://api.github.com/users/jfindlay/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jfindlay/subscriptions","organizations_url":"https://api.github.com/users/jfindlay/orgs","repos_url":"https://api.github.com/users/jfindlay/repos","events_url":"https://api.github.com/users/jfindlay/events{/privacy}","received_events_url":"https://api.github.com/users/jfindlay/received_events","type":"User","site_admin":false},"created_at":"2015-09-18T19:07:20Z","updated_at":"2015-09-18T19:07:20Z","body":"@eliasp, I do not see how removing the `else` clause in the exec module function would prevent the `Error bootstrapping` message.  Can you explain your reasoning?  Thanks."}}