• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

mozilla / relman-auto-nag / #5082

14 Jun 2024 02:51PM CUT coverage: 21.852% (-0.005%) from 21.857%
#5082

push

coveralls-python

benjaminmah
Moved `stackGraph` check to another function, replaced `.get()` with direct hash map access

716 of 3602 branches covered (19.88%)

0 of 11 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

1933 of 8846 relevant lines covered (21.85%)

0.22 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0
/bugbot/rules/not_landed.py
1
# This Source Code Form is subject to the terms of the Mozilla Public
2
# License, v. 2.0. If a copy of the MPL was not distributed with this file,
3
# You can obtain one at http://mozilla.org/MPL/2.0/.
4

5
import base64
×
6
import random
×
7
import re
×
8

9
from libmozdata import utils as lmdutils
×
10
from libmozdata.bugzilla import Bugzilla, BugzillaUser
×
11
from libmozdata.phabricator import (
×
12
    PhabricatorAPI,
13
    PhabricatorBzNotFoundException,
14
    PhabricatorRevisionNotFoundException,
15
)
16

17
from bugbot import utils
×
18
from bugbot.bzcleaner import BzCleaner
×
19

20
PHAB_URL_PAT = re.compile(r"https://phabricator\.services\.mozilla\.com/D([0-9]+)")
×
21

22

23
class NotLanded(BzCleaner):
×
24
    def __init__(self):
×
25
        super(NotLanded, self).__init__()
×
26
        self.nweeks = utils.get_config(self.name(), "number_of_weeks", 2)
×
27
        self.nyears = utils.get_config(self.name(), "number_of_years", 2)
×
28
        self.phab = PhabricatorAPI(utils.get_login_info()["phab_api_key"])
×
29
        self.extra_ni = {}
×
30

31
    def description(self):
×
32
        return "Open bugs with no activity for {} weeks and a r+ patch which hasn't landed".format(
×
33
            self.nweeks
34
        )
35

36
    def has_assignee(self):
×
37
        return True
×
38

39
    def get_extra_for_template(self):
×
40
        return {"nweeks": self.nweeks}
×
41

42
    def get_extra_for_needinfo_template(self):
×
43
        self.extra_ni.update(self.get_extra_for_template())
×
44
        return self.extra_ni
×
45

46
    def columns(self):
×
47
        return ["id", "summary", "assignee"]
×
48

49
    def handle_bug(self, bug, data):
×
50
        if self.has_bot_set_ni(bug):
×
51
            return None
×
52

53
        bugid = str(bug["id"])
×
54
        assignee = bug.get("assigned_to", "")
×
55
        if utils.is_no_assignee(assignee):
×
56
            assignee = ""
×
57
            nickname = ""
×
58
        else:
59
            nickname = bug["assigned_to_detail"]["nick"]
×
60

61
        data[bugid] = {
×
62
            "assigned_to": assignee,
63
            "nickname": nickname,
64
            "deps": set(bug["depends_on"]),
65
        }
66

67
        return bug
×
68

69
    def filter_bugs(self, bugs):
×
70
        # We must remove bugs which have open dependencies (except meta bugs)
71
        # because devs may wait for those bugs to be fixed before their patch
72
        # can land.
73

74
        all_deps = set(dep for info in bugs.values() for dep in info["deps"])
×
75

76
        def bug_handler(bug, data):
×
77
            if (
×
78
                bug["status"] in {"RESOLVED", "VERIFIED", "CLOSED"}
79
                or "meta" in bug["keywords"]
80
            ):
81
                data.add(bug["id"])
×
82

83
        useless = set()
×
84
        Bugzilla(
×
85
            bugids=list(all_deps),
86
            include_fields=["id", "keywords", "status"],
87
            bughandler=bug_handler,
88
            bugdata=useless,
89
        ).get_data().wait()
90

91
        for bugid, info in bugs.items():
×
92
            # finally deps will contain open bugs which are not meta
93
            info["deps"] -= useless
×
94

95
        # keep bugs with no deps
96
        bugs = {bugid: info for bugid, info in bugs.items() if not info["deps"]}
×
97

98
        return bugs
×
99

100
    def check_phab(self, attachment, reviewers_phid):
×
101
        """Check if the patch in Phabricator has been r+"""
102
        if attachment["is_obsolete"] == 1:
×
103
            return None
×
104

105
        phab_url = base64.b64decode(attachment["data"]).decode("utf-8")
×
106

107
        # extract the revision
108
        rev = PHAB_URL_PAT.search(phab_url).group(1)
×
109
        try:
×
110
            data = self.phab.load_revision(
×
111
                rev_id=int(rev), queryKey="all", attachments={"reviewers": 1}
112
            )
113
        except PhabricatorRevisionNotFoundException:
×
114
            return None
×
115

116
        # this is a timestamp
117
        last_modified = data["fields"]["dateModified"]
×
118
        last_modified = lmdutils.get_date_from_timestamp(last_modified)
×
119
        if (self.date - last_modified).days <= self.nweeks * 7:
