1079639 - Create a tool to submit modifications to blobs

Status: RESOLVED FIXED

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

Description

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

In lieu of being able to edit on the server, lets make it easier to submit just the bit of json you want to change.

Comment 1

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

Attachment: [tools] balrog-tweaker.py

What do you think of this Ben ? It helped me out with adding another entry to ftpFilenames (for the partial from 33.0b9). Not so wild about how it gets the product, but the api doesn't return in a GET.

Comment 2

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

Called like this
 python scripts/updates/balrog-tweaker.py --json json -b 'Firefox-33.0-build1' --api-root 'https://aus4-admin.mozilla.org'  --credentials-file cred -u '[e-mail]' -v

so you need to set up the credentials file and the json beforehand.

Comment 3

[[e-mail] (:bhearsum)]

Comment on attachment 8501494 [details] [diff] [review]
[tools] balrog-tweaker.py

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

(In reply to Nick Thomas [:nthomas] from comment #1)
> Created attachment 8501494 [details] [diff] [review]
> [tools] balrog-tweaker.py
> 
> What do you think of this Ben ? It helped me out with adding another entry
> to ftpFilenames (for the partial from 33.0b9). Not so wild about how it gets
> the product, but the api doesn't return in a GET.

One option, of course, is to have the api return the product. If you don't want to go to that trouble (I don't blame you), you could always require it on the command line - that's probably better than guessing.

This looks great overall though!

Comment 4

[[e-mail] (:bhearsum)]

I tried this out today and discovered that it doesn't work with the edge where you need to remove a key. In my case, I had Thunderbird-12.0-build1 as schema 2, and wanted to convert it to schema 1. This failed because displayVersion/platformVersion/appVersion were all in the blob still. I ended up hacking up your code to remove them in BlobTweaker.run() after the update. This still failed, because the server has the same bug (or feature, depending how you look at it): https://github.com/mozilla/balrog/blob/master/auslib/admin/views/releases.py#L262

I still think this work is totally valid and helpful, but I thought it was worth noting!

Comment 5

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

Comment on attachment 8501494 [details] [diff] [review]
[tools] balrog-tweaker.py

Hmm, that's a bit of a pain. I guess we'd need PUT to not complain if a release already exists, or something.

Upgrading to an r? because I don't have anything to add to this.

Comment 6

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

Comment on attachment 8501494 [details] [diff] [review]
[tools] balrog-tweaker.py

https://hg.mozilla.org/build/tools/rev/be3dbae13e9f

Comment 7

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

Quick docs - https://wiki.mozilla.org/Balrog/Scripts