848899 - can't submit releases with relbranches to ship it

Status: RESOLVED FIXED

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

Description

[[e-mail] (:bhearsum)]

This is probably a regression from bug 837107, where we marked "revision" as disabled when relbranch gets set. The unintended side effect of this seems to be that no data is sent for the revision field - so it fails validation. We need to adjust the validation code to allow for an empty revision when relbranch is set. We should probably say something like "tip of relbranch" on releases.html in that case, too.

Comment 1

[[e-mail] (:bhearsum)]

Rail and I just talked about this and decided that because we ignore the revision when a relbranch is present, that we should set revision to None in this case, rather than setting it to an ignored one. We'll have to update tag-release.py (and maybe other scripts) to cope with this.

Comment 2

[[e-mail] (:bhearsum)]

Attachment: make revision not required when relbranch is present

This patch hasn't been tested yet, but I wanted to get some feedback on how I'm setting "revision" when relbranch is passed. I originally set it to a string like "tip of FOO", but given that revision is sometimes used in hg operations that seemed like a bad thing. Given that, the only sensible thing I could think to set it to was the relbranch name.

How does this sound to you two?

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 723466 [details] [diff] [review]
make revision not required when relbranch is present

Yup, using relbranch as revision makes sense, since it'll always be "tip of relbranch" - exactly what we need for releases off relbranch.

Comment 4

[[e-mail] (:bhearsum)]

Attachment: make revision/relbranch mutually exclusive; update tooltips

New in this patch:
* Tested, and works. Sample bumps are here: https://hg.mozilla.org/users/bhearsum_mozilla.com/buildbot-configs/rev/9fec8e618ece, https://hg.mozilla.org/users/bhearsum_mozilla.com/buildbot-configs/rev/393bf7504b5a
* Make revision and relbranch mutually exclusive -- relbranch is disabled when revision is present and vice versa.

I'm not happy with all the duplication in the form templates, but it's not possible to set placeholder/title/disabled until you render the form. I filed an upstream bug on this (https://bitbucket.org/simplecodes/wtforms/issue/143/should-be-able-to-store-widget-arguments), but that doesn't make anything better right now.

Comment 5

[[e-mail] (:bhearsum)]

in production.