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

logs use up a lot of space on drone #34684

Open
phil-davis opened this issue Mar 5, 2019 · 3 comments
Open

logs use up a lot of space on drone #34684

phil-davis opened this issue Mar 5, 2019 · 3 comments

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Mar 5, 2019

  1. The log level for the owncloud.log when running acceptance tests is currently set to 0 debug
  2. selenium outputs information logs as each acceptance test scenario ends and a new one starts. That is because we "isolate" the scenarios, so selenium resets "itself/the browser". We do not need to see that log output.
  3. apache is logging all GET POST etc access requests. That makes a lot of log output that is being saved to the drone server.

This is generating a lot of log output. But normally nobody looks at it because CI passes, or the acceptance test fail has an understandable/useful message in the Behat test runner output.

If someone has a hard-to-diagnose test fail, and they can't reproduce it locally, they can always add a commit to their PR to temporarily change the logging in drone and collect lots more log messages.

@ownclouders
Copy link
Contributor

GitMate.io thinks possibly related issues are #18151 (files_versions taking up a lot of space), #30275 (Use updated drone/codecov plugin), #34281 (Confusing notice about used space and limit), #32299 (Use --remote for acceptance tests run on drone), and #3831 (wrong space displayed when use a encryption).

@phil-davis
Copy link
Contributor Author

Points (1) and (2) are addressed by PR #34685

@phil-davis
Copy link
Contributor Author

phil-davis commented Mar 6, 2019

Point (3) - Apache access logs:

I suggest that by default we do not show the access logs. But make it "easy" via some switch var in drone to pass the fact that you want the access logs to come to stdout.

Reason: 99% of the time nobody looks at this, or needs to look at it. The test scenarios pass. If it is needed, then it can be enabled in a temporary commit in a PR.

Solution: Technically the docker container could be enhanced to have this option. Or we could try to do some scripting in drone itself to override the default access log output.

Opinions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants