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

workflows: Add expire to tests reports #2511

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

vvarg229
Copy link
Contributor

@vvarg229 vvarg229 commented Aug 15, 2023

Add the expiration period for Allure test reports. Set the expiration period to 1 month for master, 2 weeks for pull requests and manual runs, no expiration for releases, and 1 week for other events.

First, we need to merge this PR:
nspcc-dev/neofs-testcases#601
and
add these repository variables to the neofs-node repo:

MASTER_EXPIRATION_PERIOD = 744
PR_EXPIRATION_PERIOD = 336
MANUAL_RUN_EXPIRATION_PERIOD = 336
OTHER_EXPIRATION_PERIOD = 168

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #2511 (0e02457) into master (86c5d3b) will increase coverage by 0.39%.
Report is 34 commits behind head on master.
The diff coverage is 63.09%.

❗ Current head 0e02457 differs from pull request most recent head 8d7cdd3. Consider uploading reports for the commit 8d7cdd3 to get more accurate results

@@            Coverage Diff             @@
##           master    #2511      +/-   ##
==========================================
+ Coverage   29.30%   29.69%   +0.39%     
==========================================
  Files         399      404       +5     
  Lines       30347    30711     +364     
==========================================
+ Hits         8892     9119     +227     
- Misses      20719    20823     +104     
- Partials      736      769      +33     
Files Changed Coverage Δ
cmd/neofs-adm/internal/modules/morph/initialize.go 0.00% <0.00%> (ø)
cmd/neofs-node/config.go 0.00% <0.00%> (ø)
cmd/neofs-node/fs_unix.go 0.00% <0.00%> (ø)
...kg/local_object_storage/blobstor/common/storage.go 52.94% <52.94%> (ø)
pkg/local_object_storage/blobstor/peapod/peapod.go 72.54% <72.54%> (ø)
...kg/services/object_manager/storagegroup/collect.go 64.44% <86.66%> (ø)
cmd/neofs-node/validate.go 45.83% <100.00%> (ø)
pkg/local_object_storage/writecache/writecache.go 77.55% <100.00%> (+0.46%) ⬆️

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

.github/workflows/run-tests.yml Outdated Show resolved Hide resolved
.github/workflows/run-tests.yml Outdated Show resolved Hide resolved
@vvarg229 vvarg229 force-pushed the expire-allure branch 2 times, most recently from aea3ac1 to d8d2ac0 Compare August 18, 2023 12:08
.github/workflows/run-tests.yml Outdated Show resolved Hide resolved
.github/workflows/run-tests.yml Outdated Show resolved Hide resolved
Add the expiration period for Allure test reports. Set the expiration period to
1 month for master, 2 weeks for pull requests and manual runs, no expiration
for releases, and 1 week for other events.

Signed-off-by: Oleg Kulachenko <oleg@nspcc.ru>
@roman-khimov roman-khimov merged commit 86116f7 into nspcc-dev:master Aug 23, 2023
6 of 8 checks passed
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