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

install istio with helm #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cjsrycjsyr16
Copy link
Contributor

@cjsrycjsyr16 cjsrycjsyr16 commented Nov 16, 2021

Description

Please provide a description of this PR

Affects

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

  • Configuration Infrastructure
  • Docs
  • Installation
  • Performance and Scalability
  • Test and Release
  • User Experience

memory: 128Mi
env:
- name: WATCH_NAMESPACE
value: "hango-system"
Copy link
Member

Choose a reason for hiding this comment

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

变量可以提取到values中

@ethanhanjiahao
Copy link
Member

serviceAccountName: istio-operator
containers:
- name: istio-operator
image: docker.io/istio/operator:1.10.2
Copy link
Member

Choose a reason for hiding this comment

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

image 可以提取到values中

Comment on lines +1 to +42
stages:
- package

before_script:
- CHART_VERSION="${CI_COMMIT_TAG:-v0.0.0-r$CI_PIPELINE_ID}"

variables:
NAMESPACE: istio
RELEASE_NAME: istio

helm_package:
stage: package
variables:
REPO_NAME: cloudnative
KUBECONFIG_YAML: $KUBECONFIG_DEV
script:
- mkdir -p target
- helm package --version="$CHART_VERSION" -d target . || exit 1
- echo "$CHART_VERSION"
- |
# helm upgrade -n $NAMESPACE $RELEASE_NAME . --install --create-namespace --dry-run

REPO_ADDR="$(jq -r '.repo//empty'<<<"$ONLINE_CHART_REPO")"
USERNAME="$(jq -r '.username//empty'<<<"$ONLINE_CHART_REPO")"
PASSWORD="$(jq -r '.password//empty'<<<"$ONLINE_CHART_REPO")"

helm repo add --insecure-skip-tls-verify --username="$USERNAME" \
--password="$PASSWORD" "$REPO_NAME" "$REPO_ADDR/chartrepo/$REPO_NAME" --force-update
helm push --insecure target/$(cd target; ls | grep tgz) "$REPO_NAME" || exit 1
- |
REPO_ADDR="$(jq -r '.repo//empty'<<<"$ONLINE_CHART_REPO")"
USERNAME="$(jq -r '.username//empty'<<<"$ONLINE_CHART_REPO")"
PASSWORD="$(jq -r '.password//empty'<<<"$ONLINE_CHART_REPO")"

helm repo add --insecure-skip-tls-verify --username="$USERNAME" \
--password="$PASSWORD" "$REPO_NAME" "$REPO_ADDR/chartrepo/$REPO_NAME" --force-update
helm push --insecure target/$(cd target; ls | grep tgz) "$REPO_NAME" || exit 1

only:
- develop
- tags

Copy link
Member

Choose a reason for hiding this comment

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

I think this is unnecessary for this PR

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