×
120
            # Do not do anything if recent changes in the bug
121
            return False
×
122

123
        reviewers = data["attachments"]["reviewers"]["reviewers"]
×
124

125
        if not reviewers:
×
126
            return False
×
127

128
        for reviewer in reviewers:
×
129
            if reviewer["status"] != "accepted":
×
130
                return False
×
131
            reviewers_phid.add(reviewer["reviewerPHID"])
×
132

133
        value = data["fields"]["status"].get("value", "")
×
134
        if value == "changes-planned":
×
135
            # even if the patch is r+ and not published, some changes may be required
136
            # so with the value 'changes-planned', the dev can say it's still a wip
137
            return False
×
138

139
        if value != "published":
×
140
            return True
×
141

142
        return False
×
143

144
    def handle_attachment(self, attachment, res):
×
145
        ct = attachment["content_type"]
×
146
        c = None
×
147
        if ct == "text/x-phabricator-request":
×
148
            if "phab" not in res or res["phab"]:
×
149
                c = self.check_phab(attachment, res["reviewers_phid"])
×
150
                if c is not None:
×
151
                    res["phab"] = c
×
152

153
        if c is not None:
×
154
            attacher = attachment["creator"]
×
155
            if "author" in res:
×
156
                if attacher in res["author"]:
×
157
                    res["author"][attacher] += 1
×
158
                else:
159
                    res["author"][attacher] = 1
×
160
            else:
161
                res["author"] = {attacher: 1}
×
162

163
            if "count" in res:
×
164
                res["count"] += 1
×
165
            else:
166
                res["count"] = 1
×
167

168
    def get_patch_data(self, bugs):
×
169
        """Get patch information in bugs"""
170
        nightly_pat = Bugzilla.get_landing_patterns(channels=["nightly"])[0][0]
×
171

172
        def comment_handler(bug, bugid, data):
×
173
            # if a comment contains a backout: don't nag
174
            for comment in bug["comments"]:
×
175
                comment = comment["text"].lower()
×
176
                if nightly_pat.match(comment) and (
×
177
                    "backed out" in comment or "backout" in comment
178
                ):
179
                    data[bugid]["backout"] = True
×
180

181
        def attachment_id_handler(attachments, bugid, data):
×
182
            for a in attachments:
×
183
                if (
×
184
                    a["content_type"] == "text/x-phabricator-request"
185
                    and a["is_obsolete"] == 0
186
                ):
187
                    data.append(a["id"])
×
188

189
        def attachment_handler(attachments, data):
×
190
            for attachment in attachments:
×
191
                bugid = str(attachment["bug_id"])
×
192
                if bugid in data:
×
193
                    data[bugid].append(attachment)
×
194
                else:
195
                    data[bugid] = [attachment]
×
196

NEW
197
        def search_dependencies(attachment):
×
NEW
198
            rev = PHAB_URL_PAT.search(
×
199
                base64.b64decode(attachment["data"]).decode("utf-8")
200
            ).group(1)
NEW
201
            try:
×
NEW
202
                revision_data = self.phab.load_revision(rev_id=int(rev))
×
NEW
203
                stack_graph = revision_data["fields"]["stackGraph"]
×
NEW
204
                current_revision_phid = revision_data["phid"]
×
NEW
205
                dependencies = stack_graph[current_revision_phid]
×
NEW
206
                return bool(dependencies)
×
NEW
207
            except PhabricatorRevisionNotFoundException:
×
NEW
208
                return False
×
209

210
        bugids = list(bugs.keys())
×
211
        data = {
×
212
            bugid: {"backout": False, "author": None, "count": 0} for bugid in bugids
213
        }
214

215
        # Get the ids of the attachments of interest
216
        # to avoid to download images, videos, ...
217
        attachment_ids = []
×
218
        Bugzilla(
×
219
            bugids=bugids,
220
            attachmenthandler=attachment_id_handler,
221
            attachmentdata=attachment_ids,
222
            attachment_include_fields=["is_obsolete", "content_type", "id"],
223
        ).get_data().wait()
224

225
        # Once we've the ids we can get the data
226
        attachments_by_bug = {}
×
227
        Bugzilla(
×
228
            attachmentids=attachment_ids,
229
            attachmenthandler=attachment_handler,
230
            attachmentdata=attachments_by_bug,
231
            attachment_include_fields=[
232
                "bug_id",
233
                "data",
234
                "is_obsolete",
235
                "content_type",
236
                "id",
237
                "creator",
238
            ],
239
        ).get_data().wait()
240

241
        for bugid, attachments in attachments_by_bug.items():
×
242
            res = {"reviewers_phid": set()}
×
243
            for attachment in attachments:
×
244
                self.handle_attachment(attachment, res)
×
245

246
            if "phab" in res:
×
247
                if res["phab"]:
×
248
                    # Check for stackGraph dependencies
NEW
249
                    data[bugid]["dependencies"] = search_dependencies(attachment)
×
UNCOV
250
                    data[bugid]["reviewers_phid"] = res["reviewers_phid"]
