-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Remove secrets encryption controller #10612
Conversation
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.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.
couple minor nits, lgtm otherwise!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #10612 +/- ##
==========================================
- Coverage 49.52% 43.68% -5.84%
==========================================
Files 179 178 -1
Lines 14955 14890 -65
==========================================
- Hits 7406 6505 -901
- Misses 6193 7189 +996
+ Partials 1356 1196 -160
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Derek Nola <derek.nola@suse.com>
Signed-off-by: Derek Nola <derek.nola@suse.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.
one more nit on error events
Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
* Remove secrets encryption controller Signed-off-by: Derek Nola <derek.nola@suse.com>
Proposed Changes
Types of Changes
Code simplification
Verification
CI still green
Testing
This is covered by existing E2E and integration tests
Linked Issues
#10581
User-Facing Change
Further Comments