department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
281 stars 197 forks source link

[DevOps] Terraform Apply workflows should be skipped if Plan doesn't have changes. #86414

Closed ndouglas closed 1 month ago

ndouglas commented 2 months ago

Description

Just saves time.

Tasks

Acceptance Criteria

ndouglas commented 1 month ago

This is hard because Terragrunt doesn't support the detailed exit codes that Terraform can return. It's a nice-to-have but unless that changes upstream, there's unlikely to be much we can do about this that doesn't increase fragility.

ndouglas commented 1 month ago

Or we can just plan xor apply. No reason to do both in the same operation.