Please gzip the firmware/fwts_desktop_diagnosis_results.log attachment

Bug #1460095 reported by Daniel Manrique
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
High
Sylvain Pineau

Bug Description

The job mentioned above produces a huge attachment. In a real-world submission it resulted in a 5MB xml file (usually submissions are around 1MB). Of this, 4.3 MB were from this attachment.

The job definition is as follows:

plugin: attachment
estimated_duration: 0.5
id: firmware/fwts_desktop_diagnosis_results.log
command:
 [ -e ${PLAINBOX_SESSION_SHARE}/fwts_desktop_diagnosis_results.log ] && cat ${PLAINBOX_SESSION_SHARE}/fwts_desktop_diagnosis_results.log
_description: Attaches the FWTS desktop diagnosis results log to the submission
_summary: Attach FWTS desktop diagnosis log to submission

a possibility would be

plugin: attachment
estimated_duration: 0.5
id: firmware/fwts_desktop_diagnosis_results.log
command:
 [ -e ${PLAINBOX_SESSION_SHARE}/fwts_desktop_diagnosis_results.log ] && gzip -c ${PLAINBOX_SESSION_SHARE}/fwts_desktop_diagnosis_results.log | base64
_description: Attaches the FWTS desktop diagnosis results log to the submission
_summary: Attach FWTS desktop diagnosis log to submission

In this case, the attachment was reduced to about 400KB so it's much more manageable (though still quite big).

Related branches

Changed in plainbox-provider-checkbox:
milestone: none → 0.21
importance: Undecided → Medium
status: New → Triaged
importance: Medium → High
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Note: Needless to base64 the command as all binary attachments are already converted to base64 by plainbox during xml export.

Changed in plainbox-provider-checkbox:
assignee: nobody → Sylvain Pineau (sylvain-pineau)
status: Triaged → In Progress
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.