665020 - Define exchange format for hg notifications

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P3)

Description

[Chris AtLee [:catlee]]

We're going to be receiving notifications of hg pushes from the hg mirrors once they're set up.

We need to define what information about a push we need, and in what format it should be sent.

Would running 'buildbot sendchange' work here?

We need:
- push time
- change author
- list of files changed
- change comments

I don't know if all of the changes in a push should be sent at once, or if they can be sent individually.

Comment 1

[Chris Cooper [:coop] (he/him)]

Any update here, knowing full well the catlee is currently on PTO?

Comment 2

[Dustin J. Mitchell [:dustin] (he/him)]

Update from who?

Comment 3

[Chris Cooper [:coop] (he/him)]

(In reply to Dustin J. Mitchell [:dustin] from comment #2)
> Update from who?

I'm hoping that someone who follows [e-mail] had a more in-depth discussion with catlee about it at the all-hands than I did. Mostly it's just getting myself on the cc list so I can remember to ask catlee about it when he returns.

Comment 4

[Chris AtLee [:catlee]]

since we're going to be using rabbit as our notification, let's use the same format as the json full pushlog does:

{
 "2797": {
  "date": 1319075630, 
  "changesets": [
   {
    "node": "f83e3947a7cc283daa3c5b9fe43e188c4d5d4cfb", 
    "files": [
     "layout/style/nsCSSParser.cpp", 
     "layout/style/nsCSSScanner.cpp", 
     "layout/style/nsCSSScanner.h"
    ], 
    "tags": [], 
    "author": "Zack Weinberg <[e-mail]>", 
    "branch": "default", 
    "desc": "Bug 543151 part 1: Remove stream-processing code from nsCSSScanner.cpp. r=dbaron"
   }, 
   {
    "node": "6d0fded8a2b3c01b6d8cec4c86facbf1c01cbf41", 
    "files": [
     "layout/style/nsCSSScanner.cpp"
    ], 
    "tags": [], 
    "author": "Zack Weinberg <[e-mail]>", 
    "branch": "default", 
    "desc": "Bug 543151 part 2: Remove the special (but inconsistent with the error console) handling of hard tabs; small tidiness tweaks. r=dbaron"
   }, 
   {
    "node": "67673422f7d2eb344b763465c738d83cc5e8480c", 
    "files": [
     "layout/style/nsCSSParser.cpp", 
     "layout/style/nsCSSParser.h", 
     "layout/style/nsCSSRules.cpp", 
     "layout/style/nsCSSScanner.cpp", 
     "layout/style/nsCSSScanner.h", 
     "layout/style/nsCSSValue.cpp", 
     "layout/style/nsCSSValue.h", 
     "layout/style/nsIMediaList.h"
    ], 
    "tags": [], 
    "author": "Zack Weinberg <[e-mail]>", 
    "branch": "default", 
    "desc": "Bug 659963: consistently use infallible malloc in the CSS parser. r=dbaron"
   }
  ], 
  "user": "[e-mail]"
 }
}