Closed Bug 1170919 Opened 8 years ago Closed 7 years ago

When a channel is present in fileUrls only use that block to look for products


(Release Engineering Graveyard :: Applications: Balrog (backend), defect)

Not set


(Not tracked)



(Reporter: nthomas, Assigned: njirap, Mentored)



(Whiteboard: [lang=python][good first bug])


(1 file)

Split out from bug 1168780.

(In reply to Ben Hearsum [:bhearsum] from comment #5)
> (In reply to Nick Thomas [:nthomas] from comment #4)
> > (In reply to Nick Thomas [:nthomas] from comment #0)
> > >
> > > * we should consider if we shouldn't fallback if the channel is a key in
> > > fileUrls
> > 
> > Ben, what do you think of the second point ?  It seems weird to me that if
> > s{omething is missing in the beta partials block then we go and look in the *
> > block, which otherwise is for releases.
> I totally agree with you, falling back to * when the channel is _explicitly_
> specified is wrong.

Possibly something like this:
diff --git a/auslib/blobs/ b/auslib/blobs/
index 473da46..ddc6337 100644
--- a/auslib/blobs/
+++ b/auslib/blobs/
@@ -698,8 +701,9 @@ class UnifiedFileUrlsMixin(object):
             url = None
             for c in channels:
-                url = self.get("fileUrls", {}).get(c, {}).get(patchKey, {}).get(from_)
-                if url:
+                config_block = self.get("fileUrls", {}).get(c, {})
+                if config_block:
+                    url = config_block.get(patchKey, {}).get(from_)

We'll need some tests too.
Mentor: bhearsum
Whiteboard: [lang=python][good first bug]
Assignee: nobody → njirap
Attached file njira's implentation
Nick, I tagged you in the PR but I'm not sure if you pay attention to Github notifications so I'm creating this too. The patch for this is risky, and I'd appreciate a second set of eyes on it.
Attachment #8797685 - Flags: review?(nthomas)
Attachment #8797685 - Flags: review?(nthomas) → review+
Commit pushed to master at
bug 1170919: Raise value error of channel not in 'fileUrls' (#138). r=bhearsum
This is in production. Thanks Njira!
Closed: 7 years ago
Resolution: --- → FIXED
Product: Release Engineering → Release Engineering Graveyard
You need to log in before you can comment on or make changes to this bug.