Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

new mcollective script for running mco commands from hubot #1419

Closed
wants to merge 1 commit into from

Conversation

pzim
Copy link

@pzim pzim commented Apr 17, 2014

Appreciate if you would consider merging mcollective.coffee into the repo. Thanks!
-Phil

@technicalpickles
Copy link
Contributor

We are actually moving away from adding scripts to repository in favor of separate npm packages per scripts. See #1113 for details, and let us know if you have any questions about getting going with that!


mco_host = process.env.HUBOT_PUPPET_MCOLLECTIVE_HOST
mco_cmd = process.env.HUBOT_PUPPET_MCOLLECTIVE_CMD
mco_user = "peadmin"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be worth making this configurable, but with a default of peadmin?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.comwrote:

In src/scripts/mcollective.coffee:

+# mco puppet status
+# mco help
+#
+# Notes:
+# Requires root access (NOPASSWD) to the mcollective host
+# Feel free to modify for your environment and security constraints
+#
+# Author:
+# pzim
+{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 GitHubhttps://github.com//pull/1419/files#r11956506
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants