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

PR status summary should remove Note: Objects have changed outside of Terraform #3009

Closed
nitrocode opened this issue Jan 19, 2023 · 1 comment · Fixed by #3010
Closed

PR status summary should remove Note: Objects have changed outside of Terraform #3009

nitrocode opened this issue Jan 19, 2023 · 1 comment · Fixed by #3010
Labels
bug Something isn't working
Milestone

Comments

@nitrocode
Copy link
Member

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Overview of the Issue

PR status summary should remove Note: Objects have changed outside of Terraform

Reproduction Steps

  1. find repo with attributes that have changed (not terraform)
  2. terraform 1.3.6, aws 4.50.0
  3. atlantis plan

Logs

Environment details

Note: Objects have changed outside of Terraform

Terraform detected the following changes made outside of Terraform since the
last "terraform apply" which may have affected this plan:

  # module.vpc.aws_eip.nat[0] has changed
! resource "aws_eip" "nat" {
        # ...
        # (8 unchanged attributes hidden)
    }

Additional Context

2023-01-18_19-06

@nitrocode nitrocode added the bug Something isn't working label Jan 19, 2023
@krrrr38
Copy link
Contributor

krrrr38 commented Jan 19, 2023

Summary doesn't return one line summary 😇

// Summary extracts one line summary of plan changes from TerraformOutput.
func (p *PlanSuccess) Summary() string {
note := ""
if match := reChangesOutside.FindString(p.TerraformOutput); match != "" {
note = "\n**" + match + "**\n"
}
if match := rePlanChanges.FindString(p.TerraformOutput); match != "" {
return note + match
}
return note + reNoChanges.FindString(p.TerraformOutput)
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants