-
Notifications
You must be signed in to change notification settings - Fork 34
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
fix: change terraform file style from imperative to declarative #487
Conversation
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
e4908dd
to
3e06f8a
Compare
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
I believe the commit message is good to go. Please let me know if there's anything else I need to do. |
The sign off needs to be on its own line. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #487 +/- ##
==========================================
- Coverage 25.23% 19.96% -5.27%
==========================================
Files 195 264 +69
Lines 16908 23632 +6724
==========================================
+ Hits 4267 4719 +452
- Misses 12423 18699 +6276
+ Partials 218 214 -4 ☔ View full report in Codecov by Sentry. |
3e06f8a
to
89ba6eb
Compare
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
89ba6eb
to
85461be
Compare
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
85461be
to
6ad66d9
Compare
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
Ref: #482 Signed-off-by: Steve Wall <[email protected]>
6ad66d9
to
fa3e22a
Compare
@stevewallone - I've resolved the sign-off for you in fa3e22a. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All instances of the changes to create_duration
to duration
for the hashicorp/time
provider need to be reverted.
The argument is create_duration
.
…form-provider-tanzu-mission-control into declarative-terraform Signed-off-by: Steve Wall <[email protected]>
@stevewallone, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding
|
Closing PR. Will create another one that is clean. |
Summary
The terraform files use an imperative style, i.e. create_tkg_cluster. The convention is to use a declarative style that doesn't imply an actions, i.e. tkg_cluster.
Type
Please describe:
Breaking Changes?
Test and Documentation
Issue References
Resolves #482
Additional Information