Infrastructure/distgit-bugzilla-sync
3
0
Fork 0
Code Issues 3 Pull requests Projects Releases Packages Wiki Activity Actions

Start working on generating a public diff of the changes

Browse source 
This means replacing the emails in the output with their FAS username.

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This commit is contained in:
Pierre-Yves Chibon 2019-11-21 11:28:24 +01:00 • committed by Nils Philippsen
parent 3c504653a7
commit fd66fa51c5
1 changed files with 11 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

15
distgit_bugzilla_sync/script.py
View file

@ -213,24 +213,28 @@ class BugzillaProxy:
return self.userCache[username]['bugzilla_email'].lower() return self.userCache[username]['bugzilla_email'].lower()
def add_edit_component(self, package, collection, owner, description=None, def add_edit_component(self, package, collection, owner, description=None,
qacontact=None, cclist=None): qacontact=None, cclist=None, output_user=False):
'''Add or update a component to have the values specified. '''Add or update a component to have the values specified.
''' '''
# Turn the cclist into something usable by bugzilla # Turn the cclist into something usable by bugzilla
initialCCList = [] initialCCList = []
initialCCList_username = []
for watcher in cclist: for watcher in cclist:
bz_email = self._get_bugzilla_email(watcher) bz_email = self._get_bugzilla_email(watcher)
if bz_email: if bz_email:
initialCCList.append(bz_email) initialCCList.append(bz_email)
initialCCList_username.append(watcher)
else: else:
print(f"** {watcher} has no bugzilla_email or mailing_list set " print(f"** {watcher} has no bugzilla_email or mailing_list set "
f"({collection}/{package}) **") f"({collection}/{package}) **")
# Add owner to the cclist so comaintainers taking over a bug don't # Add owner to the cclist so comaintainers taking over a bug don't
# have to do this manually # have to do this manually
owner_name = owner
owner = self._get_bugzilla_email(owner) owner = self._get_bugzilla_email(owner)
if owner not in initialCCList: if owner not in initialCCList:
initialCCList.append(owner) initialCCList.append(owner)
initialCCList_username.append(owner_name)
# Lookup product # Lookup product
try: try:
@ -278,11 +282,14 @@ class BugzillaProxy:
data['component'] = package data['component'] = package
if self.config["verbose"]: if self.config["verbose"]:
print('[EDITCOMP] %s/%s' % (data["product"], data["component"])) print(f'[EDITCOMP] {data["product"]}/{data["component"]}')
for key in ["initialowner", "description", "initialqacontact", "initialcclist"]: for key in ["initialowner", "description", "initialqacontact", "initialcclist"]:
if data.get(key): if data.get(key):
print(f" {key} changed from {product[pkgKey][key]} " if output_user and key == "initialcclist":
f"to {data.get(key)}") print(f" {key} changed from to `{initialCCList_username}`")
else:
print(f" {key} changed from `{product[pkgKey][key]} "
f"to `{data.get(key)}`")
# FIXME: initialowner has been made mandatory for some # FIXME: initialowner has been made mandatory for some
# reason. Asking dkl why. # reason. Asking dkl why.