784685 - don't depend on oldVersion in mobile release repacks

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, defect)

Description

[[e-mail] (:bhearsum)]

Aki says that we don't need the code that does this right now.

Comment 1

[[e-mail] (:bhearsum)]

Attachment: rip out old* references from android repack mozharness scripts

If we end up wanting this back we can add a partialUpdates variable to the release config instead.

Comment 2

[[e-mail] (:bhearsum)]

Comment on attachment 654231 [details] [diff] [review]
rip out old* references from android repack mozharness scripts

Aki is out this week. Can you review this, Rail? I still want Aki to look at it, but we need to land before 15.0.

Comment 3

[[e-mail] (:bhearsum)]

Attachment: remove old* again

We can remove these again once the mozharness patch lands.

Comment 4

[Rail Aliiev [:rail]]

Comment on attachment 654231 [details] [diff] [review]
rip out old* references from android repack mozharness scripts

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

::: mozharness/mozilla/release.py
@@ +50,2 @@
>                          'ftp_server', 'ftp_user', 'ftp_ssh_key',
>                          'aus_server', 'aus_user', 'aus_ssh_key',):

A nit. Could you rewrap this when land.

::: scripts/sign_android.py
@@ +103,1 @@
>       ['--buildnum',],

You can drop old_buildnum as well.

Comment 5

[Aki Sasaki (not active)]

Comment on attachment 654231 [details] [diff] [review]
rip out old* references from android repack mozharness scripts

I think this would be cleaner if we ripped out the create snippets and upload snippets actions from sign_android.py entirely, rather than have them there but crippled.

That means removing them from all_actions and removing the create_snippets() and upload_snippets() methods from sign_android.py.

I think this is an acceptable first step, however, since we're not running those two actions currently.  If you want me to write the patch to remove them entirely, I can do that.

Comment 6

[[e-mail] (:bhearsum)]

(In reply to Aki Sasaki [:aki] from comment #5)
> Comment on attachment 654231 [details] [diff] [review]
> rip out old* references from android repack mozharness scripts
> 
> I think this would be cleaner if we ripped out the create snippets and
> upload snippets actions from sign_android.py entirely, rather than have them
> there but crippled.

I'm not sure that only having partial snippets is crippling it, but I'm happy to defer to you on this. I'll take a stab at removing them entirely.

> That means removing them from all_actions and removing the create_snippets()
> and upload_snippets() methods from sign_android.py.

Thanks for the guidance!

Comment 7

[[e-mail] (:bhearsum)]

Attachment: rip out snippet support completely

I ripped out the self.fatal() when buildid isn't set, because the comment seemed to say that it's not fatal when we're not doing snippets.

Passes tests, I'm not sure how else to test it.

Comment 8

[Aki Sasaki (not active)]

Comment on attachment 654587 [details] [diff] [review]
rip out snippet support completely

I think we can rip out query_buildid() completely too, as nothing's calling it anymore.  This looks good, thanks Ben!

Comment 9

[[e-mail] (:bhearsum)]

Comment on attachment 654587 [details] [diff] [review]
rip out snippet support completely

> I think we can rip out query_buildid() completely too, as nothing's calling
> it anymore.  This looks good, thanks Ben!

I did this in the landed patch.