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

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

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.

We can remove these again once the mozharness patch lands.

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 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.

(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!

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 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 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.