Closed Bug 1296685 Opened 4 years ago Closed 4 years ago

remove dead attributes from apprelease blobs

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: ninad101, Mentored)

References

Details

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

From https://bugzilla.mozilla.org/show_bug.cgi?id=1296207#c11:
> They shouldn't cause any problems if they are still there but
> platformVersion and billboardURL should be removed from AUS for the version
> that this lands for.



Not sure when this will land yet, but we go ahead and create a new apprelease blob anytime.
See Also: → 1296207
Might be a good thing for me to review what else AUS supports. A couple of other attributes that were deprecated but I don't think we have support for except for old versions are licenseURL, version, and extensionVersion.
Whiteboard: [lang=python] → [lang=python][good first bug]
Hello Ben,

I would like to work on this bug. I feel glad if you can mentor me in fixing it.

Thank You.
(In reply to Wasim Thabraze from comment #2)
> Hello Ben,
> 
> I would like to work on this bug. I feel glad if you can mentor me in fixing
> it.
> 
> Thank You.

Hi Wasim, thanks for your interest! Before you get started it would be a good idea to fork the Balrog repo (https://github.com/mozilla/balrog), clone it locally, and make sure you can run all the tests succesfully. Once you have your local repo, try running "run-test.sh" and make sure it passes.

If you have any trouble getting set-up feel free to e-mail me, or ask in irc://irc.mozilla.org/#balrog

Once you've got your tests passing, here's the rough plan for this bug:
* Create a new "blob" based on ReleaseBlobV5 (from https://github.com/mozilla/balrog/blob/7bc4d6cb0b9466c856847af5fd9a30917745df5b/auslib/blobs/apprelease.py#L650), but remove support for the platformVersion, billboardURL, licenseURL, version, and extensionVersion.
** This will involve cloning both the Python class and schema (https://github.com/mozilla/balrog/blob/7bc4d6cb0b9466c856847af5fd9a30917745df5b/auslib/blobs/schemas/apprelease-v5.yml)
* New tests will need to be added to make sure the new blob class functions correctly. They can probably be based on the V5 ones at https://github.com/mozilla/balrog/blob/7bc4d6cb0b9466c856847af5fd9a30917745df5b/auslib/test/blobs/test_apprelease.py#L1491.
* The blob factory will need updating as well (https://github.com/mozilla/balrog/blob/7bc4d6cb0b9466c856847af5fd9a30917745df5b/auslib/blobs/base.py)
Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/8d5ed73bd838094b3616815691b773acb3b80662
bug 1296685: Added an optional attribute "product" to all blob schemas (#129). r=bhearsum
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #1)
> Might be a good thing for me to review what else AUS supports. A couple of
> other attributes that were deprecated but I don't think we have support for
> except for old versions are licenseURL, version, and extensionVersion.

Just to double check - it's okay to remove these 3 attributes as well? Anything else you can think of?
Flags: needinfo?(robert.strong.bugs)
(In reply to [github robot] from comment #4)
> Commit pushed to master at https://github.com/mozilla/balrog
> 
> https://github.com/mozilla/balrog/commit/
> 8d5ed73bd838094b3616815691b773acb3b80662
> bug 1296685: Added an optional attribute "product" to all blob schemas
> (#129). r=bhearsum

This was meant for a different bug...
(In reply to Ben Hearsum (:bhearsum) from comment #5)
> (In reply to Robert Strong [:rstrong] (use needinfo to contact me) from
> comment #1)
> > Might be a good thing for me to review what else AUS supports. A couple of
> > other attributes that were deprecated but I don't think we have support for
> > except for old versions are licenseURL, version, and extensionVersion.
> 
> Just to double check - it's okay to remove these 3 attributes as well?
> Anything else you can think of?
It is okay and I'd go with Firefox 51 and above.
I can't think of anything else that can be removed.
Flags: needinfo?(robert.strong.bugs)
Assignee: nobody → bhat.ninadmb
Summary: remove platformVersion and billboardURL from apprelease blobs → remove dead attributes from apprelease blobs
Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/a9681be962001b1f973b0b38b6cdf11ad96e0941
bug 1296685: remove dead attributes from apprelease blobs (#130). r=bhearsum
Depends on: 1305703
This is in production now, thanks Ninad!
Status: NEW → RESOLVED
Closed: 4 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.