~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/plugins/launchpad/lp_propose.py

  • Committer: Tarmac
  • Author(s): Vincent Ladeuil
  • Date: 2017-01-30 14:42:05 UTC
  • mfrom: (6620.1.1 trunk)
  • Revision ID: tarmac-20170130144205-r8fh2xpmiuxyozpv
Merge  2.7 into trunk including fix for bug #1657238 [r=vila]

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2010, 2011 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
from __future__ import absolute_import
 
18
 
 
19
from bzrlib import (
 
20
    errors,
 
21
    hooks,
 
22
    )
 
23
from bzrlib.lazy_import import lazy_import
 
24
lazy_import(globals(), """
 
25
import webbrowser
 
26
 
 
27
from bzrlib import (
 
28
    msgeditor,
 
29
    )
 
30
from bzrlib.i18n import gettext
 
31
from bzrlib.plugins.launchpad import (
 
32
    lp_api,
 
33
    lp_registration,
 
34
    )
 
35
""")
 
36
 
 
37
 
 
38
class ProposeMergeHooks(hooks.Hooks):
 
39
    """Hooks for proposing a merge on Launchpad."""
 
40
 
 
41
    def __init__(self):
 
42
        hooks.Hooks.__init__(self, "bzrlib.plugins.launchpad.lp_propose",
 
43
            "Proposer.hooks")
 
44
        self.add_hook('get_prerequisite',
 
45
            "Return the prerequisite branch for proposing as merge.", (2, 1))
 
46
        self.add_hook('merge_proposal_body',
 
47
            "Return an initial body for the merge proposal message.", (2, 1))
 
48
 
 
49
 
 
50
class Proposer(object):
 
51
 
 
52
    hooks = ProposeMergeHooks()
 
53
 
 
54
    def __init__(self, tree, source_branch, target_branch, message, reviews,
 
55
                 staging=False, approve=False, fixes=None):
 
56
        """Constructor.
 
57
 
 
58
        :param tree: The working tree for the source branch.
 
59
        :param source_branch: The branch to propose for merging.
 
60
        :param target_branch: The branch to merge into.
 
61
        :param message: The commit message to use.  (May be None.)
 
62
        :param reviews: A list of tuples of reviewer, review type.
 
63
        :param staging: If True, propose the merge against staging instead of
 
64
            production.
 
65
        :param approve: If True, mark the new proposal as approved immediately.
 
66
            This is useful when a project permits some things to be approved
 
67
            by the submitter (e.g. merges between release and deployment
 
68
            branches).
 
69
        """
 
70
        self.tree = tree
 
71
        if staging:
 
72
            lp_instance = 'staging'
 
73
        else:
 
74
            lp_instance = 'production'
 
75
        service = lp_registration.LaunchpadService(lp_instance=lp_instance)
 
76
        self.launchpad = lp_api.login(service)
 
77
        self.source_branch = lp_api.LaunchpadBranch.from_bzr(
 
78
            self.launchpad, source_branch)
 
79
        if target_branch is None:
 
80
            self.target_branch = self.source_branch.get_target()
 
81
        else:
 
82
            self.target_branch = lp_api.LaunchpadBranch.from_bzr(
 
83
                self.launchpad, target_branch)
 
84
        self.commit_message = message
 
85
        # XXX: this is where bug lp:583638 could be tackled.
 
86
        if reviews == []:
 
87
            self.reviews = []
 
88
        else:
 
89
            self.reviews = [(self.launchpad.people[reviewer], review_type)
 
90
                            for reviewer, review_type in
 
91
                            reviews]
 
92
        self.approve = approve
 
93
        self.fixes = fixes
 
94
 
 
95
    def get_comment(self, prerequisite_branch):
 
96
        """Determine the initial comment for the merge proposal."""
 
97
        if self.commit_message is not None:
 
98
            return self.commit_message.strip().encode('utf-8')
 
99
        info = ["Source: %s\n" % self.source_branch.lp.bzr_identity]
 
100
        info.append("Target: %s\n" % self.target_branch.lp.bzr_identity)
 
101
        if prerequisite_branch is not None:
 
102
            info.append("Prereq: %s\n" % prerequisite_branch.lp.bzr_identity)
 
103
        for rdata in self.reviews:
 
104
            uniquename = "%s (%s)" % (rdata[0].display_name, rdata[0].name)
 
105
            info.append('Reviewer: %s, type "%s"\n' % (uniquename, rdata[1]))
 
106
        self.source_branch.bzr.lock_read()
 
107
        try:
 
108
            self.target_branch.bzr.lock_read()
 
109
            try:
 
110
                body = self.get_initial_body()
 
111
            finally:
 
112
                self.target_branch.bzr.unlock()
 
113
        finally:
 
114
            self.source_branch.bzr.unlock()
 
115
        initial_comment = msgeditor.edit_commit_message(''.join(info),
 
116
                                                        start_message=body)
 
117
        return initial_comment.strip().encode('utf-8')
 
118
 
 
119
    def get_initial_body(self):
 
120
        """Get a body for the proposal for the user to modify.
 
121
 
 
122
        :return: a str or None.
 
123
        """
 
124
        def list_modified_files():
 
125
            lca_tree = self.source_branch.find_lca_tree(
 
126
                self.target_branch)
 
