1183106 - stop tagging mozharness repo for releases

Status: RESOLVED FIXED

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

Description

[[e-mail] (:bhearsum)]

Once all of the release jobs use in-tree Mozharness we can stop tagging the old repo.

Comment 1

[[e-mail] (:bhearsum)]

Attachment: stop tagging mozharness as part of release automation

I did some grepping and the only reference to it are in tagging, and erroneous ones caused by bug 1187894.

Comment 2

[Rail Aliiev [:rail]]

Comment on attachment 8639282 [details] [diff] [review]
stop tagging mozharness as part of release automation

\o/

Comment 3

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

How does this work with releases ? I see the likes of 'hg update -r FENNEC_40_0b4_RELEASE' in slightly old Android 'en-US' and l10n jobs. In more recent jobs we've switched to the mozharness bundles ?

bash -c 'python archiver_client.py mozharness --repo releases/mozilla-beta --tag FENNEC_40_0b8_RELEASE --destination scripts --debug'

IIRC desktop jobs are doing that too, even if almost all or all of them don't need it.

No bustage in 40.0b8 build1, but the change was only on default then.

Comment 4

[Jordan Lund (:jlund)]

(In reply to Nick Thomas [:nthomas] from comment #3)
> How does this work with releases ? I see the likes of 'hg update -r
> FENNEC_40_0b4_RELEASE' in slightly old Android 'en-US' and l10n jobs. In
> more recent jobs we've switched to the mozharness bundles ?

in the old way, we would tag the mozharness repo *and* the gecko repo with the same tag. Now in-tree mh just uses the gecko tag.

> 
> bash -c 'python archiver_client.py mozharness --repo releases/mozilla-beta
> --tag FENNEC_40_0b8_RELEASE --destination scripts --debug'

in the above example, archiver will look up the m-b rev based on the tag and grab a copy of mh that way.

so that is why we don't need any additional new tags with in-tree mh and also why we can stop tagging the old mh repo.

Does that clear up your question? Or did I misunderstand what you were asking?

Comment 5

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

Thanks for the explanation. I must have swapped places with a me from an alternate universe, and they'd forgotten everything about this change.