×
251
                    data[bugid]["author"] = res["author"]
×
252
                    data[bugid]["count"] = res["count"]
×
253

254
        data = {bugid: v for bugid, v in data.items() if v["author"]}
×
255

256
        if not data:
×
257
            return data
×
258

259
        Bugzilla(
×
260
            bugids=list(data.keys()),
261
            commenthandler=comment_handler,
262
            commentdata=data,
263
            comment_include_fields=["text"],
264
        ).get_data().wait()
265

266
        data = {bugid: v for bugid, v in data.items() if not v["backout"]}
×
267

268
        return data
×
269

270
    def get_bz_userid(self, phids):
×
271
        if not phids:
×
272
            return {}
×
273

274
        try:
×
275
            data = self.phab.load_bz_account(user_phids=list(phids))
×
276
            users = {x["phid"]: x["id"] for x in data}
×
277
        except PhabricatorBzNotFoundException:
×
278
            return {}
×
279

280
        def handler(user, data):
×
281
            data[str(user["id"])] = user["name"]
×
282

283
        data = {}
×
284
        BugzillaUser(
×
285
            user_names=list(users.values()),
286
            include_fields=["id", "name"],
287
            user_handler=handler,
288
            user_data=data,
289
        ).wait()
290

291
        return {phid: data[id] for phid, id in users.items()}
×
292

293
    def get_nicks(self, nicknames):
×
294
        def handler(user, data):
×
295
            data[user["name"]] = user["nick"]
×
296

297
        users = set(nicknames.values())
×
298
        data = {}
×
299
        if users:
×
300
            BugzillaUser(
×
301
                user_names=list(users),
302
                include_fields=["name", "nick"],
303
                user_handler=handler,
304
                user_data=data,
305
            ).wait()
306

307
        for bugid, name in nicknames.items():
×
308
            nicknames[bugid] = (name, data[name])
×
309

310
        return nicknames
×
311

312
    def get_bz_params(self, date):
×
313
        self.date = lmdutils.get_date_ymd(date)
×
314
        fields = ["flags", "depends_on"]
×
315
        params = {
×
316
            "include_fields": fields,
317
            "resolution": "---",
318
            "f1": "attachment.ispatch",
319
            "n2": 1,
320
            "f2": "attachments.isobsolete",
321
            "f3": "attachments.mimetype",
322
            "o3": "anywordssubstr",
323
            "v3": "text/x-phabricator-request",
324
            "f4": "creation_ts",
325
            "o4": "greaterthan",
326
            "v4": f"-{self.nyears}y",
327
            "f5": "days_elapsed",
328
            "o5": "greaterthaneq",
329
            "v5": self.nweeks * 7,
330
            "n6": 1,
331
            "f6": "longdesc",
332
            "o6": "casesubstring",
333
            "v6": "which didn't land and no activity in this bug for",
334
            "f7": "status_whiteboard",
335
            "o7": "notsubstring",
336
            "v7": "[reminder-test ",
337
        }
338

339
        return params
×
340

341
    def get_bugs(self, date="today", bug_ids=[]):
×
342
        bugs = super(NotLanded, self).get_bugs(date=date, bug_ids=bug_ids)
×
343
        bugs = self.filter_bugs(bugs)
×
344
        bugs_patch = self.get_patch_data(bugs)
×
345
        res = {}
×
346

347
        reviewers_phid = set()
×
348
        nicknames = {}
×
349
        for bugid, data in bugs_patch.items():
×
350
            reviewers_phid |= data["reviewers_phid"]
×
351
            assignee = bugs[bugid]["assigned_to"]
×
352
            if not assignee:
×
353
                assignee = max(data["author"], key=data["author"].get)
×
354
                nicknames[bugid] = assignee
×
355

356
        bz_reviewers = self.get_bz_userid(reviewers_phid)
×
357
        all_reviewers = set(bz_reviewers.keys())
×
358
        nicknames = self.get_nicks(nicknames)
×
359

360
        for bugid, data in bugs_patch.items():
×
361
            res[bugid] = d = bugs[bugid]
×
362
            self.extra_ni[bugid] = data["count"]
×
363
            assignee = d["assigned_to"]
×
364
            nickname = d["nickname"]
×
365

366
            if not assignee:
×
367
                assignee, nickname = nicknames[bugid]
×
368

369
            if not assignee:
×
370
                continue
×
371

372
            dependencies = data.get("dependencies", [])
×
373

374
            if dependencies:
×
375
                continue
×
376

377
            self.add_auto_ni(bugid, {"mail": assignee, "nickname": nickname})
×
378

379
            common = all_reviewers & data["reviewers_phid"]
×
380
            if common:
×
381
                reviewer = random.choice(list(common))
×
382
                self.add_auto_ni(
×
383
                    bugid, {"mail": bz_reviewers[reviewer], "nickname": None}
384
                )
385

386
        return res
×
387

388

389
if __name__ == "__main__":
×
390
    NotLanded().run()
×
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc