tests: clean journalctl logs on trusty #3432

Merged
merged 4 commits into from Jun 7, 2017

Conversation

Projects
None yet
5 participants
Contributor

sergiocazzolato commented Jun 2, 2017

This is a way to reduce the size of the journalctl logs on ubuntu 14.04.
This is needed to improve the quality of the tests bacause some of them are using journalctl to check that the actions are done.
I also will help to make more readable the logs when a test fail.

sergiocazzolato added some commits Jun 2, 2017

Reducing the size of the journalctl logs for ubuntu 14.04
This is a way to reduce the size of the journalctl logs on ubuntu 14.04.

This is needed to improve the quality of the tests bacause some of them
are using journalctl to check that the actions are done.

I also will help to make more readable the logs when a test fail.
spread.yaml
+ systemctl kill --kill-who=main --signal=SIGUSR2 systemd-journald.service
+
+ #Remove rotated journal logs
+ systemctl stop systemd-journald.service
@zyga

zyga Jun 5, 2017

Contributor

Last time I checked there was no journald on 14.04, has that changed since?

@sergiocazzolato

sergiocazzolato Jun 5, 2017

Contributor

Well, I tested that manually in a vm and then in linode with ubuntu 14.04 and it is working, I ran the suite and I was monitoring the logs and it was cleaned for every test.

Codecov Report

Merging #3432 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3432      +/-   ##
==========================================
- Coverage   77.56%   77.55%   -0.01%     
==========================================
  Files         371      371              
  Lines       25519    25519              
==========================================
- Hits        19793    19792       -1     
- Misses       3976     3977       +1     
  Partials     1750     1750
Impacted Files Coverage Δ
interfaces/sorting.go 90% <0%> (-3.34%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d455a61...c05ef23. Read the comment docs.

mvo5 approved these changes Jun 6, 2017

Looks good, thanks for this!

Thanks!

@mvo5 mvo5 merged commit 2b9d740 into snapcore:master Jun 7, 2017

7 checks passed

artful-amd64 autopkgtest finished (success)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
xenial-amd64 autopkgtest finished (success)
Details
xenial-i386 autopkgtest finished (success)
Details
xenial-ppc64el autopkgtest finished (success)
Details
yakkety-amd64 autopkgtest finished (success)
Details
zesty-amd64 autopkgtest finished (success)
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment