{"action":"closed","issue":{"url":"https://api.github.com/repos/edin-m/node-vagrant/issues/9","repository_url":"https://api.github.com/repos/edin-m/node-vagrant","labels_url":"https://api.github.com/repos/edin-m/node-vagrant/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/edin-m/node-vagrant/issues/9/comments","events_url":"https://api.github.com/repos/edin-m/node-vagrant/issues/9/events","html_url":"https://github.com/edin-m/node-vagrant/issues/9","id":158456275,"number":9,"title":"Provisioning with docker doesn't work","user":{"login":"ilbonte","id":7412876,"avatar_url":"https://avatars.githubusercontent.com/u/7412876?v=3","gravatar_id":"","url":"https://api.github.com/users/ilbonte","html_url":"https://github.com/ilbonte","followers_url":"https://api.github.com/users/ilbonte/followers","following_url":"https://api.github.com/users/ilbonte/following{/other_user}","gists_url":"https://api.github.com/users/ilbonte/gists{/gist_id}","starred_url":"https://api.github.com/users/ilbonte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ilbonte/subscriptions","organizations_url":"https://api.github.com/users/ilbonte/orgs","repos_url":"https://api.github.com/users/ilbonte/repos","events_url":"https://api.github.com/users/ilbonte/events{/privacy}","received_events_url":"https://api.github.com/users/ilbonte/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/edin-m/node-vagrant/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-06-03T21:15:58Z","updated_at":"2016-06-13T17:06:10Z","closed_at":"2016-06-13T17:06:10Z","body":"Hi, if you take the [Vagrant file from the test folder](https://github.com/edin-m/node-vagrant/blob/master/test/example1.Vagrantfile) and you run `vagrant up` it will not work and you'll get this error: ` VagrantPlugins::DockerProvisioner::Config:\r\n The following settings shouldn't exist: pull_images\r\n`\r\nYou will get the same error if you run this code:\r\n\r\n```javascript\r\nvar vagrant = require ('node-vagrant');\r\n\r\nvar machine = vagrant.create({\r\n    cwd: process.cwd(),\r\n    env: process.env\r\n});\r\n\r\nvar config = {\r\n    \"config\": {\r\n        \"vm\": {\r\n            \"box\": \"ubuntu/trusty64\"\r\n        },\r\n\r\n        \"provisioners\": {\r\n            \"docker\": {\r\n                \"pull_images \": \"\\\"ubuntu\\\"\"\r\n            }\r\n        }\r\n    }\r\n};\r\n\r\nfunction onInit(err, out) {\r\n    if (err) throw new Error(err);\r\n\r\n    machine.up(function(err, out) {\r\n        if (err) throw new Error(err);\r\n        console.log('machine is up!');\r\n        fs.unlinkSync('./Vagrantfile');\r\n    });\r\n}\r\n\r\n\r\nmachine.init('ubuntu/trusty64', config, onInit);\r\n\r\n```\r\n\r\n**I think that the problem** is that the[ template puts](https://github.com/edin-m/node-vagrant/blob/master/templates/basic.tpl#L36) a `=` in the Vagrantfile for every provisioners but in this case there should't be... For an example of the syntax scroll down to \"BUILDING IMAGES\": https://www.vagrantup.com/docs/provisioning/docker.html (I know, different syntax for every provider: sometimes there is \"=\", sometimes \":\" and sometime nothing)\r\n\r\n\r\nIn fact if you edit the Vagrantfile produced from the code above removing the \"=\" between `docker.pull_images ` and `\"ubuntu\"` it works!\r\n\r\n\r\nIf you want to try by yourself this is the vagrantfile generated by node-vagrant. To get it work you need to remove the `=` :\r\n```\r\nVagrant.configure(2) do |config|\r\n\r\n  # https://docs.vagrantup.com.\r\n\r\n \r\n  config.vm.box = \"ubuntu/trusty64\"\r\n  \r\n  \r\n  config.vm.provision \"docker\" do |docker|\r\n    \r\n    docker.pull_images  = \"ubuntu\"\r\n    \r\n  end\r\n  \r\nend\r\n```"}}
{"actor":{"display_login":"edin-m"}}