127
            source_tree = self.source_branch.bzr.basis_tree()
 
128
            files = modified_files(lca_tree, source_tree)
 
129
            return list(files)
 
130
        target_loc = ('bzr+ssh://bazaar.launchpad.net/%s' %
 
131
                       self.target_branch.lp.unique_name)
 
132
        body = None
 
133
        for hook in self.hooks['merge_proposal_body']:
 
134
            body = hook({
 
135
                'tree': self.tree,
 
136
                'target_branch': target_loc,
 
137
                'modified_files_callback': list_modified_files,
 
138
                'old_body': body,
 
139
            })
 
140
        return body
 
141
 
 
142
    def get_source_revid(self):
 
143
        """Get the revision ID of the source branch."""
 
144
        source_branch = self.source_branch.bzr
 
145
        source_branch.lock_read()
 
146
        try:
 
147
            return source_branch.last_revision()
 
148
        finally:
 
149
            source_branch.unlock()
 
150
 
 
151
    def check_proposal(self):
 
152
        """Check that the submission is sensible."""
 
153
        if self.source_branch.lp.self_link == self.target_branch.lp.self_link:
 
154
            raise errors.BzrCommandError(
 
155
                'Source and target branches must be different.')
 
156
        for mp in self.source_branch.lp.landing_targets:
 
157
            if mp.queue_status in ('Merged', 'Rejected'):
 
158
                continue
 
159
            if mp.target_branch.self_link == self.target_branch.lp.self_link:
 
160
                raise errors.BzrCommandError(gettext(
 
161
                    'There is already a branch merge proposal: %s') %
 
162
                    lp_api.canonical_url(mp))
 
163
 
 
164
    def _get_prerequisite_branch(self):
 
165
        hooks = self.hooks['get_prerequisite']
 
166
        prerequisite_branch = None
 
167
        for hook in hooks:
 
168
            prerequisite_branch = hook(
 
169
                {'launchpad': self.launchpad,
 
170
                 'source_branch': self.source_branch,
 
171
                 'target_branch': self.target_branch,
 
172
                 'prerequisite_branch': prerequisite_branch})
 
173
        return prerequisite_branch
 
174
 
 
175
    def call_webservice(self, call, *args, **kwargs):
 
176
        """Make a call to the webservice, wrapping failures.
 
177
        
 
178
        :param call: The call to make.
 
179
        :param *args: *args for the call.
 
180
        :param **kwargs: **kwargs for the call.
 
181
        :return: The result of calling call(*args, *kwargs).
 
182
        """
 
183
        from lazr.restfulclient import errors as restful_errors
 
184
        try:
 
185
            return call(*args, **kwargs)
 
186
        except restful_errors.HTTPError, e:
 
187
            error_lines = []
 
188
            for line in e.content.splitlines():
 
189
                if line.startswith('Traceback (most recent call last):'):
 
190
                    break
 
191
                error_lines.append(line)
 
192
            raise Exception(''.join(error_lines))
 
193
 
 
194
    def approve_proposal(self, mp):
 
195
        revid = self.get_source_revid()
 
196
        self.call_webservice(
 
197
            mp.createComment,
 
198
            vote=u'Approve',
 
199
            subject='', # Use the default subject.
 
200
            content=u"Rubberstamp! Proposer approves of own proposal.")
 
201
        self.call_webservice(mp.setStatus, status=u'Approved', revid=revid)
 
202
 
 
203
    def create_proposal(self):
 
204
        """Perform the submission."""
 
205
        prerequisite_branch = self._get_prerequisite_branch()
 
206
        if prerequisite_branch is None:
 
207
            prereq = None
 
208
        else:
 
209
            prereq = prerequisite_branch.lp
 
210
            prerequisite_branch.update_lp()
 
211
        self.source_branch.update_lp()
 
212
        reviewers = []
 
213
        review_types = []
 
214
        for reviewer, review_type in self.reviews:
 
215
            review_types.append(review_type)
 
216
            reviewers.append(reviewer.self_link)
 
217
        initial_comment = self.get_comment(prerequisite_branch)
 
218
        mp = self.call_webservice(
 
219
            self.source_branch.lp.createMergeProposal,
 
220
            target_branch=self.target_branch.lp,
 
221
            prerequisite_branch=prereq,
 
222
            initial_comment=initial_comment,
 
223
            commit_message=self.commit_message, reviewers=reviewers,
 
224
            review_types=review_types)
 
225
        if self.approve:
 
226
            self.approve_proposal(mp)
 
227
        if self.fixes:
 
228
            if self.fixes.startswith('lp:'):
 
229
                self.fixes = self.fixes[3:]
 
230
            self.call_webservice(
 
231
                self.source_branch.lp.linkBug,
 
232
                bug=self.launchpad.bugs[int(self.fixes)])
 
233
        webbrowser.open(lp_api.canonical_url(mp))
 
234
 
 
235
 
 
236
def modified_files(old_tree, new_tree):
 
237
    """Return a list of paths in the new tree with modified contents."""
 
238
    for f, (op, path), c, v, p, n, (ok, k), e in new_tree.iter_changes(
 
239
        old_tree):
 
240
        if c and k == 'file':
 
241
            yield str(path)