bhearsum@mozilla.com (:bhearsum) Reporter
	Description • 13 years ago

Betas should get pushed to internal mirrors automatically. This should be pretty easy, because there's no rsync modules to update (because we exclude *.*b* from mozilla-releases already). We probably need a flag in the release config to control this behaviour.

	Rail Aliiev [:rail] Assignee
	Comment 1 • 13 years ago

We can push the files after the "updates" or antivirus builder. Probably the latter is better. Any other suggestion?

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 2 • 13 years ago

Yeah, pushing after A/V is probably best for Betas.

	Rail Aliiev [:rail] Assignee
	Comment 3 • 13 years ago

For betas: we need to disable "updates available on betatest" emails and add "updates available on releasetest" emails ~10 mins after pushing the files to the internal mirrors.

	Nick Thomas [:nthomas] (UTC+12)
	Comment 4 • 13 years ago

(In reply to Rail Aliiev [:rail] from comment #3)
> For betas: we need to disable "updates available on betatest" emails and add
> "updates available on releasetest" emails ~10 mins after pushing the files
> to the internal mirrors.

The comment about releasetest confuses me, as we already get a 'ready for releasetest testing' email when we get to weight 1 in bouncer (ie when sentry polls dm-download02). +1 to disabling the betatest emails though.

	Rail Aliiev [:rail] Assignee
	Comment 5 • 13 years ago

(In reply to Nick Thomas [:nthomas] from comment #4)
> 
> The comment about releasetest confuses me, as we already get a 'ready for
> releasetest testing' email when we get to weight 1 in bouncer (ie when
> sentry polls dm-download02). +1 to disabling the betatest emails though.

Oh, right.

	Rail Aliiev [:rail] Assignee
	Comment 7 • 13 years ago

to be tested

	Rail Aliiev [:rail] Assignee
	Comment 8 • 13 years ago

Passed staging test

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 9 • 13 years ago

Comment on attachment 567924 [details] [diff] [review]
buildbotcustom

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

::: process/release.py
@@ +485,5 @@
> +    if releaseConfig.get('enableAutomaticPushToMirrors') and \
> +        releaseConfig.get('verifyConfigs'):
> +        upstream_scheduler = updates_scheduler
> +        if not releaseConfig.get('disableVirusCheck'):
> +            upstream_scheduler = antivirus_scheduler

Can you reverse this logic? Using "if not" on variable starting with "disable" is a bit confusing.

	Rail Aliiev [:rail] Assignee
	Comment 11 • 13 years ago

Comment on attachment 568055 [details] [diff] [review]
buildbotcustom

Updated patch, r=bhearsum via IRC.

	Rail Aliiev [:rail] Assignee
	Comment 12 • 13 years ago

Comment on attachment 567145 [details] [diff] [review]
configs

http://hg.mozilla.org/build/buildbot-configs/rev/40d6781359db

	Rail Aliiev [:rail] Assignee
	Comment 13 • 13 years ago

Comment on attachment 568055 [details] [diff] [review]
buildbotcustom

http://hg.mozilla.org/build/buildbotcustom/rev/67441a25106f

	Chris Cooper [:coop] (he/him)
	Comment 14 • 13 years ago

A reconfig that included this happened today.