Open Bug 890429 Opened 6 years ago Updated 6 years ago

SeaMonkey master failing to upload log files


(SeaMonkey :: Release Engineering, defect)

Windows 7
Not set


(Not tracked)


(Reporter: Callek, Assigned: Callek)



(1 file)

So, postrun.cfg hardcodes ffxbld, this is bad and seamonkey needs a different postrun

This file is managed by puppet so we need some magic here.

I can think of a few possible ways this patch's needs would make sense here. This is the one I chose but if you prefer a different option feel free to tell me and we can chat.
Attachment #771527 - Flags: review?(dustin)
Comment on attachment 771527 [details] [diff] [review]
[puppet] postrun.cfg by $org

Review of attachment 771527 [details] [diff] [review]:

This is fine, but I'd rather use $::config::org in the path to the template, e.g., template("${module_name}/${::config::org}/postrun.cfg").  Then it will fail for any org that hasn't set up a postrun.cfg.
Attachment #771527 - Flags: review?(dustin) → review+
per :dustin this could land without making the adjustment, but leaving open in prep for a better fix.
You need to log in before you can comment on or make changes to this bug.