Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reset initialization_options and vendor_class after a chef_run #841

Merged
merged 1 commit into from
May 18, 2016

Conversation

ryancragun
Copy link
Contributor

@ryancragun ryancragun commented May 18, 2016

With the chef-client change from solo mode to zero with solo legacy mode the
chef_run resource no longer properly configured and reset the FileVendor class
variables. This causes the templates rendered in the outer chef runs to fail
when trying to download their files. This change tracks the outer run variables
and resets them after the sub runs have completed.

Recipe: private-chef::plugin_chef_run
  * chef_run[recipe[test::enable]] action run[2016-05-18T20:18:21+00:00] WARN: Class Chef::Provider::ChefRun does not declare 'provides :chef_run'.
[2016-05-18T20:18:21+00:00] WARN: This will no longer work in Chef 13: you must use 'provides' to use the resource's DSL.
Installing Cookbook Gems:
Compiling Cookbooks...
Converging 1 resources
Recipe: test::enable
  * template[/tmp/works] action create
    - create new file /tmp/works
    - update content in file /tmp/works from none to 7d865e
    --- /tmp/works      2016-05-18 20:18:21.974159738 +0000
    +++ /tmp/.chef-works20160518-6651-z53916    2016-05-18 20:18:21.974159738 +0000
    @@ -1 +1,2 @@
    +bar
    - change mode from '' to '0655'
    - change owner from '' to 'root'
    - change group from '' to 'root'

    - Running chef-run with run list recipe[test::enable]

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
With the chef-client change from solo mode to zero with solo legacy mode the
chef_run resource no longer properly configured and reset the FileVendor class
variables. This causes the templates rendered in the outer chef runs to fail
when trying to download their files. This change tracks the outer run variables
and resets them after the sub runs have completed.
@ryancragun ryancragun force-pushed the ryan/fix_vendoring branch from ad6fc69 to 41af9dc Compare May 18, 2016 20:23
@ryancragun ryancragun merged commit a241d0d into master May 18, 2016
@ryancragun ryancragun deleted the ryan/fix_vendoring branch May 18, 2016 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants