sacloud / terraform-provider-sakuracloud

Terraform provider for SakuraCloud
https://docs.usacloud.jp/terraform
Apache License 2.0
71 stars 23 forks source link

シンプル監視のポート番号設定が plan 時に正しく読み取られない #466

Closed tmy closed 5 years ago

tmy commented 5 years ago

terraform plan 時に毎回差分が発生してしまっています。

バージョン

Terraform v0.12.3
+ provider.sakuracloud v1.14.1

設定例

resource "sakuracloud_simple_monitor" "test" {
  target = "alt1.gmail-smtp-in.l.google.com"
  health_check {
    protocol = "smtp"
    port     = 25
  }
  description = "テスト"
}

Plan 実行時に発生する差分

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # sakuracloud_simple_monitor.test will be updated in-place
  ~ resource "sakuracloud_simple_monitor" "test" {
        description          = "テスト"
        enabled              = true
        id                   = "113101093464"
        notify_email_enabled = true
        notify_email_html    = false
        notify_slack_enabled = false
        tags                 = []
        target               = "alt1.gmail-smtp-in.l.google.com"

      ~ health_check {
            delay_loop     = 60
          ~ port           = 0 -> 25
            protocol       = "smtp"
            remaining_days = 30
            sni            = false
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

------------------------------------------------------------------------
yamamoto-febc commented 5 years ago

シンプル監視でのポート番号はヘルスチェックで用いるプロトコルごとにデフォルト値を持っています。

tfファイル上のportにてデフォルト値と同じ値を明示的に指定している場合にplan時の差分が発生してしまいます。本来はどの値を指定しても差分が出るべきではないと思いますので修正を行います。

FYI: 暫定対応としてはtfファイル上でportをコメントアウトすることで差分が出なくなると思います。

tmy commented 5 years ago

暫定対応としてはtfファイル上でportをコメントアウトすることで差分が出なくなると思います。

手元の環境で試してみたところ、そのとおりの挙動になりました。 ありがとうございます。

yamamoto-febc commented 5 years ago

467 にて対応し、対応バージョンである v1.14.2をリリースしました。

https://github.com/sacloud/terraform-provider-sakuracloud/releases/tag/v1.14.2

もし問題が再発するようであれば再オープンしてください。

ご報告ありがとうございました!