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.

Sign up for GitHub

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: missing status update in KMP controller #1761

Merged
merged 1 commit into from
Sep 12, 2024

refactor: move writeKMProviderStatus to controller

7123a16
Select commit
Loading
Failed to load commit list.
Merged

fix: missing status update in KMP controller #1761

refactor: move writeKMProviderStatus to controller
7123a16
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Sep 12, 2024 in 1s

95.50% of diff hit (target 80.00%)

View this Pull Request on Codecov

95.50% of diff hit (target 80.00%)

Annotations

Check warning on line 36 in pkg/keymanagementprovider/mocks/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/client.go#L36 

Added line #L36 was not covered by tests

Check warning on line 43 in pkg/keymanagementprovider/mocks/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/client.go#L43 

Added line #L43 was not covered by tests

Check warning on line 55 in pkg/keymanagementprovider/mocks/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/client.go#L54-L55 

Added lines #L54 - L55 were not covered by tests

Check warning on line 59 in pkg/keymanagementprovider/mocks/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/client.go#L58-L59 

Added lines #L58 - L59 were not covered by tests

Check warning on line 64 in pkg/keymanagementprovider/mocks/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/client.go#L64 

Added line #L64 was not covered by tests

Check warning on line 54 in pkg/keymanagementprovider/mocks/types.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/keymanagementprovider/mocks/types.go#L54 

Added line #L54 was not covered by tests
View more details on Codecov