Skip to content

[ci] Avoid schedule event upload too many files to codecov#29

Merged
zhongjiajie merged 1 commit into
apache:mainfrom
zhongjiajie:f-ci-codecov-too-many-upload
Nov 25, 2022
Merged

[ci] Avoid schedule event upload too many files to codecov#29
zhongjiajie merged 1 commit into
apache:mainfrom
zhongjiajie:f-ci-codecov-too-many-upload

Conversation

@zhongjiajie

@zhongjiajie zhongjiajie commented Nov 25, 2022

Copy link
Copy Markdown
Member

codecov have 100 limit in each commit, and then if will failed the upload process, so we should not upload during schedule, because 6 day not change code base will reach the limit. see more detail in https://community.codecov.com/t/ci-failure-due-to-too-many-uploads-to-this-commit/2587/7

Brief Summary of The Change

Pull Request checklist

I confirm that the following checklist has been completed.

  • Add/Change test cases for the changes.
  • Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
  • (Optional) Add your change to UPDATING.md when it is an incompatible change.

codecov have 100 limit in each commit, and then if will failed the
upload process, so we should not upload during schedule, because
6 day not change code base will reach the limit. see more detail
in https://community.codecov.com/t/ci-failure-due-to-too-many-uploads-to-this-commit/2587/7
@zhongjiajie zhongjiajie self-assigned this Nov 25, 2022
@zhongjiajie zhongjiajie added the chore A collection of chore like CI, remove not used code label Nov 25, 2022
@codecov-commenter

codecov-commenter commented Nov 25, 2022

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.68%. Comparing base (20b2a2b) to head (e0aad7f).
⚠️ Report is 84 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ Coverage   90.60%   90.68%   +0.08%     
==========================================
  Files          56       59       +3     
  Lines        2032     2072      +40     
==========================================
+ Hits         1841     1879      +38     
- Misses        191      193       +2     
Flag Coverage Δ
unittests 90.68% <ø> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zhongjiajie zhongjiajie merged commit 79afa8b into apache:main Nov 25, 2022
@zhongjiajie zhongjiajie deleted the f-ci-codecov-too-many-upload branch November 25, 2022 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A collection of chore like CI, remove not used code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants