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

chore(monitor): set profile data 1day ttl #5857

Merged
merged 3 commits into from
May 15, 2023

Conversation

chengjoey
Copy link
Contributor

What this PR does / why we need it:

set profile data 1day ttl

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English set profile data 1day ttl
🇨🇳 中文 将可持续性观测数据的默认保留时间设置为默认1天

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot erda-bot requested a review from sfwn May 15, 2023 06:01
@chengjoey chengjoey changed the title chore/set profile data 1day ttl chore(monitor): set profile data 1day ttl May 15, 2023
@sfwn
Copy link
Member

sfwn commented May 15, 2023

/approve

@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Merging #5857 (ad30735) into master (db91955) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5857   +/-   ##
=======================================
  Coverage   22.19%   22.19%           
=======================================
  Files        1852     1851    -1     
  Lines      205256   205290   +34     
=======================================
+ Hits        45558    45570   +12     
- Misses     154460   154482   +22     
  Partials     5238     5238           

see 7 files with indirect coverage changes

@erda-bot erda-bot merged commit 445bd79 into erda-project:master May 15, 2023
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants