add BUG_COMPONENT to services/* files

RESOLVED FIXED in Firefox 55

Status

()

Firefox
Sync
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

unspecified
Firefox 55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

9 months ago
this will help match bugs up to the right team if needed.
(Assignee)

Comment 1

9 months ago
Created attachment 8845023 [details] [diff] [review]
add BUG_COMPONENT to services/*

:rfkelly- can you verify the fxAccounts
:leplatrem - can you verify the cloud services stuff

here is the final list:
$ ./mach file-info bugzilla-component services/**
Cloud Services :: Firefox: Common
  services/blocklists
Cloud Services :: cloudSync
  services/cloudsync
Core :: Build Config
  services/moz.build
Core :: fxAccounts
  services/fxaccounts
Mozilla Services :: Firefox Sync: Backend
  services/sync
Mozilla Services :: Firefox Sync: Crypto
  services/crypto
Mozilla Services :: Firefox: Common
  services/common


* NOTE: some of this was done prior to me working on this.
Attachment #8845023 - Flags: review?(rfkelly)
Attachment #8845023 - Flags: review?(mathieu)
Thanks, Joel and Richard! For Sync, we're trying to move away from the "Cloud Services :: Firefox Sync: Backend" (and "Firefox Sync: Crypto") components, and to "Firefox :: Sync". (We triage all of them just in case, but this might be a good time to change them, too). The others look great.
(Assignee)

Updated

9 months ago
Component: General → Sync
Product: Cloud Services → Firefox
(Assignee)

Comment 3

9 months ago
Created attachment 8845035 [details] [diff] [review]
add BUG_COMPONENT to services/*

thanks :kit, I have updated the patch accordingly (and realized the existing stuff referenced old components)
Assignee: nobody → jmaher
Attachment #8845023 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8845023 - Flags: review?(rfkelly)
Attachment #8845023 - Flags: review?(mathieu)
Attachment #8845035 - Flags: review?(rfkelly)
Attachment #8845035 - Flags: review?(kit)

Comment 4

9 months ago
Comment on attachment 8845035 [details] [diff] [review]
add BUG_COMPONENT to services/*

Review of attachment 8845035 [details] [diff] [review]:
-----------------------------------------------------------------

The FxA part looks good to me, thanks!
Attachment #8845035 - Flags: review?(rfkelly) → review+
Comment on attachment 8845035 [details] [diff] [review]
add BUG_COMPONENT to services/*

Review of attachment 8845035 [details] [diff] [review]:
-----------------------------------------------------------------

Whoops, sorry I missed the r? Looks great; thanks, Joel!
Attachment #8845035 - Flags: review?(kit) → review+

Comment 6

9 months ago
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a1f96cfcb21a
add BUG_COMPONENT to services/* files. r=rfkelly, kit

Comment 7

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a1f96cfcb21a
Status: ASSIGNED → RESOLVED
Last Resolved: 9 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Sorry Joel to arrive late to the party :]

It's not crucial, but I would have put ``services/blocklists`` under *Firefox: Common*, or *Toolkit: blocklisting*, but not *Firefox :: Sync*.
(Assignee)

Comment 9

9 months ago
Created attachment 8845963 [details] [diff] [review]
adjust BUG_COMPONENT for services/blocklists/*

sure, lets just adjust it, here is what we end up with:
$ ./mach file-info bugzilla-component services/**
Cloud Services :: Firefox: Common
  services/common
Cloud Services :: cloudSync
  services/cloudsync
Core :: Build Config
  services/moz.build
Core :: fxAccounts
  services/fxaccounts
Firefox :: Sync
  services/crypto
  services/sync
Toolkit :: Blocklisting
  services/blocklists
Attachment #8845963 - Flags: review?(mathieu)
Attachment #8845963 - Flags: review?(mathieu) → review+

Comment 10

9 months ago
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/388514801752
adjust BUG_COMPONENT for services/blocklist/* files. r=leplatrem

Comment 11

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/388514801752
You need to log in before you can comment on or make changes to this bug.