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

chore: support to define and reference the TLS #8538

Merged
merged 6 commits into from
Dec 2, 2024
Merged

Conversation

leon-inf
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the size/XXL Denotes a PR that changes 1000+ lines. label Nov 27, 2024
@leon-inf leon-inf marked this pull request as ready for review November 28, 2024 07:20
@leon-inf leon-inf requested review from Y-Rookie and a team as code owners November 28, 2024 07:20
Copy link

codecov bot commented Nov 28, 2024

Codecov Report

Attention: Patch coverage is 36.78930% with 189 lines in your changes missing coverage. Please review.

Project coverage is 60.95%. Comparing base (cadccb3) to head (e862fdd).
Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
...ntrollers/apps/transformer_cluster_sharding_tls.go 0.00% 112 Missing ⚠️
controllers/apps/transformer_component_tls.go 60.16% 33 Missing and 14 partials ⚠️
pkg/controller/component/vars.go 24.13% 22 Missing ⚠️
...llers/apps/transformer_cluster_sharding_account.go 0.00% 5 Missing ⚠️
pkg/controller/plan/tls.go 91.17% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8538      +/-   ##
==========================================
+ Coverage   60.73%   60.95%   +0.21%     
==========================================
  Files         376      377       +1     
  Lines       45480    46546    +1066     
==========================================
+ Hits        27622    28371     +749     
- Misses      15288    15561     +273     
- Partials     2570     2614      +44     
Flag Coverage Δ
unittests 60.95% <36.78%> (+0.21%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@apecloud-bot apecloud-bot added the approved PR Approved Test label Nov 29, 2024
}

if common.IsCompactMode(transCtx.Cluster.Annotations) {
transCtx.V(1).Info("Cluster is in compact mode, no need to create account related objects", "cluster", client.ObjectKeyFromObject(transCtx.Cluster))
Copy link
Collaborator

Choose a reason for hiding this comment

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

fix typo xxx create account xxxx

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@apecloud-bot apecloud-bot removed the approved PR Approved Test label Dec 2, 2024
@leon-inf
Copy link
Contributor Author

leon-inf commented Dec 2, 2024

/approve

@apecloud-bot apecloud-bot added the approved PR Approved Test label Dec 2, 2024
@leon-inf leon-inf merged commit 2d3deb5 into main Dec 2, 2024
36 checks passed
@leon-inf leon-inf deleted the support/tls-and-sshd branch December 2, 2024 06:28
@leon-inf
Copy link
Contributor Author

leon-inf commented Dec 2, 2024

/cherry-pick release-1.0-beta

@github-actions github-actions bot added this to the Release 0.9.2 milestone Dec 2, 2024
Copy link

github-actions bot commented Dec 2, 2024

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/apecloud/kubeblocks/actions/runs/12113898173

github-actions bot pushed a commit that referenced this pull request Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved PR Approved Test size/XXL Denotes a PR that changes 1000+ lines.
Projects
None yet
4 participants