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

feat(controllers): Implement the TerraformRun controller #167

Merged
merged 23 commits into from
Oct 12, 2023

Conversation

corrieriluca
Copy link
Member

@corrieriluca corrieriluca commented Sep 18, 2023

Fixes #165

Checklist

  • TerraformRun controller fully functional
  • Support for a default maxRetries value in the config
  • Update documentation
  • Update Helm chart and manifests
  • [WIP in PR feat(tfrun): Add cleanup of past TerraformRuns #168] Implement a TerraformRun "garbage collector" (i.e. delete after a certain time/number)

@codecov-commenter
Copy link

codecov-commenter commented Sep 18, 2023

Codecov Report

Merging #167 (4d70a9d) into main (c0abbca) will increase coverage by 1.46%.
The diff coverage is 78.12%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
+ Coverage   69.04%   70.50%   +1.46%     
==========================================
  Files          23       28       +5     
  Lines        1586     1926     +340     
==========================================
+ Hits         1095     1358     +263     
- Misses        425      492      +67     
- Partials       66       76      +10     
Files Coverage Δ
api/v1alpha1/common.go 100.00% <ø> (ø)
internal/annotations/annotations.go 100.00% <ø> (ø)
internal/controllers/terraformlayer/conditions.go 62.31% <ø> (-2.79%) ⬇️
internal/controllers/terraformlayer/controller.go 38.80% <100.00%> (-8.82%) ⬇️
internal/controllers/terraformlayer/run.go 100.00% <100.00%> (ø)
internal/controllers/terraformpullrequest/layer.go 84.00% <100.00%> (+0.43%) ⬆️
internal/lock/lock.go 93.02% <100.00%> (+1.84%) ⬆️
internal/burrito/config/config.go 48.75% <0.00%> (-0.62%) ⬇️
internal/controllers/terraformlayer/states.go 77.41% <73.33%> (+9.28%) ⬆️
internal/controllers/terraformrun/pod.go 92.18% <82.35%> (ø)
... and 5 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@corrieriluca corrieriluca force-pushed the feat/terraformrun branch 3 times, most recently from bbf492a to 20ca3cb Compare September 21, 2023 14:13
@corrieriluca corrieriluca marked this pull request as ready for review October 9, 2023 10:00
@corrieriluca corrieriluca requested a review from Alan-pad October 9, 2023 10:01
@Alan-pad Alan-pad merged commit 93a6cd3 into main Oct 12, 2023
6 checks passed
@corrieriluca corrieriluca deleted the feat/terraformrun branch October 19, 2023 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement a dedicated controller to keep track of Terraform plan / apply
3 participants