Finally getting back to this - yes, good catch - that should totally be
configurable.


On Thu, Apr 24, 2014 at 10:08 AM, Josh Nichols <notifications@github.com>wrote:

> In src/scripts/mcollective.coffee:
>
> > +#   mco puppet status <regex for group of hosts>
> > +#   mco help
> > +#
> > +# Notes:
> > +#   Requires root access (NOPASSWD) to the mcollective host
> > +#   Feel free to modify for your environment and security constraints
> > +#
> > +# Author:
> > +#   pzim <Phil Zimmerman>
> > +{spawn, exec}  = require 'child_process'
> > +module.exports = (robot) ->
> > +
> > +
> > +  mco_host = process.env.HUBOT_PUPPET_MCOLLECTIVE_HOST
> > +  mco_cmd  = process.env.HUBOT_PUPPET_MCOLLECTIVE_CMD
> > +  mco_user = "peadmin"
>
> Would it be worth making this configurable, but with a default of peadmin?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/github/hubot-scripts/pull/1419/files#r11956506>
> .
>