-
Notifications
You must be signed in to change notification settings - Fork 66
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: Keep the backend secret when clean up SubResources if the deleteResou… #381
Fix: Keep the backend secret when clean up SubResources if the deleteResou… #381
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #381 +/- ##
==========================================
- Coverage 79.31% 72.92% -6.40%
==========================================
Files 23 27 +4
Lines 1750 2068 +318
==========================================
+ Hits 1388 1508 +120
- Misses 278 471 +193
- Partials 84 89 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@@ -1569,6 +1572,16 @@ func TestTerraformDestroy(t *testing.T) { | |||
objects: []client.Object{readyProvider, configurationWithConnSecret, baseConfigurationCM, completeDestroyJob, baseVariableSecret, connectionSecret}, | |||
deletedResources: []client.Object{baseConfigurationCM, completeDestroyJob, baseVariableSecret, connectionSecret}, | |||
}, | |||
{ | |||
name: "destroy job has completes, cleanup resources but backend secret", |
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.
name: "destroy job has completes, cleanup resources but backend secret", | |
name: "destroy job has completes, cleanup resources except backend secret", |
}, | ||
want: want{}, | ||
objects: []client.Object{readyProvider, configurationWithConnSecret, baseConfigurationCM, completeDestroyJob, baseVariableSecret, connectionSecret}, | ||
deletedResources: []client.Object{baseConfigurationCM, completeDestroyJob, baseVariableSecret, connectionSecret}, |
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.
Backend secret should be added to keptResources
to test this PR. I think now this condition is not tested.
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.
I missed this test case about keptResources
, and I will add it.
…rces is false. Signed-off-by: imzhangsiwei <flysqrlboy@gmail.com>
…ce set to false Signed-off-by: imzhangsiwei <flysqrlboy@gmail.com>
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.
LGTM, Please sign the DCO.
8fb9882
to
2d33b98
Compare
…rces is false.
#380