Opened on 09/19/2017 at 12:39:26 PM

Closed on 09/19/2017 at 02:23:41 PM

#5736 closed defect (fixed)

Release automation aborts without reason

Reported by: tlucas Assignee:
Priority: P1 Milestone:
Module: Automation Keywords:
Cc: kzar, sebastian, fhd, trev, Ross Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by tlucas)

This is a regression, introduced with #4354 / #4355, git:c6f892d / hg:ef4ad61830eb

How to reproduce

Run the release automation for a setup, which should succeed (no uncommitted / outgoing / incoming changes, see #4353 / #4355) for e.g. adblockpluschrome:

  1. Clone the adblockpluschrome repository (or reset your copy to master)
  2. Clone the downloads repository (or reset your copy to master)
  3. Run the release automation:

./ -t chrome release -k <certificate> 1.14.0

(Important: configure fake remotes for your adblockpluschrome and downloads repository)

Observed behaviour

The release aborts.

Expected behaviour

The release should succeed, updating both the adblockpluschrome and downloads repository

Hints for testers

To setup fake remotes for the repositories, you can follow these steps:

  • Clone the adblockpluschrome repository to a location of your choice hg clone
  • Clone the downloads repository to a location of your choice (i suggest the same location as abp-chrome - otherwise you would have to specify the downloads location in the build process) hg clone
  • Create the fake remotes for adblockpluschrome and downloads:
    • cp -ar adblockpluschrome abpc_remote
    • cp -ar downloads downloads_remote
    • Configure your main repositories to have the fake remotes as remotes: change .hg/hgrc in both main repositories to contain default = <path_to_fake_remote> e.g.: default = /home/tlucas/abp/tmphg/abpc_remote
    • you can verify this by running hg incoming in the main repositories, it should say "comparing with <your_fake_remote>"
  • Create your own .pem (this requires openssl to be installed) in a location of your choice: openssl req -newkey rsa:2048 -nodes -x509 -days 365 -out certificate.pem - additionally to the certificate.pem, this creates a key.pem, which is the file that we need.
  • run the release in your adblockpluschrome main repository: ./ -t chrome release -k <path_to_new_key.pem> 1.14.0

Attachments (0)

Change History (6)

comment:1 Changed on 09/19/2017 at 12:44:59 PM by kzar

  • Cc Ross added
  • Priority changed from Unknown to P1
  • Ready set

From the discussion in IRC this sounds like a recent regression. Please could you bisect recent changes to figure out which commit is to blame, and then mention that in the description of this issue? That's usually a good place to start with these kind of problems.

comment:2 Changed on 09/19/2017 at 01:05:40 PM by tlucas

  • Description modified (diff)

comment:3 Changed on 09/19/2017 at 01:12:51 PM by tlucas

  • Description modified (diff)

comment:4 Changed on 09/19/2017 at 01:14:11 PM by tlucas

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:5 Changed on 09/19/2017 at 02:17:34 PM by abpbot

A commit referencing this issue has landed:
Issue 5736 - Fix causeless abort of release

comment:6 Changed on 09/19/2017 at 02:23:41 PM by tlucas

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Note: See TracTickets for help on using tickets.