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

优化kubernetes去掉nodeSelector,使用POD反亲和性(podAntiAffinity)将同一个服务的POD分散在不同的主机或者拓扑域中,提高服务本身的稳定性。 #2398

Merged
merged 2 commits into from
Jul 7, 2019

Conversation

Hwting
Copy link
Contributor

@Hwting Hwting commented Jul 1, 2019

nodeSelector只能提供简单的将pod限制为具有特定标签的节点,而podAntiAffinity将同一个服务的POD分散在不同的主机或者拓扑域中,提高服务本身的稳定性。
官方文档:https://kubernetes.io/docs/concepts/configuration/assign-pod-node/

@codecov-io
Copy link

Codecov Report

Merging #2398 into master will increase coverage by 7.93%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2398      +/-   ##
============================================
+ Coverage     50.21%   58.15%   +7.93%     
+ Complexity     2027     1162     -865     
============================================
  Files           407      196     -211     
  Lines         12457     5943    -6514     
  Branches       1278      636     -642     
============================================
- Hits           6255     3456    -2799     
+ Misses         5756     2223    -3533     
+ Partials        446      264     -182
Impacted Files Coverage Δ Complexity Δ
...work/apollo/biz/message/DatabaseMessageSender.java 56.25% <0%> (-10.42%) 6% <0%> (-2%)
.../apollo/internals/RemoteConfigLongPollService.java 79.01% <0%> (-1.24%) 27% <0%> (-1%)
.../portal/spi/configuration/LdapGroupProperties.java
...ework/apollo/portal/controller/ItemController.java
...o/configservice/wrapper/DeferredResultWrapper.java
...omponent/emailbuilder/GrayPublishEmailBuilder.java
...k/apollo/portal/controller/FavoriteController.java
...apollo/portal/entity/vo/AppRolesAssignedUsers.java
...ponent/emailbuilder/NormalPublishEmailBuilder.java
.../framework/apollo/portal/entity/vo/SystemInfo.java
... and 203 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58e5181...478e45c. Read the comment docs.

Copy link
Member

@nobodyiam nobodyiam left a comment

Choose a reason for hiding this comment

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

LGTM

@nobodyiam nobodyiam merged commit 900cb28 into apolloconfig:master Jul 7, 2019
@nobodyiam nobodyiam added this to the 1.5.0 milestone Aug 10, 2019
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