karmada icon indicating copy to clipboard operation
karmada copied to clipboard

local up install karmada by helm

Open chaosi-zju opened this issue 1 year ago • 6 comments

What type of PR is this?

/kind feature

What this PR does / why we need it:

  • A script using just one command to build multi-clusters by karmada helm chart
  • A CI test to install karmada by helm chart

This script can also execute on macOS

Which issue(s) this PR fixes:

Fixes #4305

Special notes for your reviewer:

  • if you already has clusters in real environment, please modify hack/local-up-karmada-helm.sh as:
# NEED_CREATE_KIND_CLUSTER customize whether you need to create clusters by kind, if you have clusters already, please unset this option
NEED_CREATE_KIND_CLUSTER=false

# LOAD_IMAGE_IN_ADVANCE if you fetch images in advance and you are using KinD clusters, may be you'd like to load it to KinD in advance too
LOAD_IMAGE_IN_ADVANCE=false

# IMAGE_FROM customize whether you need fetch images in advance, if you don't need, left empty
IMAGE_FROM=""

then, provided your given cluster name and kubeconfig just like:

KARMADA_HOST_NAME="karmada-host"
KARMADA_HOST_KUBECONFIG="${HOME}/.kube/karmada.config"

declare -A PUSH_MODE_MEMBERS
PUSH_MODE_MEMBERS["member1"]="${HOME}/.kube/member1.config"
PUSH_MODE_MEMBERS["member3"]="${HOME}/.kube/member3.config"

declare -A PULL_MODE_MEMBERS
PULL_MODE_MEMBERS["member2"]="${HOME}/.kube/member2.config"
  • if you need we to help you create several kind clusters, keep unchanged

Execute the script:

hack/local-up-karmada-helm.sh

You will see:

image

Does this PR introduce a user-facing change?:

none

chaosi-zju avatar Sep 07 '23 13:09 chaosi-zju

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign rainbowmango after the PR has been reviewed. You can assign the PR to them by writing /assign @rainbowmango in a comment when ready.

The full list of commands accepted by this bot can be found 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

karmada-bot avatar Sep 07 '23 13:09 karmada-bot

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e2c6ece) 51.90% compared to head (97eb5d2) 51.89%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4045      +/-   ##
==========================================
- Coverage   51.90%   51.89%   -0.01%     
==========================================
  Files         243      243              
  Lines       24146    24146              
==========================================
- Hits        12532    12531       -1     
- Misses      10931    10932       +1     
  Partials      683      683              
Flag Coverage Δ
unittests 51.89% <ø> (-0.01%) :arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Sep 07 '23 14:09 codecov-commenter

@RainbowMango comments all resolved~

chaosi-zju avatar Sep 20 '23 12:09 chaosi-zju

@jwcesign have a look again~

chaosi-zju avatar Nov 23 '23 02:11 chaosi-zju

CC @RainbowMango ready to merge

by this PR, we can install current code by helm method in CI

chaosi-zju avatar Dec 11 '23 06:12 chaosi-zju