{"action":"created","issue":{"url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/110","labels_url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/110/labels{/name}","comments_url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/110/comments","events_url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/110/events","html_url":"https://github.com/opscode-cookbooks/aws/pull/110","id":58174790,"number":110,"title":"[test me] compile_time works with Chef 11/12 including chef-sugar","user":{"login":"scalp42","id":1475276,"avatar_url":"https://avatars.githubusercontent.com/u/1475276?v=3","gravatar_id":"","url":"https://api.github.com/users/scalp42","html_url":"https://github.com/scalp42","followers_url":"https://api.github.com/users/scalp42/followers","following_url":"https://api.github.com/users/scalp42/following{/other_user}","gists_url":"https://api.github.com/users/scalp42/gists{/gist_id}","starred_url":"https://api.github.com/users/scalp42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scalp42/subscriptions","organizations_url":"https://api.github.com/users/scalp42/orgs","repos_url":"https://api.github.com/users/scalp42/repos","events_url":"https://api.github.com/users/scalp42/events{/privacy}","received_events_url":"https://api.github.com/users/scalp42/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/opscode-cookbooks/aws/labels/Signed%20CLA","name":"Signed CLA","color":"ededed"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-02-19T07:02:23Z","updated_at":"2015-03-20T18:01:40Z","closed_at":"2015-03-20T04:40:48Z","pull_request":{"url":"https://api.github.com/repos/opscode-cookbooks/aws/pulls/110","html_url":"https://github.com/opscode-cookbooks/aws/pull/110","diff_url":"https://github.com/opscode-cookbooks/aws/pull/110.diff","patch_url":"https://github.com/opscode-cookbooks/aws/pull/110.patch"},"body":"The test recipe is the following (please note that this is the patched aws from this PR, with a little difference to trigger warning, see below):\r\n\r\n```ruby\r\ninclude_recipe 'chef-sugar'\r\ninclude_recipe 'aws'\r\n```\r\n\r\n`chef_gem` patched code (`compile_time` is not set to anything to trigger the warning):\r\n\r\n```ruby\r\nchef_gem 'aws-sdk' do\r\n  version node['aws']['aws_sdk_version']\r\n  if Chef::Resource::ChefGem.instance_methods(false).include?(:compile_time)\r\n    compile_time\r\n  end\r\n  action :install\r\nend\r\n```\r\n\r\nHere are my results:\r\n\r\nChef **12.1.0.dev.0**, without specifying `true`, just `compile_time` to show the warning :\r\n\r\n```\r\n==> zs1: [2015-02-19T06:47:57+00:00] INFO: Forking chef instance to converge...\r\n==> zs1: [2015-02-19T06:47:57+00:00] INFO: *** Chef 12.1.0.dev.0 ***\r\n==> zs1: [2015-02-19T06:47:57+00:00] INFO: Chef-client pid: 14440\r\n==> zs1: [2015-02-19T06:48:15+00:00] WARN: chef_gem[chef-sugar] chef_gem compile_time installation is deprecated\r\n==> zs1: [2015-02-19T06:48:15+00:00] WARN: chef_gem[chef-sugar] Please set `compile_time false` on the resource to use the new behavior.\r\n==> zs1: [2015-02-19T06:48:15+00:00] WARN: chef_gem[chef-sugar] or set `compile_time true` on the resource if compile_time behavior is required.\r\n==> zs1: [2015-02-19T06:48:23+00:00] INFO: chef_gem[chef-sugar] installed chef-sugar at 2.5.0\r\n==> zs1: [2015-02-19T06:48:23+00:00] WARN: chef_gem[aws-sdk] chef_gem compile_time installation is deprecated\r\n==> zs1: [2015-02-19T06:48:23+00:00] WARN: chef_gem[aws-sdk] Please set `compile_time false` on the resource to use the new behavior.\r\n==> zs1: [2015-02-19T06:48:23+00:00] WARN: chef_gem[aws-sdk] or set `compile_time true` on the resource if compile_time behavior is required.\r\n==> zs1: [2015-02-19T06:48:31+00:00] INFO: chef_gem[aws-sdk] installed aws-sdk at ~> 2.0.22\r\n==> zs1: [2015-02-19T06:48:32+00:00] INFO: Chef Run complete in 28.265871351 seconds\r\n```\r\n\r\nChef **12.0.3**, don't see any log, but both gems are installed:\r\n\r\n```\r\n==> zs1: [2015-02-19T06:53:56+00:00] INFO: *** Chef 12.0.3 ***\r\n==> zs1: [2015-02-19T06:53:56+00:00] INFO: Chef-client pid: 14938\r\n==> zs1: [2015-02-19T06:54:24+00:00] INFO: Chef Run complete in 20.955133524 seconds\r\n```\r\n\r\n```\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep chef\r\nchef (12.0.3)\r\nchef-sugar (2.5.0)\r\nchef-zero (3.2.1)\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep aws\r\naws-sdk (2.0.26)\r\naws-sdk-core (2.0.26)\r\naws-sdk-resources (2.0.26)\r\n```\r\n\r\nChef **11.16.4**, don't see any log, but both gems are installed:\r\n\r\n```\r\n==> zs1: [2015-02-19T06:56:38+00:00] INFO: *** Chef 11.16.4 ***\r\n==> zs1: [2015-02-19T06:56:38+00:00] INFO: Chef-client pid: 15433\r\n==> zs1: /var/chef/cache/cookbooks/homebrew/libraries/homebrew_package.rb:26: warning: toplevel constant Homebrew referenced by Chef::Provider::Package::Homebrew\r\n==> zs1: [2015-02-19T06:57:47+00:00] INFO: Chef Run complete in 25.402963436 seconds\r\n```\r\n\r\n```\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep chef\r\nchef (11.16.4)\r\nchef-sugar (2.5.0)\r\nchef-zero (2.2)\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep aws\r\naws-sdk (2.0.26)\r\naws-sdk-core (2.0.26)\r\naws-sdk-resources (2.0.26)\r\n```\r\n\r\nChef **11.18.6**, don't see any log, but both gems are installed:\r\n\r\n```\r\n==> zs1: [2015-02-19T06:58:47+00:00] INFO: *** Chef 11.18.6 ***\r\n==> zs1: [2015-02-19T06:58:47+00:00] INFO: Chef-client pid: 16297\r\n==> zs1: /var/chef/cache/cookbooks/homebrew/libraries/homebrew_package.rb:26: warning: toplevel constant Homebrew referenced by Chef::Provider::Package::Homebrew\r\n==> zs1: [2015-02-19T06:59:22+00:00] INFO: Chef Run complete in 27.925290915 seconds\r\n```\r\n\r\n```\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep chef\r\nchef (11.18.6)\r\nchef-sugar (2.5.0)\r\nchef-zero (2.2.1)\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep aws\r\naws-sdk (2.0.26)\r\naws-sdk-core (2.0.26)\r\naws-sdk-resources (2.0.26)\r\n```\r\n\r\nI also tried with `chef_12.2.0-alpha.0-1_amd64.deb` (which does not support `compile_time`):\r\n\r\n```\r\n==> zs1: [2015-02-19T07:00:55+00:00] INFO: *** Chef 12.2.0.alpha.0 ***\r\n==> zs1: [2015-02-19T07:00:55+00:00] INFO: Chef-client pid: 16759\r\n==> zs1: [2015-02-19T07:01:41+00:00] INFO: Starting audit phase\r\n==> zs1: [2015-02-19T07:01:42+00:00] INFO: Successfully executed all controls blocks and contained examples\r\n==> zs1:\r\n==> zs1: Finished in 0.00018 seconds (files took 0.35235 seconds to load)\r\n==> zs1: 0 examples, 0 failures\r\n==> zs1: [2015-02-19T07:01:42+00:00] INFO: Chef Run complete in 39.759534584 seconds\r\n```\r\n\r\n```\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep chef\r\nchef (12.2.0.alpha.0)\r\nchef-sugar (2.5.0)\r\nchef-zero (3.2)\r\nroot@mesosphere-scalp-vagrant-vbox-1:~# /opt/chef/embedded/bin/gem list | grep aws\r\naws-sdk (2.0.26)\r\naws-sdk-core (2.0.26)\r\naws-sdk-resources (2.0.26)\r\n```\r\n\r\nI'm mostly breaking \"something\" as I don't see the resource being listed in the log but hopefully it helps."},"comment":{"url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/comments/84089564","html_url":"https://github.com/opscode-cookbooks/aws/pull/110#issuecomment-84089564","issue_url":"https://api.github.com/repos/opscode-cookbooks/aws/issues/110","id":84089564,"user":{"login":"scalp42","id":1475276,"avatar_url":"https://avatars.githubusercontent.com/u/1475276?v=3","gravatar_id":"","url":"https://api.github.com/users/scalp42","html_url":"https://github.com/scalp42","followers_url":"https://api.github.com/users/scalp42/followers","following_url":"https://api.github.com/users/scalp42/following{/other_user}","gists_url":"https://api.github.com/users/scalp42/gists{/gist_id}","starred_url":"https://api.github.com/users/scalp42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scalp42/subscriptions","organizations_url":"https://api.github.com/users/scalp42/orgs","repos_url":"https://api.github.com/users/scalp42/repos","events_url":"https://api.github.com/users/scalp42/events{/privacy}","received_events_url":"https://api.github.com/users/scalp42/received_events","type":"User","site_admin":false},"created_at":"2015-03-20T18:01:40Z","updated_at":"2015-03-20T18:01:40Z","body":"Thanks @sethvargo for reconsidering it."}}