[WIP] Do not display log message about a resource when it will not be updated #74
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| @@ -108,9 +108,23 @@ def initialize(recipe, resource_name, &block) | |||
| end | |||
|
|
|||
| def run(specific_action = nil, options = {}) | |||
| Logger.info "#{resource_type}[#{resource_name}]" | |||
| Logger.debug "#{resource_type}[#{resource_name}]" | |||
There was a problem hiding this comment.
log level が DEBUG の時には、対象のリソースを必ず表示します。
| current_value && | ||
| @attributes[key] && | ||
| ! current_value.nil? && | ||
| ! @attributes[key].nil? && | ||
| current_value != @attributes[key] |
There was a problem hiding this comment.
プルリクの本題とは関係ないのですが、current_value が false で、@attributes[key] が true といった場合に、変更があるため true を返して欲しいのに、false を返してしまうため、nil かどうかでチェックを行うように修正しました。
There was a problem hiding this comment.
この変更はLGTMです。nitpickですが!の後のスペースなくしてもらえると〜
|
|
||
| Logger.formatter.indent do | ||
| Logger.debug "(in set_current_attributes)" | ||
| set_current_attributes |
There was a problem hiding this comment.
アクションを実行した後はcurrent attributeが変わっている可能性があるので、run_actionで実行する必要があります
|
メモ。 口頭で話した結果、処理の流れを大きく変えるのは厳しい、かといって変えないとログの出力を抑えるのも難しい、なので出力は抑えずに、色を変えたり、info で出しているものを debug にする、といった方向で行くことに。Chef のログが割と見やすいので参考にする。 |
|
クローズします。レビューとアドバイスありがとうございました！ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
リソースに何も変更がなかった場合には、ログに何も表示しないようにするための修正です（まだ途中です）。
このプルリク適用前の状態で、既に httpd が running かつ enabled な状態で、action [:start, :enable] すると、次のようにログが表示されます。
プルリク適用後は、以下のようになります。