質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.01%

hubotでscriptをcronでまわすには・・?

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,706

twin_bird

score 222

hubot(slack)でcronをまわして定期的にslackの任意のチャンネルにshellの出力結果を吐き出したいのですが、coffee scriptの書き方が今ひとつわかりません。

module.exports = (robot) ->
  cronJob = new cronJob('*/10 * * * *', () ->
    envelope = room: "general"
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    msg.send "Command: #{command}"
    @exec command, (error, stdout, stderr) ->
      msg.send error if error?
      msg.send stdout if stdout?
      msg.send stderr if stderr?
  )
cronJob.start()

こんな感じでかいたみたのですが、実行されていないようでどこか間違っているような気がします。
アドバイスお願い致します。。

追記: coffee scriptを書き換えてみました。

cron = require('cron').CronJob

module.exports = (robot) ->
  new cron '*/1 * * * *', () =>
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    @exec command, (error, stdout, stderr) ->
      robot.send {room: "#general"}, "stdout"
  , null, true, "Asia/Tokyo"

これでshellの実行結果がslackに投稿されれば良いのですが、stdoutの部分が展開されず、文字列として表示されてしまいます。。。
(error,stdout,stderr)の部分がどういうものなのか理解しきれていないのですが、以下のようなコードの場合は動作するので、参考にしてみたもののも上手くいきません_| ̄|○

module.exports = (robot) ->
  robot.respond /artisan_up/, (msg) ->
    @exec = require('child_process').exec
    command = "/hoge_hubot/scripts/shell/hoge.sh"
    msg.send "Command: #{command}"
    @exec command, (error, stdout, stderr) ->
      msg.send error if error?
      msg.send stdout if stdout?
      msg.send stderr if stderr?

追記2:shell(hoge.sh)については以下のとおりです。

logwatch --print
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

check解決した方法

0

cron = require('cron').CronJob

module.exports = (robot) ->
  new cron '*/1 * * * *', () =>
    @exec = require('child_process').exec
    command = "/webyone_hubot/scripts/shell/logwatch.sh"
    @exec command, (error, stdout, stderr) ->
      robot.send {room: "#general"}, stdout
  , null, true, "Asia/Tokyo"

これで動きましたw
このコードが動くのではないかと最初踏んでいたのですが、なぜか動かず、その後試行錯誤して改めてこのコードを試してみると動きましたw
どこが原因だったか判明していませんが、これでクローズします。
コメント頂けた方、ありがとうございました!!

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

cronJob 自体についてはわかりませんが、CoffeeScript としておかしなところを指摘してみます。

cronJob = new cronJob('*/10 * * * *', () ->

変数の cronJob とクラス名の cronJob が被ってしまっています。そもそも cronJob クラスがどこから来たのか提示されたコードからだとよくわからないのですが、事前に読まれていたとしても上書きされてしまうのはまずい気がします。

cronJob.start()

この cronJob がクラスなのか 2 行目で作った変数なのかがわからないのですが、もし変数のつもりであるなら、2 行目で宣言された cronJob 変数は (robot) -> 関数内のローカル変数ですので、この位置からは参照できません。

そしておまけですが、@exec の @ は恐らく必要ないです。あっても動作に支障はないかとは思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/08/29 03:48

    回答ありがとうございます!もう少しシンプルな形に変更してみたのですが、変数?部分の展開が上手くいきません。。(質問欄参照)

    キャンセル

  • 2016/08/29 11:11

    `"stdout"` のようにクォートで囲えば当然 `stdout` と言う文字列そのものが送られます。変数 `stdout` を指定してみてください。

    キャンセル

  • 2016/08/29 13:56

    バックティック(`)で指定してみましたが、展開されないようです。。hubotはpm2でデーモンを永続化しているので、そちらのログを確認してみましたがそれらしきエラー等見当たりませんでした。。

    キャンセル

  • 2016/08/29 14:00

    すいません。バックティックは Markdown の記法のつもりで書きました(そしてコメントでは Markdown は使えませんでした…)。
    いずれにしても基本的な CoffeeScript の文法部分になるので、わからない場合は基礎の入門からやり直したほうが良いかと思います。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.01%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる