Closed Bug 363237 Opened 13 years ago Closed 13 years ago

bootstrap Update step should not run independent verification

Categories

(Release Engineering :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: rhelmer, Assigned: rhelmer)

References

Details

Attachments

(2 files)

Full verification should be done in a separate step, since it needs to be run after snippets are deployed. This patch removes the full, independent update verification, and leaves the quick "only test channels" verification.

The full verification will be reinstated later, in a different patch.
Assignee: build → rhelmer
Status: NEW → ASSIGNED
Attachment #248040 - Flags: review?(preed)
Comment on attachment 248040 [details] [diff] [review]
remove full/independent verification from Update step

This is fine; you said you still had this code in your personal branch, so as long as we can easily resurrect it, I'm cool with deleting it wholesale.
Attachment #248040 - Flags: review?(preed) → review+
This adds a standalone Verify step, plus Push/Announce support. I can split the latter out if you like, but it's pretty minor so I'd like to get it in as long as I'm touching "release".
Attachment #251298 - Flags: review?(preed)
Actually, going back to having this as a verification step for Update instead of a standalone step. It'll be short-term pain, but it'll make more sense in the long run when we can kick of builds and verifications remotely.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
Comment on attachment 251298 [details] [diff] [review]
add standalone Verify step, plus push/announce support

r- because this patch still has the separate Verify step in it.

Also, for the Push() and Announce() sub-classes that are effectively null, I'd either remove the Log() messages entirely, or say something about "No announce behavior defined."
Attachment #251298 - Flags: review?(preed) → review-
(In reply to comment #5)
> (From update of attachment 251298 [details] [diff] [review])
> r- because this patch still has the separate Verify step in it.

Yeah, I guess changing the resolution of the bug to INVALID does not remove review requests, sorry :(

> Also, for the Push() and Announce() sub-classes that are effectively null, I'd
> either remove the Log() messages entirely, or say something about "No announce
> behavior defined."
 

Ok, I think removing them completely is best.
QA Contact: mozpreed → build
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.