Merge ~jocave/checkbox/+git/support:limit-output-to-pastebin into ~checkbox-dev/checkbox/+git/support:master

Proposed by Jonathan Cave
Status: Merged
Approved by: Jonathan Cave
Approved revision: 142a57b3bc1da0949f0b7f2c642a01253b682430
Merged at revision: 52f46505867683e0662bfbe6e35292fc222d28eb
Proposed branch: ~jocave/checkbox/+git/support:limit-output-to-pastebin
Merge into: ~checkbox-dev/checkbox/+git/support:master
Diff against target: 12 lines (+1/-1)
1 file modified
test-in-lxc.sh (+1/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+401522@code.qastaging.launchpad.net

Description of the change

We've had merge fails with very large amounts of output and it seems to exceed some limit on the pastebin and we get no output as a result. Use tail to limit the amount sent.

Just guessed at a number of lines that would be useful.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

+1. Let's start with 600 and carefully monitor the next pmr runs.
Ideally if we could make the output of 'test in lxc' a bit quieter we could revert this patch. But as you pointed out let's first confirm xlsxwriter installation is no longer causing extra log entries.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches