Yes, please remove the nonfunctional code.  Thanks.

From: Wayne Allen [mailto:notifications@github.com]
Sent: Monday, March 24, 2014 9:18 AM
To: opencrowbar/core
Cc: Hirschfeld, Rob
Subject: Re: [core] Define roles for stand-alone puppet agent installation (#90)


In rails/app/models/barclamp_puppet/jig.rb:

> +      Rails.logger.info("Chefjig: Need to add #{n.role.name} to run list for #{nr.node.name}")

> +      runlist << "role[#{n.role.name}]"

> +    end

> +    runlist << "role[#{nr.role.name}]"

> +    Rails.logger.info("Chefjig: discovered run list: #{runlist}")

> +    Chef::RunList.new(*runlist)

> +  end

> +

> +  def stage_run(nr)

> +    return {

> +      :runlist => make_run_list(nr),

> +      :data => super(nr)

> +    }

> +  end

> +

> +  def run(nr,data)

this is just copied from chef jig but hasn't been reworked, yet. I can delete the incomplete files if you prefer...

—
Reply to this email directly or view it on GitHub<https://github.com/opencrowbar/core/pull/90/files#r10885722>.