Closed
Bug 1062557
Opened 11 years ago
Closed 11 years ago
incorrect try submission summary email
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: keeler, Assigned: poiru)
References
Details
Attachments
(1 file, 1 obsolete file)
3.94 KB,
patch
|
catlee
:
review+
poiru
:
checked-in+
|
Details | Diff | Splinter Review |
I just pushed to try ( https://tbpl.mozilla.org/?tree=Try&rev=9166806dc1d3 ) and received a "thanks for your try submission" email. Unfortunately, it didn't have the correct summary. It should have said something like this:
Summary:
* try: -b do -p macosx64,win32,linux32_gecko -u xpcshell -t none
* bug 1057123 - mozilla::pkix: certificates with Key Usage asserting the keyCertSign bit may act as end-entities
Instead, it said:
Summary:
* rm chromium debug utils 4 1 try: -b do -p all -u all -t none
* Merge inbound to m-c. a=merge
* Backed out changeset d5cd910dda69 (bug 1033098) for OSX assertions.
* Bug 1061618 - Extend MOZ_UPDATER ifdef to fix blank pages on about:preferences. r=Gijs
* Bug 1050214 - Add shuffle's masks constants to the SIMD object. r=bbouvier
* Bug 1047101 - Provide a way to do data-driven tests with Marionette test runner. r=jgriffin
* Bug 1032777 - Include base image version for Flame device. r=wlachance
* Bug 992894 - Add xpcshell tests for the NativeFileWatcher. r=Yoric
* Bug 992894 - Add a native file watcher component to notify about file/directory changes (Windows).
* Bug 1061028 - IsInlineFrame() is a synonym for IsFrameOfType(nsIFrame::eLineParticipant) so it
* Bug 1061027 - Remove bogus (and unnecessary) static_cast. r=roc
Comment 1•11 years ago
|
||
Those commits are from the previous push to try:
https://tbpl.mozilla.org/?tree=Try&rev=15f443714732
Comment 2•11 years ago
|
||
I think the initialization of the commit_titles etc. needs to be moved inside the loop here:
http://hg.mozilla.org/build/buildbotcustom/file/default/changes/hgpoller.py#l275
Assignee | ||
Comment 4•11 years ago
|
||
This should (probably) do it.
Comment 5•11 years ago
|
||
Will this fix the case in bug 1061359 where the additional commits mentioned in the email that shouldn't be there came from a *later* push?
Comment 6•11 years ago
|
||
Comment on attachment 8483788 [details] [diff] [review]
Ignore commits from older pushes when gathering commit titles
Review of attachment 8483788 [details] [diff] [review]:
-----------------------------------------------------------------
hm, I don't think this is right. We could legitimately be looking at multiple try pushes at once here, and so need to treat them separately.
Attachment #8483788 -
Flags: review?(catlee) → review-
Comment 8•11 years ago
|
||
FTR, the incorrect summary in my email came from newer pushes than mine.
Assignee | ||
Comment 11•11 years ago
|
||
(In reply to Cameron McCormack (:heycam) from comment #5)
> Will this fix the case in bug 1061359 where the additional commits mentioned
> in the email that shouldn't be there came from a *later* push?
This patch should handle that.
Attachment #8483788 -
Attachment is obsolete: true
Attachment #8492731 -
Flags: review?(catlee)
Comment 13•11 years ago
|
||
:catlee, given the dev confusion caused by this bug since Bug 1048322 landed, and the many dupes generated, I'm starting to think we should backout 1048322 until we have the proper fix ready.
The idea is sound and a GOOD thing, but the current bugs make it more harmful than helpful imho. Agree/Disagree?
Flags: needinfo?(catlee)
Updated•11 years ago
|
Attachment #8492731 -
Flags: review?(catlee) → review+
Comment 14•11 years ago
|
||
(In reply to Justin Wood (:Callek) from comment #13)
> :catlee, given the dev confusion caused by this bug since Bug 1048322
> landed, and the many dupes generated, I'm starting to think we should
> backout 1048322 until we have the proper fix ready.
>
> The idea is sound and a GOOD thing, but the current bugs make it more
> harmful than helpful imho. Agree/Disagree?
I think the current patch should fix it. Let's see how that goes, and then re-evaluate then.
Flags: needinfo?(catlee)
Assignee | ||
Comment 15•11 years ago
|
||
Comment on attachment 8492731 [details] [diff] [review]
Make commit titles push specific
https://hg.mozilla.org/build/buildbotcustom/rev/79649203c361
Attachment #8492731 -
Flags: checked-in+
Comment 16•11 years ago
|
||
Merged to production, and deployed.
Assignee | ||
Comment 17•11 years ago
|
||
This should be fixed now. Please reopen if the problem still exists.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•