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

bugfix: log index cache got overwritten when multi esurls exists in erda cluster #2887

Merged

Conversation

snakorse
Copy link
Contributor

@snakorse snakorse commented Nov 5, 2021

What type of this PR

Add one of the following kinds:
/kind bugfix

What this PR does / why we need it:

bugfix: log index cache got overwritten when multi esurls exists in erda cluster

Specified Reviewers:

/assign @liuhaoyang @recallsong

ChangeLog

Language Changelog
🇺🇸 English Fix the bug that log index cache got overwritten when multi esurls exists in erda cluster
🇨🇳 中文 修复同一集群中存在多个日志分析es时预加载日志索引缓存数据被相互覆盖的问题

Need cherry-pick to release versions?

/cherry-pick release/1.4

@snakorse snakorse added msp microsevice platform bugfix labels Nov 5, 2021
@erda-bot
Copy link
Member

erda-bot commented Nov 5, 2021

Automated cherry pick can ONLY be triggered when this PR is MERGED!

@codecov
Copy link

codecov bot commented Nov 5, 2021

Codecov Report

Merging #2887 (cc0afa6) into master (00f3b38) will increase coverage by 0.01%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2887      +/-   ##
==========================================
+ Coverage   17.23%   17.24%   +0.01%     
==========================================
  Files        1295     1295              
  Lines      131855   131869      +14     
==========================================
+ Hits        22719    22743      +24     
+ Misses     106785   106755      -30     
- Partials     2351     2371      +20     
Impacted Files Coverage Δ
modules/extensions/loghub/index/query/clients.go 41.02% <0.00%> (-0.27%) ⬇️
modules/extensions/loghub/index/query/index.go 27.27% <85.71%> (+2.90%) ⬆️
modules/cmp/cache/cache.go 54.54% <0.00%> (+3.67%) ⬆️

@liuhaoyang liuhaoyang merged commit 8d6ee3a into erda-project:master Nov 5, 2021
@snakorse
Copy link
Contributor Author

snakorse commented Nov 5, 2021

/cherry-pick release/1.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix msp microsevice platform
Development

Successfully merging this pull request may close these issues.

3 participants