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.

Sign up for GitHub

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

Jump to bottom

Build and deploy optimize #115

Merged
merged 5 commits into from
Nov 30, 2021
Merged

Build and deploy optimize #115

merged 5 commits into from
Nov 30, 2021

Conversation

NigelWu95
Copy link

@NigelWu95 NigelWu95 commented Nov 25, 2021 •
edited
Loading

Describe what this PR does / why we need it

  1. add network setting in operator deployment.
  2. optimize operator image build without operator-sdk(0.19.0) dependency.
  3. fix chaosblade-operator build_yaml in chaosblade release build_all.
    image
    image

Does this pull request fix one issue?

Fixes #111

Describe how you did it

update Makefile and helm charts.

Describe how to verify it

image

Special notes for reviews

秋旭 added 5 commits November 25, 2021 17:26
chore: add spec.hostNetwork and spec.dnsPolicy in operator deployment…
df3d9c8 
….yaml and support set value by helm values.yaml.

Signed-off-by: 秋旭 <bingheng.wbh@antgroup.com>
chore: update operator image build approach, new operator-sdk can not…
6f0a0c5 
… use subcommand: build as it has many changes different from old version: 0.19.0.

Signed-off-by: 秋旭 <bingheng.wbh@antgroup.com>
chore: fix build_yaml, build/cache/chaosblade/yaml/chaosblade-jvm-spe…
420faba 
…c-.yaml is required.

Signed-off-by: 秋旭 <bingheng.wbh@antgroup.com>
chore: update default dnsPolicy to ClusterFirst for operator deployme…
6fdbdba 
…nt in helm charts.

Signed-off-by: 秋旭 <bingheng.wbh@antgroup.com>
chore: fix docker build for linux exec binary and change the runtime …
dc7d0c2 
…image to centos so that conveniently using some command like

ps etc. in operator container.

Signed-off-by: 秋旭 <bingheng.wbh@antgroup.com>
@xcaspar xcaspar self-requested a review November 30, 2021 11:41
@xcaspar xcaspar added the type/enhancement New feature or request label Nov 30, 2021
@xcaspar xcaspar added this to the v1.4.0 milestone Nov 30, 2021
@xcaspar xcaspar merged commit 1a70eda into chaosblade-io:master Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xcaspar xcaspar xcaspar approved these changes

Assignees
No one assigned
Labels
type/enhancement New feature or request
Projects
None yet
Milestone
v1.4.0
Development

Successfully merging this pull request may close these issues.

can not make build_all
2 participants
@NigelWu95 @xcaspar