Next Generation Checkbox (CLI)

Merge ~sylvain-pineau/checkbox-ng:last_line_buildstamp into checkbox-ng:master

  1. Git
  2. lp:~sylvain-pineau/checkbox-ng
  3. last_line_buildstamp
  4. Merge into master
Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 29d341eee2a4d1635af699a7abda321a29a1c733
Merged at revision: 2106108c5025134021e7ae26b0212a11b61752a1
Proposed branch: ~sylvain-pineau/checkbox-ng:last_line_buildstamp
Merge into: checkbox-ng:master
Diff against target: 16 lines (+1/-5)
1 file modified
plainbox/impl/providers/exporters/data/checkbox.json (+1/-5)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Patrick Liu (community) Approve
Review via email: mp+400091@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Patrick Liu (patliu) wrote :

+1

review: Approve
Revision history for this message
Jonathan Cave (jocave) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/plainbox/impl/providers/exporters/data/checkbox.json b/plainbox/impl/providers/exporters/data/checkbox.json
2index a9f8771..5b4cff8 100644
3--- a/plainbox/impl/providers/exporters/data/checkbox.json
4+++ b/plainbox/impl/providers/exporters/data/checkbox.json
5@@ -175,10 +175,6 @@
6 {%- endif %}
7 {%- if ns ~ 'info/buildstamp' in state.job_state_map and state.job_state_map[ns ~ 'info/buildstamp'].result.outcome == 'pass' %},
8 {%- set buildstamp = state.job_state_map[ns ~ 'info/buildstamp'].result.io_log_as_text_attachment.rstrip().splitlines() %}
9-{%- if buildstamp|length >= 2 %}
10- "buildstamp": {{ buildstamp[1] | jsonify | safe }}
11-{%- else %}
12- "buildstamp": ""
13-{%- endif %}
14+ "buildstamp": {{ buildstamp[-1] | jsonify | safe }}
15 {%- endif %}
16 }

Subscribers

People subscribed via source and target branches