Ubuntu CI Services

Merge lp://staging/~cjohnston/ubuntu-ci-services-itself/temp-remove-tmpfs into lp://staging/~canonical-ci-engineering/ubuntu-ci-services-itself/tarmac-config

  1. temp-remove-tmpfs
  2. Merge into tarmac-config
Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 9
Merged at revision: 9
Proposed branch: lp://staging/~cjohnston/ubuntu-ci-services-itself/temp-remove-tmpfs
Merge into: lp://staging/~canonical-ci-engineering/ubuntu-ci-services-itself/tarmac-config
Diff against target: 14 lines (+3/-1)
1 file modified
tarmac.conf (+3/-1)
To merge this branch: bzr merge lp://staging/~cjohnston/ubuntu-ci-services-itself/temp-remove-tmpfs
Reviewer Review Type Date Requested Status
Vincent Ladeuil (community) Approve
Review via email: mp+222199@code.staging.launchpad.net

Commit message

Disable tmpfs for uci-engine to test tarmac broken pipe fix

Description of the change

Tarmac was hitting a 'broken pipe' error when attempting to run on lp:uci-engine.. We implemented a workaround to use TMPFS for running the tests in order to speed things up to get them under 5 minutes... Tarmac now has an attempted fix, so we need to test it. Let's disable TMPFS to have a chance to test the fixes. After the fix has been tested, let's re-enable TMPFS because speed == good!

To post a comment you must log in.
Revision history for this message
Vincent Ladeuil (vila) wrote :

As in: you will deploy https://code.launchpad.net/~dobey/tarmac/close-conn/+merge/221977 and test that even if our tests take longer than 5 minutes tarmac doesn't fail right ?

review: Approve

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