Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

add default tolerations for addons-manager package #3560

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

ggpaue
Copy link
Contributor

@ggpaue ggpaue commented Oct 5, 2022

What this PR does / why we need it

add default tolerations for addons-manager package

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

Release note

add default tolerations for addons-manager package values.yaml

Additional information

Special notes for your reviewer

@ggpaue ggpaue requested review from a team as code owners October 5, 2022 20:58
@ggpaue ggpaue requested a review from shyaamsn October 5, 2022 20:58
@codecov
Copy link

codecov bot commented Oct 6, 2022

Codecov Report

Merging #3560 (50f0a22) into main (4f21910) will increase coverage by 0.93%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3560      +/-   ##
==========================================
+ Coverage   45.74%   46.67%   +0.93%     
==========================================
  Files         435      414      -21     
  Lines       42607    41162    -1445     
==========================================
- Hits        19489    19213     -276     
+ Misses      21420    20249    -1171     
- Partials     1698     1700       +2     
Impacted Files Coverage Δ
...ons/controllers/packageinstallstatus_controller.go 77.99% <0.00%> (-1.16%) ⬇️
cmd/cli/plugin/tkr/v1alpha3/os.go 73.50% <0.00%> (-0.86%) ⬇️
addons/controllers/clusterbootstrap_controller.go 63.26% <0.00%> (-0.60%) ⬇️
cmd/cli/plugin/cluster/set_machinehealthcheck.go
.../cli/plugin/cluster/set_machinehealthcheck_node.go
...md/cli/plugin/cluster/delete_machinehealthcheck.go
cmd/cli/plugin/cluster/create.go
.../cli/plugin/cluster/get_machinehealthcheck_node.go
cmd/cli/plugin/cluster/get_node_pools.go
cmd/cli/plugin/cluster/kubeconfig_get.go
... and 27 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vijaykatam vijaykatam added the ok-to-merge PRs should be labelled with this before merging label Oct 6, 2022
@ggpaue ggpaue merged commit 2153619 into main Oct 6, 2022
@ggpaue ggpaue deleted the gguanepng/addons-manager-tolerations branch October 6, 2022 22:20
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants