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

Null_resource lifecycle is not in sync with it's contents #345

Open
1 task
TechnicallyWilliams opened this issue Oct 1, 2019 · 0 comments
Open
1 task
Labels
bug Something isn't working pri-Low Low priority issue tech-debt Work to resolve technical debt rather than implement new features or fix bugs

Comments

@TechnicallyWilliams
Copy link
Contributor

TechnicallyWilliams commented Oct 1, 2019

Background

null_resource gets created even if it's local-exec command line fails. Continuous failures create new null_resource references in the state file and will not clean up previously created null_resources references in the state file. Continuous local-exec failures of a null_resource will result in a ever expanding state file.

Description

As a user, I'd like the null_resource to be destroyed during current deployment when local-exec command fails in order to prevent pollution of state file.

Acceptance Criteria

Reference: [Done-Done Checklist] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/DoneDone.md)

  • Should delete null_resource references when local-exec commands fail.

Also, here are a few points that need to be addressed:

  1. Successful local-exec commands clean up all previous null-resource references that live within the state file that were created as a result of continuous local-exec failures.

Resources

image

https://www.terraform.io/docs/provisioners/local-exec.html

Tasks

Stories are intended to be completed in a single sprint; if task breakdown creates addition work then team should discuss promoting the Story to an Epic.
Reference: [Minimal Valuable Slices] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/MinimalSlices.md)

Reference: [How to Write Better Tasks] (http://agilebutpragmatic.blogspot.com/2012/04/splitting-story-into-tasks-how-to-write.html)

Assignee should break down work into tasks here

  1. Investigate ways to prevent continuous creation of null_resource references upon consecutive local-exec failures
@TechnicallyWilliams TechnicallyWilliams added bug Something isn't working pri-Low Low priority issue tech-debt Work to resolve technical debt rather than implement new features or fix bugs labels Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pri-Low Low priority issue tech-debt Work to resolve technical debt rather than implement new features or fix bugs
Projects
None yet
Development

No branches or pull requests

1 participant