jenkins-launchpad-plugin

Merge lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds into lp://qastaging/jenkins-launchpad-plugin

  1. triggered-builds
  2. Merge into trunk
Proposed by Alexandros Frantzis
Status: Merged
Approved by: Michał Sawicz
Approved revision: 133
Merged at revision: 129
Proposed branch: lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds
Merge into: lp://qastaging/jenkins-launchpad-plugin
Diff against target: 244 lines (+130/-3)
3 files modified
jlp/jenkinsutils.py (+57/-1)
tests/__init__.py (+56/-1)
tests/test_jenkinsutils.py (+17/-1)
To merge this branch: bzr merge lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds
Reviewer Review Type Date Requested Status
Michał Sawicz Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+284117@code.qastaging.launchpad.net

Commit message

Include triggered builds in reported downstream builds when creating MP vote message.

This is required because new versions of the triggered build plugin don't report triggered builds as downstream.

Description of the change

Include triggered builds in reported downstream builds when creating MP vote message.

This is required because new versions of the triggered build plugin don't report triggered builds as downstream.

To test locally run the following script from within the project directory:

import jlp.jenkinsutils
import sys

print(jlp.jenkinsutils.get_executed_test_runs_message(sys.argv[1]))

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:129
http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-ci/48/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-amd64-ci/17/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-i386-ci/13/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/jenkins-launchpad-plugin-ci/48/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds updated
130. By Alexandros Frantzis

Fix PEP8 complaints

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:130
http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-ci/49/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-amd64-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-i386-ci/14/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/jenkins-launchpad-plugin-ci/49/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds updated
131. By Alexandros Frantzis

More PEP8 fixes for trusty

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:131
http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-ci/50/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-amd64-ci/19
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-i386-ci/15

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/jenkins-launchpad-plugin-ci/50/rebuild

review: Approve (continuous-integration)
lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds updated
132. By Alexandros Frantzis

Fix variable initialization

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:132
http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-ci/52/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-amd64-ci/21
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-i386-ci/17

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/jenkins-launchpad-plugin-ci/52/rebuild

review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) :
review: Needs Fixing
lp://qastaging/~afrantzis/jenkins-launchpad-plugin/triggered-builds updated
133. By Alexandros Frantzis

Added comment for upstream project check

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:133
http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-ci/54/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-amd64-ci/23
    SUCCESS: http://jenkins.qa.ubuntu.com/job/jenkins-launchpad-plugin-trusty-i386-ci/19

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/jenkins-launchpad-plugin-ci/54/rebuild

review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Replying to self: set() could break ordering, so using lists is better here.

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