Skip to content

Conversation

lixd
Copy link
Contributor

@lixd lixd commented Mar 22, 2023

What type of PR is this?

/kind feature

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:


Signed-off-by: lixd <xueduan.li@gmail.com>
@kubeclipper-bot kubeclipper-bot added kind/feature Categorizes issue or PR as related to a new feature. dco-signoff: yes do-not-merge/release-note-label-needed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. release-note-none and removed do-not-merge/release-note-label-needed labels Mar 22, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #566 (89b3275) into master (7c82bda) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #566   +/-   ##
=======================================
  Coverage   14.11%   14.11%           
=======================================
  Files         138      138           
  Lines       19394    19394           
=======================================
  Hits         2738     2738           
  Misses      16360    16360           
  Partials      296      296           

@x893675
Copy link
Collaborator

x893675 commented Mar 22, 2023

/lgtm
/approve

@kubeclipper-bot kubeclipper-bot added the lgtm Indicates that a PR is ready to be merged. label Mar 22, 2023
@kubeclipper-bot
Copy link
Collaborator

LGTM label has been added.

Git tree hash: 8c79761fc36042950cd7479cc49efeee04432963

@kubeclipper-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lixd, x893675

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeclipper-bot kubeclipper-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 22, 2023
@kubeclipper-bot kubeclipper-bot merged commit fbe6970 into kubeclipper:master Mar 22, 2023
Metrora pushed a commit to Metrora/kubeclipper that referenced this pull request Mar 24, 2023
Signed-off-by: lixd <xueduan.li@gmail.com>
kubeclipper-bot pushed a commit that referenced this pull request Mar 28, 2023
* feat: update k8s apiserver cert SANs when update externalIP
fix: k8s api endpoint support domain

Signed-off-by: Liucw <liu.chuwei@99cloud.net>

* fix: k8s api endpoint support domain (#566)

Signed-off-by: lixd <xueduan.li@gmail.com>

---------

Signed-off-by: Liucw <liu.chuwei@99cloud.net>
Signed-off-by: lixd <xueduan.li@gmail.com>
Co-authored-by: Liucw <liu.chuwei@99cloud.net>
Co-authored-by: 意琦行 <31980412+lixd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note-none size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants