1538565 - promoting Thunderbird 60.6.1 Error: Schema Validation Failed - InputValidationError * statusCode: 400 *

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Wayne Mery (:wsmwk)]

+++ This bug was initially created as a clone of Bug #1538478 +++

(to be directed to some release or build component)

promoting Thunderbird 60.6.1 I get

Error: Schema Validation Failed! Rejecting Schema: https://schemas.taskcluster.net/queue/v1/create-task-request.json# Errors: * data.routes[2] should be string --- * method: createTask * errorCode: InputValidationError * statusCode: 400 * time: 2019-03-24T03:43:57.726Z

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

From the papertrail logging it looks like the routes are defined as ['a', 'b', ['c', 'd']]:

b'l-completed","retries":5,"routes":["tc-treeherder.v2.comm-esr60.794a7302a1cc'
b'25cd9073ece4f5eba8602efd6bb4.176","tc-treeherder-stage.v2.comm-esr60.794a730'
b'2a1cc25cd9073ece4f5eba8602efd6bb4.176",["notify.email.taskcluster-notificati'
b'[e-mail]-failed","notify.email.taskcluster-notificatio'
b'd.176.actions.SOY6RjjjSLOeRa46oby7UA"]],"schedulerId":"gecko-level-3","scope'

There was a change to .taskcluster.yml between 60.6.0 and 60.6.1 - https://hg.mozilla.org/releases/comm-esr60/rev/3d01fb2a6a9b.

Comment 2

[Jorg K (CEST = GMT+2)]

Sure, Bug 1492219 - Update .taskcluster.yml with (some) changes from M-C r=tomprince f=rail for ESR a=jorgk

Did I mess something up?

Comment 3

[Rail Aliiev [:rail]]

I just diffed comm-esr60/.taskcluster.yml against beta and looks like it'd be better to update it. I'm not the expert and haven't followed the changes in .taskcluster.yml, but the following patch may work.

https://gist.github.com/9b66137088574bb14e7cdb4748257288

Comment 4

[Rail Aliiev [:rail]]

Especially https://gist.github.com/rail/9b66137088574bb14e7cdb4748257288#file-foo-diff-L18 looks very suspicious - esr60 uses a list, while beta uses a string

Comment 5

[Jorg K (CEST = GMT+2)]

Thanks, Rail, for looking into it. I'm not very familiar with those .yml files and I don't understand your comment #4, list vs. string? As far as I can see, the first hunk of your patch aligns esr60 with beta, but that's just syntactic gloss.

The second hunk shows the missed uplift of bug 1492664 comment #53:
https://hg.mozilla.org/comm-central/rev/3409b3f717ea

So I'll just do that, right? Am I missing anything?

Comment 6

[Rail Aliiev [:rail]]

The main issue is that esr60 uses "- $if" instead of "$if", what turns a list into a list of lists. And Nick's tracebacks show that we submit a list of lists instead of a plain list.

Comment 7

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/releases/comm-esr60/rev/e2546e5f5182a80f4824b8c29f6328e4a16653c1
https://hg.mozilla.org/releases/comm-esr60/rev/ed1296af783ca23d9b07aac774cd104b14d246c3

Thanks again, fingers crossed at 2:17 AM

Comment 8

[Rail Aliiev [:rail]]

\o/wsmwk: feel free

Comment 9

[Rail Aliiev [:rail]]

I was going to say that you can feel free to start a new build when you are ready. build1 needs to be cancelled.

Comment 10

[Wayne Mery (:wsmwk)]

success