Merge lp://staging/~psivaa/uci-engine/lander-jenkins-plugin into lp://staging/uci-engine/mthood

Proposed by Para Siva
Status: Superseded
Proposed branch: lp://staging/~psivaa/uci-engine/lander-jenkins-plugin
Merge into: lp://staging/uci-engine/mthood
Diff against target: 163 lines (+63/-36)
2 files modified
charms/precise/lander-jenkins/hooks/hooks.py (+61/-34)
juju-deployer/lander.yaml.tmpl (+2/-2)
To merge this branch: bzr merge lp://staging/~psivaa/uci-engine/lander-jenkins-plugin
Reviewer Review Type Date Requested Status
Francis Ginther Needs Fixing
Review via email: mp+219018@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2014-05-16.

Commit message

Change to include parameterized-trigger plugin inside the charm instead of downloading it from the jenkins site to be made suitable for private mthood cloud

Note: For this to work well https://code.launchpad.net/~psivaa/charms/precise/jenkins/bundle-jenkins/+merge/215635 needs merging first

Description of the change

Change to include parameterized-trigger plugin inside the charm instead of downloading it from the jenkins site to be made suitable for private mthood cloud

Note: For this to work well https://code.launchpad.net/~psivaa/charms/precise/jenkins/bundle-jenkins/+merge/215635 needs merging first

To post a comment you must log in.
415. By Para Siva

Include mthood specific jenkins charm branch

Revision history for this message
Francis Ginther (fginther) wrote :

I don't see how the plugin gets copied from the charm local directory to the jenkins plugin directory (see inline comment).

review: Needs Fixing

Unmerged revisions

415. By Para Siva

Include mthood specific jenkins charm branch

414. By Para Siva

plugin name format in yaml

413. By Para Siva

Plugins included in lander-jenkins

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