643042 - updates builder shouldn't fail when config files have been bumped

Status: RESOLVED FIXED

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

Description

[[e-mail] (:bhearsum)]

the updates builder currently fails if it has no update verify bumping to do. as long as the update-verify-bump.pl script exited successfully, it should continue even if there's onthing to commit. I think we can do this by:
- setting flunkOnFailure/haltOnFailure=True for the bumping
- setting haltOnFailure=False for the commit/push

Not sure what to do about the builder turning red still in those cases. Maybe we need to use doStepIf?

Comment 1

[[e-mail] (:bhearsum)]

Mass move of bugs to Release Automation component.

Comment 3

[[e-mail] (:bhearsum)]

This is a bit tougher than I thought because a) there's no way I can find to create a commit without any changes on a modern hg, and b) "hg push" returns non-zero when there's nothing to push.

The best I can come up with is adding supporting for skipping bumping when a certain property is set. This isn't quite as good, because it means you need to know upfront that there won't be any changes necessary, but it's better than the current state. (Worst case scenario, you try rerunning updates, it fails because it didn't bump anything, you rerun it again with the property set). To implement this, we need to:
* Add a method that checks to see if "skipBump" exists (see https://github.com/mozilla/buildbotcustom/blob/master/process/factory.py#L2224 for a similar method), and returns False if it does.
* Pass that method to all of the BuildStep's in the bumpConfigs method: https://github.com/mozilla/buildbotcustom/blob/72d08bd748b9d71a85a1fbef2e0a06b1d488824b/process/factory.py#L4187

Comment 4

[Rail Aliiev [:rail]]

fixed in release promotion