Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warn when ignore_changes includes a Computed attribute #30517

Merged
merged 5 commits into from Feb 18, 2022

Commits on Feb 14, 2022

  1. ignore_changes attributes must exist in schema

    Add a test verifying that attempting to add a nonexistent attribute to
    ignore_changes throws an error.
    kmoe committed Feb 14, 2022
    Copy the full SHA
    b4e64cc View commit details
    Browse the repository at this point in the history
  2. ignore_changes cannot be used with Computed attrs

    Return a warning if a Computed attribute is present in ignore_changes,
    unless the attribute is also Optional.
    
    ignore_changes on a non-Optional Computed attribute is a no-op, so the user
    likely did not want to set this in config.
    An Optional Computed attribute, however, is still subject to ignore_changes
    behaviour, since it is possible to make changes in the configuration that
    Terraform must ignore.
    kmoe committed Feb 14, 2022
    Copy the full SHA
    0da22d7 View commit details
    Browse the repository at this point in the history
  3. add traversalToPath func

    kmoe committed Feb 14, 2022
    Copy the full SHA
    d6cc3d9 View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2022

  1. Copy the full SHA
    4e3507f View commit details
    Browse the repository at this point in the history
  2. clarify error message

    kmoe committed Feb 17, 2022
    Copy the full SHA
    006e14c View commit details
    Browse the repository at this point in the history