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.

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 simple refactor] Differentiate between kubuclient and certclient #1205

Merged
merged 6 commits into from
Jul 19, 2023
Merged

[fix simple refactor] Differentiate between kubuclient and certclient #1205

merged 6 commits into from
Jul 19, 2023

Conversation

sjmshsh
Copy link

@sjmshsh sjmshsh commented Jul 19, 2023

The concepts of kubeclient and certclient were a bit confusing before. This PR mainly separates them.

@sjmshsh sjmshsh changed the title Refactor with go components experimental fix simple refactor Differentiate between kubuclient and certclient Jul 19, 2023
@sjmshsh sjmshsh changed the title fix simple refactor Differentiate between kubuclient and certclient [fix simple refactor] Differentiate between kubuclient and certclient Jul 19, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2023

Codecov Report

Merging #1205 (cf91498) into refactor-with-go-components-experimental (887c510) will increase coverage by 0.21%.
The diff coverage is 30.43%.

@@                             Coverage Diff                              @@
##           refactor-with-go-components-experimental    #1205      +/-   ##
============================================================================
+ Coverage                                     48.99%   49.21%   +0.21%     
============================================================================
  Files                                           113      113              
  Lines                                          6043     6010      -33     
  Branches                                         41       41              
============================================================================
- Hits                                           2961     2958       -3     
+ Misses                                         2827     2797      -30     
  Partials                                        255      255              
Impacted Files Coverage Δ
pkg/core/cert/provider/client.go 0.00% <0.00%> (-1.39%) ⬇️
pkg/core/cert/provider/storage.go 4.67% <0.00%> (ø)
pkg/authority/server/authority.go 50.00% <50.00%> (ø)
pkg/core/election/kube/leaderelection.go 85.71% <75.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

"k8s.io/client-go/util/homedir"
)

type KubuClient interface {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo? Should it be KubeClient?

@sonarcloud
Copy link

sonarcloud bot commented Jul 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@chickenlj chickenlj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@chickenlj chickenlj merged commit d292375 into apache:refactor-with-go-components-experimental Jul 19, 2023
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants