Project

General

Profile

Feature #46059

vstart_runner.py: optionally rotate logs between tests

Added by Patrick Donnelly 7 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
Development
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

When running lots of tests with debugging, the size of the logs quickly reaches 10s of gigabytes. Have vstart_runner.py rotate logs between tests to get around this. This should be doable by running logrotate manually with a vstart specific log rotation config (perhaps generated by vstart.sh itself).


Related issues

Related to mgr - Bug #47032: mgr/dashboard: ceph api test failure: "Invalid URL 'None/api/auth'" (test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest)) Resolved
Related to mgr - Fix #47755: qa: rotate logs in dashboard API tests New

History

#1 Updated by Rishabh Dave 7 months ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 35824

#2 Updated by Patrick Donnelly 5 months ago

  • Status changed from Fix Under Review to Resolved

#3 Updated by Patrick Donnelly 5 months ago

  • Related to Bug #47032: mgr/dashboard: ceph api test failure: "Invalid URL 'None/api/auth'" (test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest)) added

#4 Updated by Patrick Donnelly 5 months ago

  • Status changed from Resolved to In Progress

Reverted by https://github.com/ceph/ceph/pull/36711 to fix api tests. Rishabh, please open a new PR.

#5 Updated by Rishabh Dave 5 months ago

  • Pull request ID changed from 35824 to 36732

#6 Updated by Rishabh Dave 5 months ago

  • Status changed from In Progress to Fix Under Review

#7 Updated by Rishabh Dave 4 months ago

Got some time to work on this finally. Fixed the PR after some scrutiny, ceph API tests pass for this PR now.

#8 Updated by Kefu Chai 4 months ago

  • Status changed from Fix Under Review to Resolved

#9 Updated by Patrick Donnelly 4 months ago

  • Related to Fix #47755: qa: rotate logs in dashboard API tests added

Also available in: Atom PDF