OK!  You're also welcome to make a second PR for the convergence strategy
:)  Whatever works best for you.


On Thu, Mar 13, 2014 at 9:22 AM, Curtis Stewart <notifications@github.com>wrote:

> In lib/chef_metal/convergence_strategy/bootstrap.rb:
>
> > +module ChefMetal
> > +  class ConvergenceStrategy
> > +    class Bootstrap < PrecreateChefObjects
> > +      @@install_sh_cache = {}
> > +
> > +      def initialize(options = {})
> > +        @install_sh_url = options[:install_sh_url] || 'http://www.opscode.com/chef/install.sh'
> > +        @install_sh_path = options[:install_sh_path] || '/tmp/chef-install.sh'
> > +        @client_rb_path ||= '/etc/chef/client.rb'
> > +        @client_pem_path ||= '/etc/chef/client.pem'
> > +      end
> > +
> > +      attr_reader :install_sh_url
> > +      attr_reader :install_sh_path
> > +
> > +      def setup_convergence(provider, machine, machine_resource)
>
> I forgot this was a part of the PR. I'd like to make some changes before
> getting it merged.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/opscode/chef-metal/pull/15/files#r10572303>
> .
>