ROX-34562: Delete the declarative config reconciliation from fleetshard-sync#2696
Conversation
| }) | ||
| } | ||
| return &declarativeconfig.AuthProvider{ | ||
| Name: authProviderName(remoteCentral), |
There was a problem hiding this comment.
By removing this authProviderName in pkg/central/reconciler/util.go became unused and can be deleted.
| }) | ||
| } | ||
|
|
||
| func TestGetAuditLogNotifierConfig(t *testing.T) { |
There was a problem hiding this comment.
Lots of test, testing the old code removed, which is expected.
Is there a test that make sure the authProviderClientCredentialsSecret gets actually created, if not I think that would be good to add.
There was a problem hiding this comment.
Tests for authProviderClientCredentialsSecret added, thanks
|
1000 LoC removed, awesome 🔥 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johannes94, kovayur The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
declarativeConfig.enabledin FSS toconfig.WantsAuthProviderChecklist (Definition of Done)
Test manualROX-12345: ...Test manual
TODO: Add manual testing efforts